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

Add support for distribution prefixes with non-patch release versions #239

Merged
merged 3 commits into from
Jun 8, 2022

Conversation

Malax
Copy link
Member

@Malax Malax commented Jun 8, 2022

Allow OpenJDK distribution prefixes to be used in conjunction with major versions. Previously, a specific patch version was required when using a distribution prefix. In addition, an alias prefix for openjdk was introduced: heroku. The new prefix makes it more clear that it will select a build of OpenJDK that was built by Heroku.

For example, previously unsupported version strings such as zulu-11, openjdk-17 or heroku-8 are now supported and will resolve to the latest patch release of the specified major version.

The CHANGELOG was updated for immediate release as v133.

@Malax Malax changed the title WIP Distribution Selection Jun 8, 2022
@Malax Malax force-pushed the malax/distribution-select branch 4 times, most recently from 6524b87 to 2868b92 Compare June 8, 2022 11:27
@Malax Malax force-pushed the malax/distribution-select branch from 39bb2fc to cb7cf7e Compare June 8, 2022 11:31
@Malax Malax changed the title Distribution Selection Add support for distribution prefixes with non-patch release versions Jun 8, 2022
@Malax Malax marked this pull request as ready for review June 8, 2022 11:40
@Malax Malax requested a review from a team as a code owner June 8, 2022 11:40
lib/jvm.sh Show resolved Hide resolved
lib/jvm.sh Show resolved Hide resolved
test/spec/java_spec.rb Show resolved Hide resolved
@Malax Malax force-pushed the malax/distribution-select branch 3 times, most recently from b0f039c to f0a655a Compare June 8, 2022 12:25
@Malax Malax force-pushed the malax/distribution-select branch from f0a655a to f1dd3cd Compare June 8, 2022 12:27
@Malax Malax merged commit 25840a5 into main Jun 8, 2022
@Malax Malax deleted the malax/distribution-select branch June 8, 2022 13:40
edmorley added a commit to heroku/heroku-buildpack-gradle that referenced this pull request Jun 10, 2022
The test assertions need updating after:
heroku/heroku-buildpack-jvm-common#239
edmorley added a commit to heroku/heroku-buildpack-java that referenced this pull request Jun 10, 2022
The test assertions need updating after:
heroku/heroku-buildpack-jvm-common#239
edmorley added a commit to heroku/heroku-buildpack-clojure that referenced this pull request Jun 13, 2022
The test assertions need updating after:
heroku/heroku-buildpack-jvm-common#239
edmorley added a commit to heroku/heroku-buildpack-java that referenced this pull request Jun 13, 2022
* Vendor buildpack-stdlib

Since:
- The buildpack-stdlib repo has been sunset/archived, so no future
  changes to it are expected.
- It's a script rather than a binary and is a small file, so a great candidate
  for vendoring.
- Downloading it at runtime causes more potential failures modes,
  as seen in heroku/heroku-buildpack-clojure#118

* Fix test failures on `main`

The test assertions need updating after:
heroku/heroku-buildpack-jvm-common#239
edmorley added a commit to heroku/heroku-buildpack-gradle that referenced this pull request Jun 13, 2022
* Vendor buildpack-stdlib

Since:
- The buildpack-stdlib repo has been sunset/archived, so no future
  changes to it are expected.
- It's a script rather than a binary and is a small file, so a great candidate
  for vendoring.
- Downloading it at runtime causes more potential failures modes,
  as seen in heroku/heroku-buildpack-clojure#118

* Fix test failures on `main`

The test assertions need updating after:
heroku/heroku-buildpack-jvm-common#239
edmorley added a commit to heroku/heroku-buildpack-clojure that referenced this pull request Jun 13, 2022
* Switch to the recommended regional S3 domain

Whilst the global S3 endpoint (`s3.amazonaws.com`) still works,
AWS now recommends using the appropriate regional endpoint
to access the bucket:
https://docs.aws.amazon.com/AmazonS3/latest/userguide/VirtualHosting.html#s3-legacy-endpoints

Our buildpack buckets are in `us-east-1`, whose regional domain
is `*.s3.us-east-1.amazonaws.com`:
https://docs.aws.amazon.com/general/latest/gr/s3.html#s3_region

GUS-W-11283397.

* Fix test failures on `main`

The test assertions need updating after:
heroku/heroku-buildpack-jvm-common#239
EtienneM pushed a commit to Scalingo/buildpack-jvm-common that referenced this pull request Jul 21, 2022
…heroku#239)

* Add support for distribution prefixes with non-patch release versions

* Update CHANGELOG

* Improve output when using heroku distribution prefix
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.

Zulu not working
2 participants