Pull Request Overview
- Opened on August 3, 2026
- Status Closed
- Commit count 1 with first commit August 3, 2026
Total Delta
0 Total Diff Delta
Open Days
Open 35 weekdays
Test Delta
0 Diff Delta in Test Files
Breakdown by Phase
How long has this pull request spent in each phase of its lifecycle?
| Fraction of total time | Business days | Phase |
|---|---|---|
|
|
0 days | Authoring 1 commit before pull request opened for review |
|
|
0 days | Awaiting first review |
|
|
0 days | Revising work with 0 commits in response to 2 reviews that left 1 comment |
Total time for pull request 0 business days from first commit to closed PR
fix: exclude Python from AppImage to fix Show In Folder with non-ASCII paths
The AppImage bundles Python with a non-standard prefix (usr/bin), causing AppRun to set PYTHONHOME={AppImage}/usr/bin. External tools invoked by Shotcut (e.g. xdg-open for "Show In Folder") inherit this env var and fail when the path contains non-ASCII characters: Could not parse arguments: Invalid byte sequence in conversion input.
-
packaging/linux/appimage/appimage.yml: Remove Python from the AppDir after tarball extraction. Covers the non-standard prefix location (usr/bin/lib/python*) and the standard one (usr/lib/python*):yaml
- rm -rf usr/bin/python* usr/bin/lib/python* usr/lib/python*
With no Python in the AppDir, AppRun does not setPYTHONHOME/PYTHONPATH, and external processes handle all paths correctly.
Comments Threads Pending Resolution
ddennedy
reviewed
on August 3, 2026
packaging/linux/appimage/appimage.yml
|
14
|
||||
|
15
|
||||
|
16
|
||||
|
17
|
Opus 4.8 (1M) |
Resolved Comment Threads
No resolved comments have been left on this PR.