-
-
Notifications
You must be signed in to change notification settings - Fork 16.6k
micropython: fix cross #407238
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
base: master
Are you sure you want to change the base?
micropython: fix cross #407238
Conversation
@FliegendeWurst would you be able to take a peek? Would be greatly appreciated, since you seem to have familiarity with cross-compilation issues. |
Found a few things, but they won't fix this issue yet. https://github.com/micropython/micropython/blob/master/ports/unix/Makefile#L34
https://github.com/micropython/micropython/blob/master/ports/unix/Makefile#L154 pkg-config needs to be prefixed as |
The rather more annoying issue is that this project vendors dozens of third-party libraries. All of them at an outdated version, of course. |
In fact you got "lucky" by cross-compiling for riscv, when I try to build for aarch64 I get a false-positive notification about a logic bug in mbedtls... |
Anyway, let's get to the real issue. The project contains "mpy-cross", used to turn scripts into precompiled bytecode. This compiler runs as part of the build process. I imagine package users also use it. So, to perform a cross build, we must get |
039eb06
to
e000884
Compare
|
e000884
to
5eb42e1
Compare
I will merge if @FliegendeWurst gives an ACK |
@FliegendeWurst ping |
5eb42e1
to
86856d1
Compare
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.