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
js_of_ocaml: fix jbuild (explicitly call bash) #29749
Conversation
@@ -11,5 +11,7 @@ stdenv.mkDerivation rec { | |||
|
|||
propagatedBuildInputs = [ js_of_ocaml-compiler uchar ]; | |||
|
|||
patches = [ ./jbuild.diff ]; |
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.
This is usually done with patchShebangs
in the postPatch
phase.
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.
Thank you. That works an is indeed much simpler: master...ibor:master
Do you need a new pull request?
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.
This pull request is fine (although its individual commits could be squashed together for the better), but I don't have the permission to merge yet. (I hope to get it soon.)
In general, please squash the commits. Thank you for your contribution! |
js_of_ocaml-camlp4: init at 3.0.0 js_of_ocaml-ppx: init at 3.0.0
* js_of_ocaml: fix jbuild (explicitly call bash) * js_of_ocaml: Use patchShebangs instead of manual patch (cherry picked from commit 00422b2)
ocamlPackages.js_of_ocaml-3.0.0 currently fails to build because bash is not called properly.
The patch fixes jbuild to explicitly call generate_stubs.sh via bash.
BTW: Modifying generate_stubs.sh's shebang (#!/bin/bash -> #!/usr/bin/env bash ) did not help.