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

Automate the process of building the uf2 dev image #239

Merged
merged 2 commits into from
Mar 30, 2023

Conversation

mjaskula
Copy link
Collaborator

The image is build using docker and is directly based on the current release uf2 build procedure. This is intended for use during development to build the uf2 image locally. The existing github workflow will still be used to build the release. This means that we now have the uf2 build process described in three places (github workflow, documentation, uf2_build directory). In the future we should try to consolidate.

The image is build using docker.
@mjaskula mjaskula added the firmware Software related issue label Mar 16, 2023
@mjaskula mjaskula self-assigned this Mar 16, 2023
@mjaskula mjaskula requested a review from awonak March 16, 2023 13:40
Copy link
Collaborator

@awonak awonak left a comment

Choose a reason for hiding this comment

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

Good idea to automate this and make it as easy as possible for development and testing. No need to consolidate things now, but I agree we would benefit from reuse or consolidation at some point.

The instructions below describe the steps needed to build the uf2 locally on your machine for
development purposes. Alternatively you can run the script `software/uf2_build/build_uf2.sh` to
build the image in a docker container. You will need to have docker installed and running. See the
script for execution details.
Copy link
Collaborator

Choose a reason for hiding this comment

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

[discussion] Are there prerequisites or dependencies we can add here or somewhere? Even if it's just a Docker Getting Started link, I think that'd help.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done.

@mjaskula mjaskula merged commit 689b6d5 into Allen-Synthesis:main Mar 30, 2023
@mjaskula mjaskula deleted the mjj/docker_build_env branch March 30, 2023 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
firmware Software related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants