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
expect: patch configure after the autoreconf phase, not before it #80006
expect: patch configure after the autoreconf phase, not before it #80006
Conversation
@GrahamcOfBorg build expect |
90a75e1
to
51c57c5
Compare
@GrahamcOfBorg build expect |
well the autoreconfHook is certainly at odds with the postPatch: Can the stty fixup be changed to modify configure.ac or perhaps by setting a variable? |
@dtzWill you're spot on. The only change needed is to change the |
51c57c5
to
aaa60de
Compare
@GrahamcOfBorg build expect pkgsMusl.libffi |
diff LGTM, but doesn't fix your use case |
x86_64 is still good with these changes:
|
preConfigure = '' | ||
sed -i "s,/bin/stty,$(type -p stty),g" configure |
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.
Doing this still sounds more natural to me:
preConfigure = '' | |
sed -i "s,/bin/stty,$(type -p stty),g" configure | |
postPatch = '' | |
sed -i "s,/bin/stty,$(type -p stty),g" configure.in |
Ideally we would fix this upstream by adding --with-stty
configure flag or something.
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.
Good catch @dtzWill and thanks for taking care of this @basvandijk! :)
@jonringer What did you refer to here? This PR fixes the issue for me, see #79863 (comment) |
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 fixes static-haskell-nix's CI.
oh, i saw the pkgsMusl.libffi failure for aarch64, but he was referencing x86_64 musl build |
I'll open @jtojnar's suggestion in a new pr |
Motivation for this change
Fixes #79863
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)