-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
test precedence for setting the search path #9066
base: master
Are you sure you want to change the base?
Conversation
3fe2c8e
to
813701c
Compare
this is just to show the principle. the test cases are incomplete and way too manual.
813701c
to
a5b4d8e
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/2023-11-13-nix-team-meeting-minutes-103/35400/1 |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/nix-path-is-not-recognized/38404/6 |
Note: Update documentation accordingly once this is fixed https://github.com/NixOS/nix/pull/9113/files#r1487801059 |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/2024-05-06-nix-team-meeting-minutes-143/45021/1 |
this is just to show the principle for now.
the test cases are incomplete and way too manual.
the complete specification should follow this table, and the tests ideally be generated from that automatically:
Motivation
The current behavior being inconsistent and surprising (#8890 #8784) has spawned multiple competing and conflicting attempts of fixing it:
{restrict,pure}-eval
#7689Context
developed together with @thufschmitt
this should also resolve the confusion around behavior of
restrict-eval
, as that is supposed to only allow explicitly defined search paths.Priorities
Add 👍 to pull requests you find important.