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
lighttpd: fix crosscompilation #61128
lighttpd: fix crosscompilation #61128
Conversation
@@ -45,6 +47,7 @@ stdenv.mkDerivation rec { | |||
++ stdenv.lib.optional enableExtendedAttrs "--with-attr"; | |||
|
|||
preConfigure = '' | |||
export PATH=$PATH:${pcre.dev}/bin |
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 we can avoid this by putting pcre.dev in nativeBuildInputs
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.
it should also be dev
by default? Just put pcre
in both buildInputs
and nativeBuildInputs
. No PATH
by hand, and no .dev
by hand.
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.
Thing is pcre.dev
provides pcre-config
, which is bash script that's needed to configure script, and we need target version. If i put it to nativeBuildInputs it will gladly cofigure, but at compile time it will try and fail to link it against host pcre library.
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.
Oh it's *-config
hell. See #51176 .
OK what you did is fine, but add a comment explaining that. Also, it's build and host platform, not host and target platform for this. At least it's a bash script not real executable so we can run it at all.
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.
Yeah nevermind. What's being used now is reasonable.
Motivation for this change
Lighttpd fails to cross compile.
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)