Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix permissions of files by changing owner to builder user #10

Merged
merged 1 commit into from
Sep 21, 2022

Conversation

vchernin
Copy link
Contributor

@vchernin vchernin commented Aug 5, 2022

Found this when trying to use this action for this PKGBUILD: https://github.com/wwmm/easyeffects/blob/1aae2d16ac1972c41eb9df8b91f2c1b0c8a4d9b9/PKGBUILD

Without this later on makepkg tries to change the owner itself, which fails:

chmod: changing permissions of '/github/workspace/src': Operation not permitted
==> ERROR: An unknown error has occurred. Exiting...
/entrypoint.sh: line 55:   297 User defined signal 1   sudo -H -u builder makepkg --syncdeps --noconfirm ${INPUT_MAKEPKGARGS:-}

I’m not entirely sure why other repositories wouldn’t have been affected by this though.

Otherwise later on makepkg tries to change the owner itself, which fails:
chmod: changing permissions of '/github/workspace/src': Operation not permitted

==> ERROR: An unknown error has occurred. Exiting...
/entrypoint.sh: line 55:   297 User defined signal 1   sudo -H -u builder makepkg --syncdeps --noconfirm ${INPUT_MAKEPKGARGS:-}
@edlanglois edlanglois merged commit df3a6da into edlanglois:master Sep 21, 2022
@edlanglois
Copy link
Owner

Thanks for the fix!

@vchernin vchernin deleted the fix-permissions branch September 21, 2022 19:26
@vchernin vchernin restored the fix-permissions branch September 22, 2022 00:24
@vchernin vchernin deleted the fix-permissions branch September 22, 2022 00:24
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.

None yet

2 participants