-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
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
Chili sddm theme #59016
Chili sddm theme #59016
Conversation
This pull request has been mentioned on Nix community. There might be relevant details there: https://discourse.nixos.org/t/sddm-theme-location-in-nixpkgs/2604/3 |
sha256 = "036fxsa7m8ymmp3p40z671z163y6fcsa9a641lrxdrw225ssq5f3"; | ||
}; | ||
|
||
buildInputs = [ qtbase qtquickcontrols qtgraphicaleffects ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These don't actually seem to do anything - afaict there is no build for this derivation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, just an unzip. What is the recommended phase to add the qt deps?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, where are they needed? Presumably sddm uses them when it loads the theme. If this works (which I guess it does if you tested it?) then presumably sddm already has the dependencies around when it loads it. In which case you can delete these.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested it. Without this line it does not work. I guess some qt path are set automatically.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm indeed, somehow something is putting them in propagated build inputs. I always find that a bit magical, but it seems to work.
/* | ||
This derivation installs the chili theme. Use | ||
`services.xserverdisplayManager.sddm.theme = "chili";` | ||
to enable the theme. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You also need to install the package.
I don't think it's that useful to put installation instructions in comments in the .nix
files. Probably the right place would be a section on sddm in the manual, or an extension to the module option doc for the sddm.theme
option.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TBH it would probably be better to teach the sddm module a themePackages
option rather than relying on installing it system-wide, but that's another kettle of fish.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Certainly. There is an example for lightdm greeters in nixpkgs.
IMHO it should rather be placed under |
pkgs/top-level/all-packages.nix
Outdated
@@ -147,6 +147,8 @@ in | |||
|
|||
castxml = callPackage ../development/tools/castxml { }; | |||
|
|||
chili-sddm-theme = libsForQt5.callPackage ../misc/themes/sddm/chili {}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be further down in the ### DATA
section
More recently themes are being put at pkgs/data/themes. |
Oops, that's what I wanted to point out in the first place but then I pasted the wrong path... |
e09959c
to
8fc2453
Compare
8fc2453
to
7769856
Compare
Thank you for your contributions.
|
I would still be interested in seeing this happen. I just found out about it, but I used to use chili as well. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
@@ -17207,6 +17207,8 @@ in | |||
|
|||
camingo-code = callPackage ../data/fonts/camingo-code { }; | |||
|
|||
chili-theme = libsForQt5.callPackage ../data/themes/chili/chili.nix {}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about use default.nix instead of chili.nix? Other files under theme dir use default.nix
license = licenses.gpl3; | ||
maintainers = with maintainers; [ mschneider ]; | ||
homepage = https://github.com/MarianArlt/sddm-chili; | ||
description = "The chili login theme for SDDM"; | ||
longDescription = '' | ||
Chili is hot, just like a real chili! Spice up the login experience for your users, your family and yourself. Chili reduces all the clutter and leaves you with a clean, easy to use, login interface with a modern yet classy touch. | ||
''; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
license = licenses.gpl3; | |
maintainers = with maintainers; [ mschneider ]; | |
homepage = https://github.com/MarianArlt/sddm-chili; | |
description = "The chili login theme for SDDM"; | |
longDescription = '' | |
Chili is hot, just like a real chili! Spice up the login experience for your users, your family and yourself. Chili reduces all the clutter and leaves you with a clean, easy to use, login interface with a modern yet classy touch. | |
''; | |
description = "The chili login theme for SDDM"; | |
longDescription = '' | |
Chili is hot, just like a real chili! Spice up the login experience for your users, your family and yourself. Chili reduces all the clutter and leaves you with a clean, easy to use, login interface with a modern yet classy touch. | |
homepage = https://github.com/MarianArlt/sddm-chili; | |
license = licenses.gpl3; | |
maintainers = with maintainers; [ mschneider ]; | |
''; |
Result of 1 package built:
|
I marked this as stale due to inactivity. → More info |
I'm gonna close this since it's bitrotted and upstream hasn't made any updates anymore either. Reading the comments this may still work fine though, so: if someone wants to pick this up, they're free to do it. It doesn't feel like the original author is still willing to get this in, but please reopen or resubmit if so.
|
Motivation for this change
Added the chili login theme for SDDM: https://github.com/MarianArlt/sddm-chili
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)