Fix for #10797: qvm-open-in-vm should now work in KDE-based cubes#642
Fix for #10797: qvm-open-in-vm should now work in KDE-based cubes#642rysiekpl wants to merge 1 commit intoQubesOS:mainfrom
Conversation
`kde-open`/`kde-open5` start the process that handles the file in a way that the old `| cat` hack does not work. This fixes it. Tested on GNOME-, Xfce-, and KDE-based templates.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #642 +/- ##
==========================================
- Coverage 70.30% 69.89% -0.41%
==========================================
Files 3 3
Lines 495 495
==========================================
- Hits 348 346 -2
- Misses 147 149 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Suggestion: If you intend for this pull request to resolve the associated issue and would like for it to be linked to the issue automatically, you can put |
Good point, PR updated. |
Is it still a "hack"? I get describing |
Normal approach would be if |
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026042500-4.3&flavor=pull-requests Test run included the following:
New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026032404-devel&flavor=update
Failed tests67 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/170766#dependencies 33 fixed
Unstable testsDetails
Performance TestsPerformance degradation:9 performance degradations
Remaining performance tests:75 tests
|
5 participants
kde-open/kde-open5start the process that handles the file in a way that the old| cathack does not work.This fixes it. Tested on GNOME-, Xfce-, and KDE-based templates.
Closes QubesOS/qubes-issues#10797