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
xfce.xfdesktop: appending bug fix #32204
Conversation
Contains Bug NixOS#12832 fix which solves issue with gtk 2.24.31 regression - grayed out folders in wallpaper choose dialog
patches = | ||
[ | ||
(fetchurl { | ||
url = https://git.xfce.org/users/eric/xfdesktop/patch/?id=cc311b61b82b7510a3a6cb0952d3a331e3551e05; |
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.
@@ -19,6 +19,13 @@ stdenv.mkDerivation rec { | |||
libglade xfce4panel thunar exo garcon libnotify hicolor_icon_theme | |||
]; | |||
|
|||
patches = | |||
[ | |||
(fetchurl { |
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.
please use fetchpatch
instead of fetchurl
.
@yellowgh0st have you considered @Mic92's review? |
@joachifm Thanks. Done. |
This is somewhat worrisome:
Hopefully Xfce patch digests don't change. |
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.
Patch solves the issue. I'm pushing it to master along with some cosmetic changes and a hash change. Thanks!
Merged in d9f4f9e. |
Contains Bug#12832 fix which solves issue with gtk 2.24.31 regression - grayed out folders in wallpaper choose dialog
https://bugzilla.xfce.org/show_bug.cgi?id=12832
Patch source:
https://git.xfce.org/xfce/xfdesktop/commit?id=157f5b55cfc3629d595ef38984278de5915aac27
Motivation for this change
Unable to change wallpapers folder in wallpaper choose dialog.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)