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

[nix] replace streamLayeredImage to buildImage #612

Merged
merged 2 commits into from
May 27, 2024
Merged

Conversation

Avimitin
Copy link
Contributor

Proot have too many bugs to be fixed, it is better to use qemu to build the docker image.

@sequencer
Copy link
Member

Since we are using qemu now. How about providing multiple arch(aarch64,x86,riscv) in our release?

@Avimitin Avimitin force-pushed the hotfix/release branch 2 times, most recently from 5f2176a to 99087d3 Compare May 27, 2024 05:57
@@ -2,6 +2,7 @@ name: Build Docker Image

# Release on any git tag
on:
pull_request:
Copy link
Member

Choose a reason for hiding this comment

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

how about on tagging?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is on tagging, pull_request is for testing only.

@Avimitin Avimitin force-pushed the hotfix/release branch 2 times, most recently from f7432b1 to d5afc56 Compare May 27, 2024 06:31
Avimitin added 2 commits May 27, 2024 14:34
Proot have too many bugs to be fixed, it is better to use qemu to build
the docker image.

Signed-off-by: Avimitin <[email protected]>
@sequencer sequencer merged commit 0a9d1b4 into master May 27, 2024
80 checks passed
@sequencer sequencer deleted the hotfix/release branch May 27, 2024 06:46
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