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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: never attempt to call builtins.storePath in pure evaluation mode #63

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

Conversation

yajo
Copy link

@yajo yajo commented Nov 22, 2023

After 5a16547, I started getting errors like nix-community/home-manager#2409.

@moduon MT-1075

@yajo
Copy link
Author

yajo commented Jan 23, 2024

@edolstra would you mind to review this one? It's pretty obvious and I'm having to add this patch everywhere...

@daneov
Copy link

daneov commented Aug 7, 2024

@yajo : How do you add this patch manually? Hitting this issue as well

@yajo
Copy link
Author

yajo commented Aug 12, 2024

Like this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants