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

users-groups module: use buildEnv in per-user profiles #31283

Merged
merged 1 commit into from Nov 25, 2017

Conversation

michalrus
Copy link
Member

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@michalrus
Copy link
Member Author

Copy link
Contributor

@4z3 4z3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 for changing to buildEnv if that's the right way to do it (but it would be nice to keep the user's name in the derivation's name). I'm indifferent about the change of the path within /etc. I don't like changing LOGNAME to USER, because the former is defined by POSIX (though, I know we don't care in nixpkgs).

@Mic92 Mic92 merged commit 3ea4934 into NixOS:master Nov 25, 2017
@michalrus michalrus deleted the per-user-profiles-buildEnv branch November 25, 2017 15:31
@michalrus
Copy link
Member Author

Thank you very much!

@vcunat
Copy link
Member

vcunat commented Dec 20, 2017

Is this too intrusive to pick into 17.09? #32728

@michalrus
Copy link
Member Author

It would be wonderful. ♥

@roberth
Copy link
Member

roberth commented Jul 25, 2023

What's the reasoning for applying system.path.postBuild to user environments instead of just the system path?

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

Successfully merging this pull request may close these issues.

users-groups module: per-user packages should use buildEnv instead of symlinkTree.
7 participants