search

catppuccin / sddm

🔒 Soothing pastel theme for SDDM
MIT License
290 stars 17 forks source link

readme: add nix #31

Closed ElysaSrc closed 4 months ago

ElysaSrc commented 4 months ago

Add NixOS to the list of supported OS with details on how to use the derivation

ElysaSrc commented 4 months ago

(force pushed: fixed typo)

ElysaSrc commented 4 months ago

Thanks for merging quickly ❤️

ecarlson94 commented 3 months ago

This doesn't work for me because of this PR. In fact, this only works for NixOS users that use Qt (which I don't). I had to remove this line to get the theme to work. This tells me that we should remove this line from here and manage Qt support within the install phase of the catppuccin-sddm package instead (it should be opt in with an option like qt.enable).

ElysaSrc commented 3 months ago

This doesn't work for me because of this PR. In fact, this only works for NixOS users that use Qt (which I don't). I had to remove this line to get the theme to work. This tells me that we should remove this line from here and manage Qt support within the install phase of the catppuccin-sddm package instead (it should be opt in with an option like qt.enable).

I think we should not change it. NixOS is not the only OS supported, and if using NixOS unstable, currently you must change the sddm package to the Qt6 version (pkgs.kdePackages.sddm) because, by default, it starts the Qt5 version.

@isabelroses said on this issue: https://github.com/catppuccin/nix/pull/168#issuecomment-2111754109

This seems like an issue you should take up with the upstream packager (nixpkgs). Since sddm is under the KDE banner so it makes sense for the catppuccin-sddm theme (and its maintainer, me) to ship it with the qt-platform set to 6 by default.