fix(linux): the AppImage's parts list names the repository it is built in #1312
No reviewers
Labels
No labels
accessibility
autorelease: pending
autorelease: tagged
bug
dependencies
documentation
duplicate
enhancement
github_actions
good first issue
help wanted
invalid
question
testing wanted
wontfix
No milestone
No assignees
1 participant
Due date
No due date set.
Dependencies
No dependencies set
Reference
MagicBeanDev/Hookshot!1312
Loading…
Reference in a new issue
No description provided.
Delete branch "pr/linux-parts-base"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
What
scripts/linux/appimage.shnow passes--basetomake_parts.py, naming thelinux-partsrelease of$GITHUB_REPOSITORY(falling back toNotRexed/Hookshotwhen it is unset, as in a local build).Why
publish_parts.shuploads the parts to thelinux-partsrelease of$GITHUB_REPOSITORY, but the listhookshot-linux.txtwas made without--base, so it always namedNotRexed/Hookshot's release. On this repository the two are the same place, and nothing changes here. On a fork, CI uploads the parts to the fork while the list still points at this repository's release, which holds none of them, so a fork's Linux update cannot find its own parts. (Found in the fork's v2.6.4 release: its list named parts with no file behind them.)Testing
make_parts.pyrun over a stand-in AppDir through the same${GITHUB_REPOSITORY:-NotRexed/Hookshot}expansion the script now uses:GITHUB_REPOSITORY=notquiteog/Hookshot: every URL inhookshot-linux.txtishttps://github.com/notquiteog/Hookshot/releases/download/linux-parts/...https://github.com/NotRexed/Hookshot/releases/download/linux-parts/..., identical to beforebash -n scripts/linux/appimage.shOne file, five lines.
🤖 Generated with Claude Code
Looks good. I checked that
make_parts.pytakes--base(default is the same NotRexed URL, so builds here are unchanged), that it strips a trailing slash, and thatpublish_parts.shuploads to$GITHUB_REPOSITORY'slinux-partsrelease, so the list and the upload now agree. The${GITHUB_REPOSITORY:-NotRexed/Hookshot}fallback covers local builds. CI is green, including the appimage job.