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
WIP: pkgsEmscripten: emscripten package set using cross toolchain #59784
Conversation
Implement the emscripten package set using the cross toolchain in a similar way as `pkgsCross`.
Unfortunately I'm getting an evaluation error. @dtzWill the structure is now roughly based on |
Looks okay. TBH I'd like everybody to use WASM/WASI, but I suppose it's nice to work with emscripten as it's more popular. emconfigure and emmake are mostly just a collection of hacks, so the likelihood of working with many packages out of the box is not great. |
buildInputs = [ emscripten python ] ++ buildInputs; | ||
nativeBuildInputs = [ emscripten python ] ++ nativeBuildInputs; | ||
# name = "emscripten-${args.name}"; | ||
buildInputs = (args.buildInputs or []) ++ [ emscripten python ]; |
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.
These won't cross compile
{ | ||
stdenv = super.stdenvAdapters.emscriptenAdapter super.stdenv; | ||
|
||
json_c = super.json_c.overrideDerivation (old: { | ||
nativeBuildInputs = [ autoreconfHook pkgconfig ]; | ||
propagatedBuildInputs = [ zlib ]; | ||
buildInputs = old.buildInputs ++ [ automake autoconf ]; |
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.
Same thing here.
It would also be good to update emscripten. I believe it's a little bit out of date. |
I agree with you. Right now, I just want to see if we can get pyodide running. |
I won't continue with this. |
Implement the emscripten package set using the cross toolchain in a similar
way as
pkgsMusl
.Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)