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

Remove the "package = pkgs.kdePackages.sddm;" line #38

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Ambioid
Copy link

@Ambioid Ambioid commented Jul 27, 2024

As it causes conflicts and error messages when rebuilding

…rs conflicts and error messages when rebuilding
@Ambioid Ambioid changed the title Remove the "package = pkgs.kdePackages.sddm;" line, as it causes erro… Remove the "package = pkgs.kdePackages.sddm;" line Jul 27, 2024
@isabelroses
Copy link
Member

We have this added as it is a strict requirement for running our sddm theme since we need qt6 builds of sddm and the default is qt5, you may find that the theme does not work otherwise. If your running either plasma5 then you should use lib.mkForce to change the package and if your using plasma6 you can just ignore this line.

@Ambioid
Copy link
Author

Ambioid commented Aug 16, 2024

Perhaps we could add something to the readme to clarify, in this case? I am running plasma 6 and can only successfully build when this line is omitted

@isabelroses
Copy link
Member

Feel free too. Most the time I would just tell users to prefer our nix flake

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