Skip to content

Conversation

@Redlonghead
Copy link
Contributor

@Redlonghead Redlonghead commented Dec 12, 2025

SDDM theme "Elegant" but with support for qt6

With the dropping of qt5 with sddm in #444954 the package elegant-sddm will not work and another repo does fix it for qt6. I just was not sure if we should replace the old package since in theory it's not usable now or just create a new package.

@GaetanLepage thoughts on replacing vs. just a new package?

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Dec 12, 2025
@Redlonghead
Copy link
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 470060
Commit: bb14e6c24b54cfebd631b9657bac650063dbfdbe


x86_64-linux

✅ 1 package built:
  • elegant-sddm-qt6

aarch64-linux

✅ 1 package built:
  • elegant-sddm-qt6

@GaetanLepage
Copy link
Contributor

If keeping the QT5-only package does not make any technical sense, then I would say that we should just replace it with this new one.
@K900 any thoughts about this?

@K900
Copy link
Contributor

K900 commented Dec 12, 2025

It should be replaced, yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

9.needs: reviewer This PR currently has no reviewers requested and needs attention. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants