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

Sudo required while running coala/base using docker #513

Closed
newbazz opened this issue Jan 13, 2018 · 14 comments
Closed

Sudo required while running coala/base using docker #513

newbazz opened this issue Jan 13, 2018 · 14 comments

Comments

@newbazz
Copy link
Contributor

newbazz commented Jan 13, 2018

difficulty/newcomer

@gitmate-bot
Copy link

Hi, I'm GitMate.io!

It seems you've just enabled the issue triaging. I'm just scraping all issues from your repository and will give you some more information about this in a few minutes or so.

Because of the rate limit we can't scrape all information (including all comments and authors) right now - our system is already set up to scrape this in the next days over which the predictions will become more precise every day.

If you want me to use a different account for triaging your issues, simply create one and log in with it.

Sit tight!

@Makman2
Copy link
Member

Makman2 commented Jan 13, 2018

@newbazz what do you want to change specifically actually?

@CaptainDredge
Copy link

I think newbazz wanted to change some command in documentation which requires sudo to be written but he has not mentioned where he want changes to be done!

@Makman2
Copy link
Member

Makman2 commented Jan 13, 2018

we should be always careful with sudo because it depends on the environment. Some don't even have sudo available, or they already have root rights. This really depends on the system^^

@CaptainDredge
Copy link

Yup! you're right. Let newbazz clear what he wants then!

@Makman2
Copy link
Member

Makman2 commented Jan 13, 2018

And I think it's usually safe-explanatory to use root-rights if something is globally installed. And actually the usual docker installation involves to add a package repository, which forwards this problem to the package manager used.

@vibhanshuvaibhav
Copy link

@newbazz I am a newcomer and with more specific details about the changes to be made in coala/documentation, I would be able to work on this.

@newbazz
Copy link
Contributor Author

newbazz commented Jan 14, 2018

@Makman2 while i was making a fresh install using docker image, it always gave some permission denied error and I then had to run it using sudo so i filed this issue ;P

@Makman2
Copy link
Member

Makman2 commented Jan 14, 2018

Don't we defer the docker installation instructions to the official docker documentation?

@Makman2
Copy link
Member

Makman2 commented Jan 14, 2018

And are you talking actually about running the coala/base image or installing docker itself?

@newbazz
Copy link
Contributor Author

newbazz commented Jan 14, 2018

running the coala/base image.

@Makman2 Makman2 changed the title Sudo required while installation using docker Sudo required while running coala/base using docker Jan 14, 2018
@Makman2
Copy link
Member

Makman2 commented Jan 14, 2018

That shouldn't be written in the docs with sudo. Usually users add their username to the docker group so they can access docker functionality without being root, this is very common.

I would vote for closing this issue as invalid, this highly depends on the user setup, and people should read the official docker docs.

@newbazz what do you think? ;)

@newbazz
Copy link
Contributor Author

newbazz commented Jan 14, 2018

@Makman2 yeah i agree in your view. I am than closing it.

@newbazz
Copy link
Contributor Author

newbazz commented Jan 14, 2018

Closing due to invalid issue.

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

No branches or pull requests

5 participants