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

Added forgotten parameter #3974

Merged
merged 2 commits into from
Oct 9, 2024
Merged

Conversation

judovana
Copy link
Contributor

@judovana judovana commented Oct 4, 2024

Simplify the parsing logic of makejdk_any_platform_args (#3957) have forgot to add mandatpory parameter to usage of
removeSystemModulesHashBuilderParams in
tooling/reproducible/comparable_patch.sh#L351

Simplify the parsing logic of makejdk_any_platform_args (adoptium#3957) have
forgot to add mandatpory parameter to usage of
removeSystemModulesHashBuilderParams in
tooling/reproducible/comparable_patch.sh#L351
Copy link
Contributor

@andrew-m-leonard andrew-m-leonard left a comment

Choose a reason for hiding this comment

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

looks good, thanks

@andrew-m-leonard
Copy link
Contributor

CI check failures must be unrelated problem

@judovana
Copy link
Contributor Author

judovana commented Oct 4, 2024

Indeeed. The linuxes are

 { [2 bytes data]
curl: (35) error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure

windows are

 Downloading Temurin release of boot JDK version 17 failed.
Attempting to download GA release of boot JDK version 17 from https://api.adoptopenjdk.net/v3/binary/latest/17/ga/windows/x64/jdk/hotspot/normal/adoptopenjdk
Error: Process completed with exit code 4.

@karianna karianna merged commit 16cf218 into adoptium:master Oct 9, 2024
26 checks passed
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.

3 participants