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

New project structure and build scripts #11

Merged
merged 11 commits into from Oct 7, 2021

Conversation

brunoborges
Copy link
Member

@brunoborges brunoborges commented Sep 7, 2021

This PR simplifies the Dockerfile structure on a per-base image, and then defines a list of available JDK versions in the jdk_versions file, with a build.sh script that is capable of building all images, or a single one.

With this script, it is easier to define later on a PR for building/testing only specific base-images and/or JDK versions. Also enables better GitHub Actions workflow for.

@karianna
Copy link
Member

karianna commented Sep 8, 2021

@brunoborges Did you take a look at adoptium/containers? Might find some inspiration there (+ script to create official DockerHub config).

docker/cbld/Dockerfile Outdated Show resolved Hide resolved
@brunoborges
Copy link
Member Author

@brunoborges Did you take a look at adoptium/containers? Might find some inspiration there (+ script to create official DockerHub config).

Anything in particular you believe could be leveraged?

I looked into and did not find. Adoptium uses tar.gz while here we use Linux repositories. No need for checksum, for example.

Multi-arch is interesting though, but I'm afraid we are not there yet.

Copy link
Contributor

@d3r3kk d3r3kk left a comment

Choose a reason for hiding this comment

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

Just some minor nits looks good otherwise. I like how you've collapsed the individual Dockerfiles and parameterized them.

build.sh Show resolved Hide resolved
build.sh Outdated Show resolved Hide resolved
@brunoborges brunoborges added the enhancement New feature or request label Sep 8, 2021
@brunoborges brunoborges self-assigned this Sep 8, 2021
@karianna karianna marked this pull request as draft September 10, 2021 09:52
Copy link
Member

@karianna karianna left a comment

Choose a reason for hiding this comment

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

LGTM

@brunoborges brunoborges marked this pull request as ready for review October 7, 2021 21:45
@brunoborges brunoborges changed the base branch from main to new-proj-layout October 7, 2021 21:51
@brunoborges brunoborges merged commit 7b77a75 into microsoft:new-proj-layout Oct 7, 2021
@brunoborges brunoborges deleted the new-proj-layout branch October 7, 2021 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants