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

feat(nixos): sddm #168

Merged
merged 1 commit into from
May 14, 2024
Merged

feat(nixos): sddm #168

merged 1 commit into from
May 14, 2024

Conversation

isabelroses
Copy link
Member

very cool theme 0 bias

@isabelroses isabelroses force-pushed the feat/sddm branch 6 times, most recently from 4176e6d to fcebcec Compare May 13, 2024 17:15
@isabelroses

This comment was marked as resolved.

@isabelroses isabelroses force-pushed the feat/sddm branch 4 times, most recently from 57c6343 to 6661622 Compare May 13, 2024 21:03
@isabelroses isabelroses changed the title feat: sddm feat(nixos): sddm May 13, 2024
Copy link
Member

@getchoo getchoo left a comment

Choose a reason for hiding this comment

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

whoever's maintaining this theme is doing awesome. wonder who it is??

i think this is also going to keep me on plasma 6 for another week before going back to gnome on 46. thanks!

@getchoo getchoo merged commit d8a6d8a into catppuccin:main May 14, 2024
2 checks passed
Weathercold pushed a commit to Weathercold/nix that referenced this pull request May 14, 2024
@ecarlson94
Copy link

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 the metadata file 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).

@isabelroses
Copy link
Member Author

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 the metadata file 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).

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.

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.

3 participants