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

devos/darwin: refine users #460

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

devos/darwin: refine users #460

wants to merge 3 commits into from

Conversation

montchr
Copy link
Collaborator

@montchr montchr commented Apr 28, 2022

I meant to include these changes in #438:

  • Demonstrates the ability to define reusable home-manager user configurations
    via home.users and import them into any arbitrary home-manager user within a
    nixos or darwin profile.
  • Rename the system/home users to primary to distinguish them from all the other
    references to 'darwin' and also to aid in demonstrating the flexibility of
    "portable" home-manager users
  • Use Digga's collectHosts function to... collect hosts for deriving homeConfigurations
  • Remove some extraneous comments
  • Add some extraneous comments

@montchr montchr force-pushed the darwin-user-tweaks branch from b637919 to dd0eee2 Compare April 30, 2022 06:25
@montchr montchr requested review from Pacman99 and blaggacao April 30, 2022 06:35
@Pacman99
Copy link
Member

Could we remove the nixos user and use admin/primary everywhere? It would make writing docs easier too, since we only have to reference one user to change.

@montchr montchr added this to the Release v0.12.0 milestone Jul 9, 2022
@montchr
Copy link
Collaborator Author

montchr commented Jul 9, 2022

yeah, good point. will update.

@montchr montchr force-pushed the darwin-user-tweaks branch from dd0eee2 to fb4107a Compare July 12, 2022 22:50
@montchr
Copy link
Collaborator Author

montchr commented Jul 12, 2022

Could we remove the nixos user and use admin/primary everywhere?

I took a halfway approach, keeping the nixos user around but only using it in the bootstrap host since that's the expected user for live media.

except in installation media
@montchr montchr force-pushed the darwin-user-tweaks branch from 4163b06 to cf0d820 Compare July 12, 2022 23:50
@montchr montchr self-assigned this Jul 12, 2022
@montchr
Copy link
Collaborator Author

montchr commented Jul 13, 2022

Now I'm remembering that one of the reasons I split up the users was because nix-darwin lacks a lot of the basic users.users.<name>.* options. extraGroups, for one. Builds are failing during checks for that reason. But I have some prior art from my old work MacBook that I need to resurrect anyway, so I'll move those workarounds over to these examples in the process.

https://github.com/divnix/digga/runs/7312053064?check_suite_focus=true#step:7:312

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