-
-
Notifications
You must be signed in to change notification settings - Fork 300
hyprpaper: update configuration format to support v0.8 #2087
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
Conversation
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.
LGTM, but AFAICT hyprwm/hyprpaper@1d8df14 is not yet available on Nixpkgs. Once it is, our nixpkgs input should be updated in this PR.
Sorry I should've mentioned that it is not in nixpkgs yet, I'm guessing it will be added with the next hyprpaper release. |
|
hyprpaper v0.8 has been released, will keep an eye out for when it makes it way onto nixpkgs. |
|
Running in to the configuration change issue on my machines, looks like $ nix run nixpkgs#hyprpaper -- --help
┏ hyprpaper v0.8.0
┣━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┣ --config -c [str] | Set a custom config path ┃
┣ --verbose | Enable more logging ┃
┣ --help -h | Show the help menu ┃
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛At the time of writing I still see 0.7.6 on search.nixos.org But I see 27aae640a25f20c24c1a014e05772242275d20c4 landing on $ git log -n1 origin/nixpkgs-unstable pkgs/by-name/hy/hyprpaper/
commit 27aae640a25f20c24c1a014e05772242275d20c4
Author: SchweGELBin <[email protected]>
Date: Mon Dec 29 21:51:16 2025 +0100
hyprpaper: 0.7.6 -> 0.8.0
hyprpaper got rewritten into hyprtoolkit,
this is why so many buildInputs have been changed |
|
According to the PR tracker it is in nixpkgs-unstable but not nixos-unstable yet https://nixpkgs-tracker.ocfox.me/?pr=475150 |
|
@trueNAHO I believe hyprpaper v0.8 is in nixos-unstable now, would you like me to update the nixpkgs flake input? |
yes, please do that |
- hyprland uwsm is now managed by the hyprland nixos module - missing '--' have been added to the nix gc configuration - custom tuigreet module has been renamed to greetd - custom hyprland module has been moved to nixos modules having to wait for stylix to update their hyprlock config to the new syntax nix-community/stylix#2087
|
For reference, resolving the build failures is being worked on in #2117. |
Closes: nix-community#408 Link: nix-community#2087 Link: https://wiki.hypr.land/Hypr-Ecosystem/hyprpaper Reviewed-by: NAHO <[email protected]> Reviewed-by: 0xda157 <[email protected]>
trueNAHO
left a comment
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.
Closes: nix-community#408 Closes: nix-community#2122 Link: nix-community#2087 Link: https://wiki.hypr.land/Hypr-Ecosystem/hyprpaper Reviewed-by: NAHO <[email protected]> Reviewed-by: 0xda157 <[email protected]>
Closes: nix-community#408 Closes: nix-community#2122 Link: nix-community#2087 Link: https://wiki.hypr.land/Hypr-Ecosystem/hyprpaper Reviewed-by: NAHO <[email protected]> Reviewed-by: 0xda157 <[email protected]>
The
checks are successfull, while the following fail:
The CI failure can be locally reproduced by running: $ nix build .#ci.buildbot.x86_64-linux.testbeds.ashell
[...]
error: stack overflow; max-call-depth exceeded
[...]The previous commit a525e47 ("flake: update all inputs (#2117)") does not have this problem. CC: @nix-community/stylix-committers |
Use an explicit toString conversion to hopefully resolve the confusing CI errors. This goes against [1], but IIRC I already tried removing toString in [2] but did not mention it then because I might have been running into the exact issues we are facing now. [1]: nix-community#2087 (comment) [2]: nix-community#2087 (review)
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.
while the following fail [...]
strange, i'd vote to merge this now that the checks aren't failing and investigate the toString issue later.
Removing this explicit |
Updates the configuration format for the hyprpaper wallpaper list to match the new format as described at https://wiki.hypr.land/Hypr-Ecosystem/hyprpaper/.
Configuration format changed as of hyprwm/hyprpaper@1d8df14.