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
buildRustPackage: fix regex for separating lib and bin #48020
Conversation
E.g. exa was wrongly put into /lib, as it matches .*.a but not .*\.a
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.
Seems reasonable to me, but I don't feel comfortable merging a change to the core rust infrastructure myself.
Already noted in the PR. |
I tested this with a package that produces .so / .a and it seems to work so I'm happy with it. |
Thank you for the fix @erictapen :) |
Backport in 5647e9a (I hope the staging-18.09 branch is still used post release). |
@timokau I think this backport hasn't made it into 18.09 yet - are you sure it'll trickle through from the staging branch? (the installation of |
IIRC the PR was already built in https://hydra.nixos.org/eval/1483497, so it just needs to be merged into |
Building this fix alone isn't enough. It causes a mass rebuild. We need to wait for the mass rebuild to finish and see if all dependent packages still build. We typically bundle several mass rebuilding changes on |
This just made it into |
Correction: the commit is in @FRidh could you elaborate on what kind of failures occured after the merge? I wonder wether they correspond to this PR. Maybe, the stuff affected by this PR was already built by Hydra and we can cherry-pick it safely on |
A range of failures. I suggest tracking down the corresponding Hydra evaluation. |
E.g. after #47709,
exa
was wrongly put into/lib
, as it matches.*.a
but not.*\.a
.I tested, that
exa
now is being put into/bin
as it should.Should be backported to
release-18.09
, or maybestaging-18.09
, as this will trigger a lot of rebuilds.sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)