8000 Building micropython master fails if pypi / mpy_cross package is installed. · Issue #9255 · micropython/micropython · GitHub
[go: up one dir, main page]
Skip to content

Building micropython master fails if pypi / mpy_cross package is installed. #9255

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.

Sign up for GitHub

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

Jump to bottom
Closed
andrewleech opened this issue Sep 9, 2022 · 0 comments · Fixed by #9256
Closed

Building micropython master fails if pypi / mpy_cross package is installed. #9255

andrewleech opened this issue Sep 9, 2022 · 0 comments · Fixed by #9256
Labels
bug

Comments

@andrewleech
Copy link
Contributor

With the merge of #8914 there's now a dependency on import mpy_cross to build micropython.

If a user already has my python mpy_cross package installed however it gets picked up instead, throwing the error during make:

Traceback (most recent call last):
  File "../../tools/makemanifest.py", line 197, in main
    mpy_cross.compile(
AttributeError: module 'mpy_cross' has no attribute 'compile'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "../../tools/makemanifest.py", line 264, in <module>
    main()
  File "../../tools/makemanifest.py", line 205, in main
    except mpy_cross.CrossCompileError as ex:
AttributeError: module 'mpy_cross' has no attribute 'CrossCompileError'

The fix currently is to pip uninstall mpy_cross but we should look at publishing the new mpy_cross package / wrapper to pypi directly from here?
@andrewleech andrewleech added the bug label Sep 9, 2022
@jimmo jimmo mentioned this issue Sep 9, 2022
Merged
tannewt added a commit to tannewt/circuitpython that referenced this issue May 17, 2024
@tannewt
Merge pull request micropython#9255 from CDarius/cores3_camera
ba22fc8 
Fix for M5Stack Cores3 camera
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tools/makemanifest.py: Force the repo version of the mpy_cross package. jimmo/micropython
1 participant
@andrewleech
0