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

Try to not rebuild the base image and download it instead. #35

Merged
merged 2 commits into from
Dec 4, 2023

Conversation

Bluebugs
Copy link
Contributor

When building the android image, we currently don't have it locally as buildx multiplatform automatically push it and does not make it available locally. This is trying to download locally the amd64 v1.1.4-base image that was just build for multi target and reuse it for building the android image.

@@ -31,4 +31,6 @@ jobs:
- name: Build all images
run: |
make -f Makefile.multi docker-all
docker pull docker.io/fyneio/fyne-cross-images:v1.1.4-base
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should 1.1.4 be used for all updates, or will we want to change this frequently? Perhaps a variable if we can't automatically figure out the latest version?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, good catch, it should be latest instead. Will fix.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected now.

Copy link
Member

@andydotxyz andydotxyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch

@Bluebugs Bluebugs merged commit 25ee723 into main Dec 4, 2023
10 checks passed
@Bluebugs Bluebugs deleted the feature/no-rebuild-base-image branch December 4, 2023 15:16
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.

2 participants