-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[zulu-openjdk] Bugfixes for the Zulu JDK package #25485
base: master
Are you sure you want to change the base?
Conversation
Recently the Zulu package was enhance by adding many different binaries for version 11.0.24. However, other changes to support these binary packages should also have been made at that time, as none of them were functional. host settings should identfy the binary type of the package: conan-io#25484 validation should support all the new configurations: conan-io#25483 Zip files with executables need an extra flag: conan-io#25482
Conan v1 pipeline ✔️Warning Conan Center will stop receiving updates for Conan 1.x packages soon - please see announcement. All green in build 1 (
Conan v2 pipeline ✔️
All green in build 1 (
|
Hooks produced the following warnings for commit 5061f16zulu-openjdk/11.0.24@#4879f535c336c9c84ecd8b13e95c3538
zulu-openjdk/21.0.4@#7507846af1f702153594a14a1996b509
zulu-openjdk/11.0.12@#31ee00f80e81df1d11261f88469f5666
zulu-openjdk/11.0.19@#47151ecebf84c4fa99443024a848f203
zulu-openjdk/17.0.9@#a5113261164e089bafb13e65f917af94
zulu-openjdk/21.0.1@#5daf6540dffee73b8d1df140f133bfb2
zulu-openjdk/11.0.15@#8dd715e558a916d85a8f64b302e6195b
|
Recently the Zulu package was enhanced by adding additional binaries for version 11.0.24.
However, other changes to support these binary packages should also have been made at that time, as none of them were functional.
This pull request resolves three of these problems and has been tested on these platforms.
Summary
Changes to recipe: zulu-openjdk/11.0.24
Motivation
host settings should identfy the binary type of the package:
#25484
validation should support all the new configurations:
#25483
Zip files with executables need an extra flag:
#25482
Details
keep_permissions=True
for theget()
tool