search

Gallopsled / pwntools

CTF framework and exploit development library
http://pwntools.com
Other
11.99k stars 1.7k forks source link

remove python2 shabangs #2301

Closed xambroz closed 10 months ago

xambroz commented 10 months ago

Pwntools Pull Request

In the pwnlib there were handfull of modules being called from the commandlint pwn tools. As these are called from the lib by the pwn executable or wrappers, the executable shabang is not needed.

peace-maker commented 10 months ago

The tools can be ran directly from the source directory, which is useful during development. So it might be better to just switch them to python3 instead! Although the __init__.py file doesn't need one either way.

Arusekk commented 10 months ago

I think the files can be made without shebangs, as most of them are not executable anyway; some of them actually are — please make them not executable so we can merge this.

peace-maker commented 10 months ago

Ah, right. You can just run them through python manually too during development.