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

Now uses wiringpi2 instead of wiringpi. #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bdrobinson
Copy link

I made this change because the install instructions in README.md failed (it wouldn't let me build wiringpi-python as I didn't have a necessary C header or something). I thought it'd be nice to get the module working in a more straightforward way, using pip to install wiringpi2 (which seems to have superceded wiringpi) rather than the git submodule method in the readme.

This commit means you can just run sudo pip install wiringpi2. The only relevant API difference seems to be that wiringpi.OUTPUT is now wiringpi2.GPIO.OUTPUT in wiringpi2.

I've successfully run a "real world test" with an RF transmitter on my Pi B+, and I also successfully ran the unit test for the RF sender. I'm not able to test the receiver as I don't have a receiver hardware module.

Hope this helps! If the changes I've made don't make sense then that's cool - let me know if I've unintentionally broken anything. This library rocks!

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.

1 participant