-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
sddm: 0.17.0 -> 0.18.0 #43978
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
sddm: 0.17.0 -> 0.18.0 #43978
Conversation
This breaks the plasma5 test. I will first need to fix that! |
@GrahamcOfBorg eval |
Hmm, the OCR in the plasma5 test is breaking again... Anyone an idea how we could get the OCR back working? |
@bkchr I think that's a good idea. Also I noticed that sddm now when it's inactive hides the information the test would normally be looking for. Edit: I think that might only be when you're already logged and it functions as a screensaver |
@bkchr did you manage to get the tests to behave? |
Sorry, did not had time yet to look into creating a custom theme :( |
Beep boop? :) |
@GrahamcOfBorg test sddm |
Success on aarch64-linux Attempted: tests.sddm No partial log is available. |
Success on x86_64-linux Attempted: tests.sddm No partial log is available. |
But the plasma test will fail. |
I created a candidate for the world's ugliest theme for use with the plasma tests but I haven't tried it out yet. I'll give it a go this weekend. |
Nice. It does not need to be beautiful :D If you want and you already have it online somewhere. I could test it for you. |
That'd be great: https://gitlab.com/peterhoeg/sddm-high-contrast |
Did you manage to get it working @bkchr? |
Did not had time yet to test it. Sorry. |
@bkchr can you confirm (and close if so) that this is not needed anymore? |
0.18 is already in unstable so this is safe to close. |
Motivation for this change
@adisbladis @peterhoeg
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)