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

Chili sddm theme #59016

Closed
wants to merge 1 commit into from
Closed

Chili sddm theme #59016

wants to merge 1 commit into from

Conversation

feijoas
Copy link
Contributor

@feijoas feijoas commented Apr 5, 2019

Motivation for this change

Added the chili login theme for SDDM: https://github.com/MarianArlt/sddm-chili

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.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@nixos-discourse
Copy link

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 ];
Copy link
Contributor

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?

Copy link
Contributor Author

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?

Copy link
Contributor

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.

Copy link
Contributor Author

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.

Copy link
Contributor

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.
Copy link
Contributor

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.

Copy link
Contributor

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.

Copy link
Contributor Author

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.

@c0bw3b
Copy link
Contributor

c0bw3b commented Apr 21, 2019

IMHO it should rather be placed under pkgs/data/themes/
(less random than top-level 'misc')

@@ -147,6 +147,8 @@ in

castxml = callPackage ../development/tools/castxml { };

chili-sddm-theme = libsForQt5.callPackage ../misc/themes/sddm/chili {};
Copy link
Contributor

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

@romildo
Copy link
Contributor

romildo commented Apr 21, 2019

IMHO it should rather be placed under pkgs/misc/themes/
(less random than than top-level 'misc')

More recently themes are being put at pkgs/data/themes.

@c0bw3b
Copy link
Contributor

c0bw3b commented Apr 21, 2019

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...
I edited, thanks. 🐱

@stale
Copy link

stale bot commented Jun 1, 2020

Thank you for your contributions.
This has been automatically marked as stale because it has had no activity for 180 days.
If this is still important to you, we ask that you leave a comment below. Your comment can be as simple as "still important to me". This lets people see that at least one person still cares about this. Someone will have to do this at most twice a year if there is no other activity.
Here are suggestions that might help resolve this more quickly:

  1. Search for maintainers and people that previously touched the
    related code and @ mention them in a comment.
  2. Ask on the NixOS Discourse. 3. Ask on the #nixos channel on
    irc.freenode.net.

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 1, 2020
@rileyinman
Copy link
Contributor

I would still be interested in seeing this happen. I just found out about it, but I used to use chili as well.

@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 10, 2020
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/353

@@ -17207,6 +17207,8 @@ in

camingo-code = callPackage ../data/fonts/camingo-code { };

chili-theme = libsForQt5.callPackage ../data/themes/chili/chili.nix {};
Copy link
Member

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

Comment on lines +22 to +28
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.
'';
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
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 ];
'';

@SuperSandro2000
Copy link
Member

Result of nixpkgs-review pr 59016 run on x86_64-linux 1

1 package built:
  • chili-theme

@stale
Copy link

stale bot commented Jun 3, 2021

I marked this as stale due to inactivity. → More info

@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 3, 2021
@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jan 8, 2022
@Mindavi
Copy link
Contributor

Mindavi commented Apr 25, 2022

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.

error: attribute 'lib' missing

       at /home/rick/.cache/nixpkgs-review/pr-59016/nixpkgs/pkgs/data/themes/chili/chili.nix:21:15:

           20|
           21|   meta = with stdenv.lib; {
             |               ^
           22|     license = licenses.gpl3;
(use '--show-trace' to show detailed location information)

@Mindavi Mindavi closed this Apr 25, 2022
@sents sents mentioned this pull request Apr 2, 2023
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet