Bug 387659

Summary: Transaction should not have a timeout.
Product: [Unmaintained] libqapt Reporter: sbw <sbw>
Component: generalAssignee: Harald Sitter <sitter>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: The patch to resolve this problem

Description sbw 2017-12-07 08:09:59 UTC
Created attachment 109231 [details]
The patch to resolve this problem

In QProcess::waitForFinsiehd, it has a 30s timeout in default.
But Transaction should not have it.

Because some packages maybe have a long install time.
for example, we can easy create a DEB package that has a  sleep(31) in postinst script.

I think that's the reason for many `Backend disappear` or `Transaction crash` error.
Comment 1 Christoph Feck 2017-12-31 13:07:48 UTC
Please submit your patch via https://phabricator.kde.org/differential/diff/create/
Comment 2 Harald Sitter 2018-01-08 02:16:40 UTC
Git commit 46f45d734f5de8d856582783af65d854c590147b by Harald Sitter.
Committed on 08/01/2018 at 02:16.
Pushed by sitter into branch 'master'.

dpkg -i should not have a timeout

Summary:
we have no idea how long installing a package will take, it could be super
large or the disk could be super slow, or its maint scripts could do
network-bound IO.

if anything should make judgment on what is too long it should be dpkg
itself, not qapt.

Reviewers: sitter, vanini

Reviewed By: sitter

Differential Revision: https://phabricator.kde.org/D9611

M  +1    -1    src/worker/aptworker.cpp

https://commits.kde.org/libqapt/46f45d734f5de8d856582783af65d854c590147b