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
[20.09] chez-modules: Fix path to csv-site. #98577
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.
Diff LGTM.
[1 built, 6 copied (38.3 MiB), 10.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/98577
6 packages built:
chez chez-matchable chez-mit chez-scmutils chez-srfi idris2
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.
Built chez-* and idris2 on darwin
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.
Something I missed in my review last night:
Please cherry-pick these commits from the master branch (for reference, see https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#backporting-changes; start from step 4). Once that is done, my approval stands.
It seems that the installation path for chez modules no longer includes the Chez version. This fixes the build for chez-mit and chez-scmutils. (cherry picked from commit 72ccc9b)
As a follow-up to NixOS#97927, chez can now find its libraries in a nix-shell, and derivations for such libraries don't need to handle the search path themselves. (cherry picked from commit 8f60607)
c31e29c
to
17b6d3f
Compare
@cole-h Sorry, I have missed the |
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.
Much better, thanks. Should be good to go!
Sorry for the wait, thanks! |
Motivation for this change
backport of #97927 and #98128
ZHF: #97479
cc @NixOS/nixos-release-managers
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
nixpkgs-review
seems to want to merge my branch with master before building the diffs. The changes were tested before backporting, though./result/bin/
)nix path-info -S
before and after)