wayfire: update ecosystem to 0.11.0-unstable#492535
wayfire: update ecosystem to 0.11.0-unstable#492535hustlerone wants to merge 2 commits intoNixOS:masterfrom
Conversation
ede86b0 to
6fe96a0
Compare
| security = { | ||
| polkit.enable = true; | ||
| pam.services.swaylock = { }; | ||
| pam.services.wf-locker = { }; |
There was a problem hiding this comment.
I can't find anything about a wf-locker service. Can you link where you found it?
There was a problem hiding this comment.
We do the same thing for swaylock.
If I recall correctly, it'll most likely be handled by home manager (in the future).
There was a problem hiding this comment.
There should be no reason to patch GitHub work flow files.
There was a problem hiding this comment.
WayfireWM/wf-shell#349 has been merged so the diff can be withdrawn now
| wcm = callPackage ./wcm.nix { }; | ||
| wf-shell = callPackage ./wf-shell.nix { }; | ||
| wf-shell = callPackage ./wf-shell { }; | ||
| pixdecor = callPackage ./pixdecor.nix { }; |
There was a problem hiding this comment.
This could cause some issue since wayfire-plugins-extra also includes pixdecor.
Files from later plugins don't overwrite already existing ones, but there are no warning about conflicts during rebuilds (nix does log something when building wayfire-wrapped, but this is easily missed).
So this would use the newer pixdecor:
programs.wayfire = {
enable = true;
plugins = with pkgs.wayfirePlugins; [
pixdecor
wayfire-plugins-extra
];
};But this one wouldn't:
programs.wayfire = {
enable = true;
plugins = with pkgs.wayfirePlugins; [
wayfire-plugins-extra
pixdecor
];
};I'd prefer if we'd push for the open PR in wayfire-plugins-extra to be merged/updated: WayfireWM/wayfire-plugins-extra#312
(Or patch the submodule to use a newer pixdecor version.)
I'm generally unsure about using an unstable Wayfire version in nixpkgs because this would mean dealing more with plugin conflicts (what if someone wants Wayfire from commit XY because it adds support for a useful Wayland protocol, but some plugins aren't compatible with the development version yet?)
There was a problem hiding this comment.
I actually haven't noticed pixdecor being there.
As for compatibility, other than wf-config's breaking change I don't think it's going to be a big deal.
I'll withdraw the separate pixdecor addition.
Bumps wayfire, wf-shell, wf-plugins-extras and wf-config.
Updates the whole ecosystem.
Adds pixdecor, updates wayfire-plugins-extra to latest. Now you can enjoy the burn and carpet animations. These are the more notable changes one might look for.
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.