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

fixes for docker.yml + toolchain.dockerfile #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tracedgod
Copy link

fixes for docker.yml + toolchain.dockerfile

This pull request features changes to the Github Actions docker runner job as well as the libxenon.dockerfile build. My reason for this PR is I had noticed the Free60 Docker Hub images appear to be quite behind from what the Github repo has.

Github Actions

  • Fixes deprecation warnings encountered by updating the tags for the respective actions

Dockerfile(s)

  • Fixes failing build steps to the docker images by upgrading the base image from ubuntu:21.04 to ubuntu:22.04
  • Combine both the toolchain.dockerfile & the libxenon.dockerfile builds into a single libxenon docker image build by utilizing a multi-stage dockerfile solution (this removes the toolchain.dockerfile and instead runs those steps as the base image in the libxenon.dockerfile build)

Also a question: Is there a specific reason the toolchain and libxenon images are separate other than the ability to use the free60/toolchain image once it's built in the next stage for building free60/libxenon?

add PULL=0 build-arg to LibXenon build step to force local cache

add load: true to build steps to store built images locally

enable BUILDKIT_INLINE_CACHE on the build steps

switch to multi-stage dockerfile for libxenon for single image build

fix env exports in multi-stage dockerfile

Update libxenon.dockerfile

Fix formatting issue for stage 2 build
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.

1 participant