Skip to content

make NIX_PATH configurable#3440

Closed
aristidb wants to merge 1 commit intoNixOS:masterfrom
aristidb:master
Closed

make NIX_PATH configurable#3440
aristidb wants to merge 1 commit intoNixOS:masterfrom
aristidb:master

Conversation

@aristidb
Copy link
Copy Markdown
Contributor

@aristidb aristidb commented Aug 3, 2014

No description provided.

@peti
Copy link
Copy Markdown
Member

peti commented Aug 3, 2014

+1

@lucabrunox
Copy link
Copy Markdown
Contributor

@peti I'm against this patch. As mentioned in #3426 (similar problem) I prefer there was a way to override attribute sets that are not nixos options, rather than add an option for every environment variable.

@peti
Copy link
Copy Markdown
Member

peti commented Aug 3, 2014

@lethalman, do you have an implementation that does this?

@lucabrunox
Copy link
Copy Markdown
Contributor

@peti No I'm discussing this, @edolstra in #3426 acted as the override should have worked already... perhaps adding this nix-path option is going to be an hack for a bigger problem.

@edolstra
Copy link
Copy Markdown
Member

edolstra commented Aug 3, 2014

Try environment.sessionVariables.NIX_PATH = mkForce [ ... ].

@aristidb
Copy link
Copy Markdown
Contributor Author

aristidb commented Aug 3, 2014

OK, mkForce works. You might want to consider using mkDefault on the default?

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.

4 participants