Skip to content

use mkDefault on NIX_PATH to make it easier to override#3444

Closed
aristidb wants to merge 1 commit intoNixOS:masterfrom
aristidb:nix-path-default
Closed

use mkDefault on NIX_PATH to make it easier to override#3444
aristidb wants to merge 1 commit intoNixOS:masterfrom
aristidb:nix-path-default

Conversation

@aristidb
Copy link
Copy Markdown
Contributor

@aristidb aristidb commented Aug 3, 2014

supersedes #3440

@shlevy
Copy link
Copy Markdown
Member

shlevy commented Aug 4, 2014

Generally I'd expect setting NIX_PATH to include the default, not override it.

@aristidb
Copy link
Copy Markdown
Contributor Author

mkForce also works.

@aristidb aristidb closed this Aug 30, 2014
@nbp
Copy link
Copy Markdown
Member

nbp commented Aug 30, 2014

This change sounds wrong as mkDefault property is only interpreted while merging options, but the NIX_PATH is not an option, but it is an attribute of the sessionVariable option.

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.

3 participants