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

Needs sudo to install #237

Merged
merged 2 commits into from
Dec 5, 2015
Merged

Needs sudo to install #237

merged 2 commits into from
Dec 5, 2015

Conversation

davetcoleman
Copy link
Contributor

Otherwise I get the following errors:

error: could not create '/usr/local/lib/python2.7/dist-packages/tests': Permission denied

and then

error: could not delete '/etc/bash_completion.d/catkin_tools-completion.bash': Permission denied

Otherwise I get the following errors:

    error: could not create '/usr/local/lib/python2.7/dist-packages/tests': Permission denied

and then 

    error: could not delete '/etc/bash_completion.d/catkin_tools-completion.bash': Permission denied
@wjwwood
Copy link
Member

wjwwood commented Dec 5, 2015

It only needs sudo on some machines and depending on how you install it (you wouldn't need it in a virtual env). I'd prefer that there is a notice encouraging uses to use sudo if they get permission problems, rather than suggesting people always use it.

@davetcoleman
Copy link
Contributor Author

I'm using what most everyone is using: Ubuntu 14.04. But sure, I've updated the commit

@wjwwood
Copy link
Member

wjwwood commented Dec 5, 2015

Thanks. (I'll have to fix the CI in a separate PR, something in travis changed).

wjwwood added a commit that referenced this pull request Dec 5, 2015
@wjwwood wjwwood merged commit 1730e12 into catkin:master Dec 5, 2015
@wjwwood
Copy link
Member

wjwwood commented Dec 5, 2015

Also, if you're on Ubuntu you should use sudo apt-get install python-catkin-tools. If you're invoking the setup.py directly then you're either not on Ubuntu or you're installing a development version from source (in which case you should be using a virtualenv). For both cases, most of the time using sudo would not be correct and it might lead developers to think they need sudo even when they actually don't which is misleading.

@davetcoleman
Copy link
Contributor Author

Correct, I was attempting to test @jbohren's #196

in which case you should be using a virtualenv

What kind of virutalenv are you referring to? Docker?

@davetcoleman davetcoleman deleted the patch-3 branch December 5, 2015 20:51
@wjwwood
Copy link
Member

wjwwood commented Dec 5, 2015

@davetcoleman just search for virtualenv.

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.

2 participants