dfaure added a comment.
(Intrusive) fix is up at
https://invent.kde.org/frameworks/kio/-/merge_requests/3
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D29738
To: marten, #frameworks, dfaure
Cc: meven, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
dfaure added a comment.
Well spotted. Indeed, when using kioexec (because of the DeleteTemporaryFiles
option) we no longer detect non-existing executables. I'll look into fixing
this.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D29738
To: marten, #frameworks, dfaure
meven added a comment.
It seems it is causing two failures in kiogui_applicationlauncherjobtest :
https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20SUSEQt5.14/105/testReport/junit/projectroot/autotests/kiogui_applicationlauncherjobtest/
REPOSITORY
R241 KIO
REVISION DETAIL
http
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:6452a34cf01d: Fix service file specifying 'Run in
terminal' giving an error code 100 (authored by marten).
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D29738?vs=82823
dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.
Nice! In addition to the bugfix, calling resultingArguments() only once is a
definite improvement, given everything that happens in there. I didn't realize
we were calling it twice.
REPOSI
ngraham edited the summary of this revision.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D29738
To: marten, #frameworks, dfaure
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
marten created this revision.
marten added reviewers: Frameworks, dfaure.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
marten requested review of this revision.
REVISION SUMMARY
https://bugs.kde.org/show_bug.cgi?id=421374 describes how a service file
spec