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
ocamlPackages.js_of_ocaml*: 3.7.0 -> 3.8.0 #106343
Conversation
ffdd59b
to
1506ecd
Compare
1506ecd
to
7330eb7
Compare
@vbgl rebased against the janeStreet update seems like this fixed things, currently running a |
Result of 65 packages built:
|
|
||
nativeBuildInputs = [ ocaml findlib dune_2 cppo menhir ]; |
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.
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.
The reason is obvious -- all of those are nativeBuildInputs
(they are build-time dependencies, so have to run on build
platform) and not buildInputs
(which would run on host
platform).
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.
Nevermind what I wrote earlier, I wouldn't see why this made any difference or they would be required as buildDunePackage
adds them to buildInputs
. I think we don't support cross compiling for OCaml packages so this is no issue per se?!
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.
Thanks. But my question was more: is there a realistic scenario in which the host and build platforms are different?
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.
I see. I'd say it'd be simpler to address that in buildDunePackage for the whole of ocamlPackages
.
7330eb7
to
569a41b
Compare
569a41b
to
6fed468
Compare
6fed468
to
b048d5b
Compare
Motivation for this change
Tried to see if I could do the broken auto-update properly. Unfortunately it seems that this update currently breaks
eliom
which should be fixed as soon as they support ocaml-migrate-parsetree 2.1.0 or we have a patch for it:Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)