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
vdagent: fixing 90s timeout on shutdown #85853
Conversation
Is it possible to use fetchpatch on the commits which were merged instead of copying the patch in the repo ? |
No idea how to do that with gitlab - but feel free to do so. |
@GrahamcOfBorg eval |
please rebase your branch ontop of master, and force push to fix eval:
|
(fetchpatch { url = "https://gitlab.freedesktop.org/spice/linux/vd_agent/-/commit/7b0435ef66af088c1a1be20b6bc6b0fcb76e4e1a.patch"; sha256 = "<FIXME>";}) | ||
(fetchpatch { url = "https://gitlab.freedesktop.org/spice/linux/vd_agent/-/commit/9b8c0ebb9fb573e6ce3c5416371509f416503d0c.patch"; sha256 = "<FIXME>";}) |
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.
Just so you don't forget: these two hashes need to be changed.
Also, you need to specify fetchpatch
in the arguments at the top of the file in order to use it.
Also also, you can drop timeout.diff
because of this 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.
Yeah the <FIXME>
part was meant as a place holder.
That's not working because:
So you'd need to patch those patch files also that are aimed at a slightly modified source code... |
Since the original provided patches can't patch the stable version used because some alterations have been meanwhile made upstream, I think it's best again to revert to our own patch file. |
cec36cb
to
b426995
Compare
Any suggestion? |
Fixed according to Mic92's suggestion. |
@symphorien |
Result of 1 package built:- spice-vdagent |
Thanks. |
Motivation for this change
In vdagent 0.20.0 was a bug introduced that causes a 90s timeout during shutdown. Upstream it's already fixed: https://gitlab.freedesktop.org/spice/linux/vd_agent/-/merge_requests/7 so the patch just needs to be applied here.
Fixes: #85235
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)