Skip to content

Conversation

@isabelroses
Copy link
Member

@isabelroses isabelroses commented Feb 26, 2025

previously it made no sense to add this since we didn't support any darwin modules but now we have the global cache.enable and some users may want to set it in a "shared" module between nixos and darwin

closes #162

@isabelroses isabelroses requested a review from getchoo February 26, 2025 12:32
previously it made no sense to add this since we didn't support any
darwin modules but now we have the global `cache.enable` and some users
may want to set it in a "shared" module between nixos and darwin
@getchoo
Copy link
Member

getchoo commented Mar 21, 2025

Sorry it took me a bit to review

When I first saw this in my inbox I initially just thought "no", but after mulling over it again, this would be very helpful for Darwin HM users instead of needing to configure this at a per-user level or shove it in a Flake config

The only problem I have now is the expectations that exposing this module sets. As someone taking a quick look at this project (or well, our flake.nix), I would assume we support Darwin the same as NixOS/HM. It'd be a bit weird to find out this is the only option...so I dunno

@isabelroses
Copy link
Member Author

When I first saw this in my inbox I initially just thought "no"

Honestly this was my reaction last time we had one of these 😔

this would be very helpful for Darwin HM users instead of needing to configure this at a per-user level or shove it in a Flake config

Honestly thats the only reason it makes any sense right now since. And it somewhat makes sense to reduce that complexity when setting up but thats pretty niche.

The only problem I have now is the expectations that exposing this module sets. As someone taking a quick look at this project (or well, our flake.nix), I would assume we support Darwin the same as NixOS/HM.

Totally valid. I think it maybe a good idea to get a consensus on this. Perhaps in the discord?

@isabelroses
Copy link
Member Author

isabelroses commented May 24, 2025

Actually might be worth merging now that #545 is open. At least to give them the ability to access catppuccin.sources.

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.

Support Nix Darwin

2 participants