installer: allow overriding nix user GID and UIDs #4345
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Much better solution: #4346. |
|
Closing in favor of #4346. |
bjornfor
force-pushed
the
installer-configurable-uid-gid
branch
from
1fec96f
to
8799962
Compare
|
#4346 still isn't merged. I'm opening this again since it's a simple fix we can apply right now. (This UID/GID conflict issue came up again today, after having been hacked around with |
|
macos CI seemed to have timed out after 60 minutes? And I don't know why the other checks were skipped. Will rebase to force another CI attempt. |
Needed to resolve conflict in case the default GID and UIDs are in use.
bjornfor
force-pushed
the
installer-configurable-uid-gid
branch
from
8799962
to
f4d57aa
Compare
|
All checks are green, please review (2 lines) and merge :-) |
|
Thanks! |
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Needed to resolve conflict in case the default GID and UIDs are in use.