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
texinfo: fix cross #34180
texinfo: fix cross #34180
Conversation
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 all my comments apply to both
stdenv.lib.optional stdenv.isSunOS "AWK=${gawk}/bin/awk" | ||
++ optionals crossCompiling [ | ||
"PERL=${buildPackages.perl}/bin/perl" | ||
"BUILD_CC=${buildPackages.stdenv.cc.targetPrefix}gcc" |
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.
Should be not needed anymore?
@@ -19,6 +26,13 @@ stdenv.mkDerivation rec { | |||
installTargets="install install-tex"; | |||
''; | |||
|
|||
configureFlags = | |||
stdenv.lib.optional stdenv.isSunOS "AWK=${gawk}/bin/awk" | |||
++ optionals crossCompiling [ |
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.
Shouldn't be conditional
# We need a native compiler to build perl XS extensions | ||
# when cross-compiling. | ||
depsBuildBuild = [ buildPackages.stdenv.cc ]; | ||
nativeBuildInputs = [ perl ]; |
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.
Do we need native perl too then?
|
||
with stdenv.lib; | ||
|
||
stdenv.mkDerivation rec { | ||
let | ||
crossCompiling = stdenv.buildPlatform != stdenv.hostPlatform; |
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 this is not needed if the stuff below is unconditional?
15a6bff
to
3182c47
Compare
3182c47
to
f57fc78
Compare
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)Have not re-tested on top of latest staging,
but works well on my local patch queue.