-
Notifications
You must be signed in to change notification settings - Fork 15
Fix pypi install #22
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
Fix pypi install #22
Conversation
I may have done something incorrectly but I am getting Error upon importing still after attempting to test this fix in the same way I did for Blinka_Displayio repo. I pulled this branch and ran
But afterward attempting to import I get this error:
|
One more quick note: Both imports do work correctly if I start the python3 shell inside of this repo's directory. Only after the pip3 install that the imports are not working. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confirmed imports are working correctly for me after pip3 install .
with the latest commit.
Updating https://github.com/adafruit/Adafruit_CircuitPython_BluefruitConnect to 1.1.3 from 1.1.2: > Merge pull request adafruit/Adafruit_CircuitPython_BluefruitConnect#22 from dhalbert/pypi-packages Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Text to 2.7.3 from 2.7.2: > Merge pull request adafruit/Adafruit_CircuitPython_Display_Text#53 from makermelissa/master Updating https://github.com/adafruit/Adafruit_CircuitPython_PyBadger to 3.0.0 from 2.2.2: > Merge pull request adafruit/Adafruit_CircuitPython_PyBadger#34 from FoamyGuy/adding_pewpewm4_support
This is a multifile package, but setup.py thought it was a single-file module.
@makermelissa This is essentially the same fix you just found for blinka displayio.