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
dhall: passthru dhall prelude #35558
Conversation
Failure on aarch64-linux (full log) Partial log (click to expand)
|
|
||
in static.overrideAttrs (old: { | ||
passthru = old.passthru or {} // { | ||
prelude = stdenv.mkDerivation { |
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 think you can use runCommand
instead of stdenv.mkDerivation
. Besides simplifying the expression it also involves an unnecessary build-time dependence on the C compiler:
prelude = runCommand "dhall-prelude" {} ''
mkdir $out
cp -r ${old.src}/Prelude/* $out/
''
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.
That doesn’t work (tried it) because old.src
is a .tar.gz
. What could work is invoking stdenv
manually, but I don’t know if it’s worth the effort.
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.
In that case you can use stdenvNoCC
instead of stdenv
, which at least removes the build-time dependency on the C compiler
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.
Done.
Success on x86_64-linux (full log) Partial log (click to expand)
|
d2e1d21
to
e06789b
Compare
Failure on aarch64-linux (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
@GrahamcOfBorg build dhall.prelude |
Success on aarch64-linux (full log) Partial log (click to expand)
|
Makes it possible to reference `dhall.prelude`, the same version that comes with the dhall exetutable’s source code.
e06789b
to
4b70328
Compare
Corrected the invocation in |
Failure on aarch64-linux (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Makes it possible to reference
dhall.prelude
, the same version that comes withthe dhall exetutable’s source code.
cc @Gabriel439 @ocharles