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
kdegraphics-thumbnailers: Add patch for thumbnail.so hang. #85494
Conversation
Could we do 20.03 also? |
@@ -9,6 +9,7 @@ mkDerivation { | |||
license = [ lib.licenses.lgpl21 ]; | |||
maintainers = [ lib.maintainers.ttuegel ]; | |||
}; | |||
patches = [ ./kdegraphics-thumbnailers-fix-hang.patch ]; |
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.
can we get the details of the bug fix linked here as a reference?
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.
Added.
038ab82
to
7bde25e
Compare
Ah I wrote wrong, I'm using this on top of 20.03, in any case I think it should be backported to both. |
7bde25e
to
31c1540
Compare
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'll defer to @ttuegel, but this looks correct.
@@ -0,0 +1,31 @@ | |||
diff --git a/ps/gscreator.cpp b/ps/gscreator.cpp |
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.
Could we fetch this from Phabricator using fetchpatch
, instead of including it in the Nixpkgs tree?
31c1540
to
57e6799
Compare
Changed to fetchpatch. |
I would really appreciate a backport of this too... |
ping @ttuegel |
The patch url 404s and since it's not in the binary cache, that means the build fails. |
Thanks @adisbladis for fixing this in 5e20609 |
Fixes a bug in the KDE PDF thumbnail generator which causes thumbnail.so processes to hang and accumulate as well as a long delay during shutdown/reboot as systemd tries to terminate the processes and eventually gives up. Reproduction is probably easy, have a folder with a PDF and browse it with Dolphin. The patch here is the same as was accepted upstream.
https://bugs.kde.org/show_bug.cgi?id=404652
https://phabricator.kde.org/D26635
This is a pretty annoying bug so I think that a patch is appropriate as is backporting to 19.09. Tested on release-19.09 but it surely works in master because of the same kdegraphics-thumbnailers version.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)