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

Rename Docker build images #1171

Merged
merged 10 commits into from
Nov 11, 2024
Merged

Rename Docker build images #1171

merged 10 commits into from
Nov 11, 2024

Conversation

DanRStevens
Copy link
Collaborator

Change naming scheme to more clearly indicate these are build environments (as opposed to containing a fully built library). This impacts the image name, the folder name, the workflow name, and associated variables.

Change the default Docker registry from DockerHub to GitHub.

Allow changes to the makefile to cause a rebuild of all environments. The makefile is responsible for the naming, and any changes to the naming will require rebuilding images for all environments.

Part of:

Naming is controlled by the `makefile`, and would require rebuilding images if it were to change. As such, it should be included in the set of changed files that will trigger a rebuild for any given platform.
We don't want to confuse people by suggesting the images contain a fully built NAS2D library. Rather, they are the build environments needed to build NAS2D, as used by the CI workflows.
The Docker build commands and imported into the main `makefile`, so can be run from there. There is no need to explicitly specify the subfolder.

This simplifies renaming the subfolder.
This is perhaps a better description of the folder contents. For apps, we might expect a "docker" folder to be for building images that contain the app. What we are doing here is building images that can be used to build the library.
@DanRStevens DanRStevens merged commit b853a3d into main Nov 11, 2024
9 checks passed
@DanRStevens DanRStevens deleted the renameDockerBuildImages branch November 11, 2024 07:05
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