Skip to content

Conversation

zardus
Copy link
Member

@zardus zardus commented Sep 5, 2025

The existing simple image is too weird. Python in /usr/local/bin? That's lame. Let's use use ubuntu?

@ConnorNelson
Copy link
Member

I wanted just python and exec-suid in this image, and I wanted the image to be tiny.

Probably this is decently bigger, and also suddenly it comes with two python packages. Should we install anything else too? Or create another Dockerfile? :)

Nix nix nix.

On a more serious note (or at least on a less poke poke poke note), this repo’s contents/purpose now also belongs in pwncollege/challenges.

@zardus
Copy link
Member Author

zardus commented Sep 5, 2025

I removed the two packages. I do think that having python randomly in /usr/local/bin has challenge impact, and while I agree that eventually (very soon!) this repo should no longer be necessary, I think we should have an image here that has python in /usr/bin. I think we'll survive the extra 30mb or so of the image :-)

@ConnorNelson
Copy link
Member

Yeah, beyond size though, we would need to PR anything currently using this image to change the path to the python interpreter from /usr/local/bin/python to /usr/bin/python, e.g.:

https://github.com/pwncollege/example-dojo/blob/10b0b68c57b87c7b963b50e01f279797e10d0c4c/hello/apple/apple#L1

@ConnorNelson
Copy link
Member

To be fair, after a very quick glance, that might be the only spot.

@zardus
Copy link
Member Author

zardus commented Sep 5, 2025

To be safe, I added a symlink for backwards compatibility!

@snowy-fluff
Copy link

I'm curious is there a reason behind not using python in /usr/local/bin/python? Without it wouldn't pip install packages directly to /usr/bin/python, aka system python?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants