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.

Sign up for GitHub

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

Jump to bottom

Fix chown error in newer Alpine images [affects new builds without caches] #804

Merged
merged 1 commit into from
Dec 20, 2024
Merged

Fix chown error in newer Alpine images [affects new builds without caches] #804

merged 1 commit into from
Dec 20, 2024

Conversation

mikeage
Copy link
Member

@mikeage mikeage commented Dec 20, 2024

No description provided.

@mikeage mikeage added the infrastructure Build or tooling infrastructure label Dec 20, 2024
@mikeage
Copy link
Member Author

mikeage commented Dec 20, 2024 •
edited
Loading

Tested at https://github.com/mikeage/open-brush/actions/runs/12430506835/job/34706103104, where I deleted the Windows cache (also technically the Android caches, but the results will be from the windows build)

Actually, I just realized it didn't test it, because it skipped that step. Well, it worked when running locally, so it should be fine!

It also passed on main on my fork, which is where it did save the cache. So this PR is ok

@mikeage mikeage marked this pull request as draft December 20, 2024 11:34
@mikeage mikeage marked this pull request as ready for review December 20, 2024 12:21
@mikeage mikeage merged commit 22a22a2 into main Dec 20, 2024
42 checks passed
@mikeage mikeage deleted the infra/fix_chown branch December 20, 2024 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
infrastructure Build or tooling infrastructure
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mikeage