-
Notifications
You must be signed in to change notification settings - Fork 7
use ubuntu 24.04 for the simple image #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
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. |
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 :-) |
Yeah, beyond size though, we would need to PR anything currently using this image to change the path to the python interpreter from |
To be fair, after a very quick glance, that might be the only spot. |
To be safe, I added a symlink for backwards compatibility! |
This reverts commit 153f0a1.
I'm curious is there a reason behind not using python in |
It's time to YOLO!!!!! |
This just broke CI @zardus |
It also broke at least 2 people's private dojos. |
Rolled back for now :-) |
The existing simple image is too weird. Python in /usr/local/bin? That's lame. Let's use use ubuntu?