Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tests/functional/lang/eval-fail-readDir-storeDir-pure #10546

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

roberth
Copy link
Member

@roberth roberth commented Apr 17, 2024

Motivation

Add missing test case.

  • It's the wrong error. Make --pure-eval --expr without flakes work. Reproducable outside the test framework. Not sensitive to NIX_PATH environment variable or --nix-path "".
$ nix-instantiate --pure-eval tests/functional/lang/eval-fail-readDir-storeDir-pure.nix
error: access to absolute path '/home' is forbidden in pure evaluation mode (use '--impure' to override)

Context

Priorities and Process

Add 馃憤 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Apr 17, 2024
@roberth roberth added the bug label Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug with-tests Issues related to testing. PRs with tests have some priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant