Skip to content
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

Latest image complains about missing sudo and exits right away #19

Open
jackxuhong opened this issue Apr 25, 2020 · 4 comments · May be fixed by #20
Open

Latest image complains about missing sudo and exits right away #19

jackxuhong opened this issue Apr 25, 2020 · 4 comments · May be fixed by #20

Comments

@jackxuhong
Copy link

The latest image doesn't seem to work, please advise, thanks.

$ docker run -it -p 25565:25565 ashdev/minecraft-spigot:latest
/spigot_init.sh: line 6: sudo: command not found
QuietKerb added a commit to QuietKerb/docker-spigot that referenced this issue Apr 27, 2020
sudo isn't installed by default on newer images of ubuntu
fixes AshDevFr#19
@QuietKerb QuietKerb linked a pull request Apr 27, 2020 that will close this issue
@Nykaer
Copy link

Nykaer commented Apr 28, 2020

Just tested on ubuntu and a Centos hostOS. sudo present on both systems, and path seems ok.

Still throws 'sudo not found' error

@jackxuhong
Copy link
Author

Nykaer, sudo is missing in the docker image, not the host system.

@Nykaer
Copy link

Nykaer commented May 2, 2020

Yup, and I believe 'RUN apt-get install -y software-properties-common sudo' should fix that. But it doesn't.

@QuietKerb
Copy link

QuietKerb commented May 9, 2020

Thanks Nykaer,

You're right my fix was not working, I had actually used apt to test which does work.
I've updated my pull request to use apt instead as apt-get was getting 404s on some repos when installing sudo dependencies by the looks of it.

This is the change you can make: 8e582a8

raphaelreyna added a commit to raphaelreyna/docker-spigot that referenced this issue Jul 24, 2020
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 a pull request may close this issue.

3 participants