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

comment de udev section to pass the setup.py #16

Closed
wants to merge 1 commit into from

Conversation

clemaitre58
Copy link

Here some changes in order to install properly when you use anaconda python distribution.

without this comment in the setup.py (udev), you can't install with sudo python setup.py install. In fact, the installer select system python and the anaconda python.

So the solution is :

  • comment line 120 in setup.py
  • install with python setup.py install (without sudo)
  • install manually :

sudo cp resources/ant-usb-sticks.rules /etc/udev/rules.d
sudo udevadm control --reload-rules
sudo udevadm trigger --subsystem-match=usb --attr-match=idVendor=0fcf --action=add

@coveralls
Copy link

Coverage Status

Coverage remained the same at 51.621% when pulling d6ca313 on clemaitre58:master into ba4c276 on Tigge:master.

@Tigge
Copy link
Owner

Tigge commented Apr 2, 2016

Hi! Unfortunately this will break most other distributions which then all would have to rely on manually installing the udev rules. Is there perhaps another solution to make this work that will not remove existing functionality from other distributions?

@glemaitre
Copy link

What about calling the udev install using os.system('sudo whatever_command'). It will call the default python - conda for anaconda user and system for whatever distribution - and will ask for the super user password at install time.

@ironhouzi
Copy link

A non-ideal workaround is to use sudo -E python setup.py install. Copies the udev rules, but litters your virtualenv directory with a few root owned files (PKG-INFO and txt-files).

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.

5 participants