Skip to content

Fix: Binary releases missing execute permissions#870

Merged
mattisonchao merged 2 commits intomainfrom
copilot/fix-binary-release-permission
Jan 20, 2026
Merged

Fix: Binary releases missing execute permissions#870
mattisonchao merged 2 commits intomainfrom
copilot/fix-binary-release-permission

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 20, 2026

GitHub Actions artifacts don't preserve file permissions—all files become 644 after upload/download. When packaged into release tarballs, the oxia binary remained non-executable, causing permission denied errors for users.

Changes

  • Added --mode=a+x flag to tar command in .github/workflows/release.yaml to set execute permissions on all files in the archive
tar --mode=a+x -czf "../../release-assets/oxia-${PLATFORM}.tar.gz" *

This ensures the binary is executable immediately after extraction without requiring manual chmod +x.

Original prompt

This section details on the original issue you should resolve

<issue_title>fixes: permission lost for binary release</issue_title>
<issue_description>### Motivation

When I use the binary release of oxia. I met a permission issue.

➜  bin ./oxia standalone
zsh: permission denied: ./oxia

plus, do I need to sign the released binary? </issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: mattisonchao <74767115+mattisonchao@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix permission issue for binary release of oxia Fix: Binary releases missing execute permissions Jan 20, 2026
Copilot AI requested a review from mattisonchao January 20, 2026 06:40
@mattisonchao mattisonchao marked this pull request as ready for review January 20, 2026 06:48
@mattisonchao mattisonchao merged commit 8da5685 into main Jan 20, 2026
10 of 11 checks passed
@mattisonchao mattisonchao deleted the copilot/fix-binary-release-permission branch January 20, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fixes: permission lost for binary release

2 participants