Re: RFR: 8287917: System.loadLibrary does not work on Big Sur if JDK is built with macOS SDK 10.15 and earlier [v2]
> Please review this PR. > SDK 10.15 and earlier reports os.version as 10.16 on Big Sur. This fix will > check if dynamic linker support, which is supported from Big Sur, is > available or not on the OS even if os.version is reported as 10.16 instead of > 11. The os.version 10.16 doesn't include the update version like y of > 10.x.y. Hence the change only see if it is 10.16 or not. Yoshiki Sato has updated the pull request incrementally with one additional commit since the last revision: Update copyright end year - Changes: - all: https://git.openjdk.org/jdk/pull/9077/files - new: https://git.openjdk.org/jdk/pull/9077/files/a9b95965..bdd97130 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk=9077=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk=9077=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk/pull/9077.diff Fetch: git fetch https://git.openjdk.org/jdk pull/9077/head:pull/9077 PR: https://git.openjdk.org/jdk/pull/9077
Re: RFR: 8287917: System.loadLibrary does not work on Big Sur if JDK is built with macOS SDK 10.15 and earlier
On Fri, 10 Jun 2022 17:51:37 GMT, Mandy Chung wrote: >> Please review this PR. >> SDK 10.15 and earlier reports os.version as 10.16 on Big Sur. This fix will >> check if dynamic linker support, which is supported from Big Sur, is >> available or not on the OS even if os.version is reported as 10.16 instead >> of 11. The os.version 10.16 doesn't include the update version like y of >> 10.x.y. Hence the change only see if it is 10.16 or not. > > test/jdk/java/lang/RuntimeTests/loadLibrary/exeLibraryCache/LibraryFromCache.java > line 45: > >> 43: public class LibraryFromCache { >> 44: public static void main(String[] args) throws IOException { >> 45: System.out.println("os.version = " + >> System.getProperty("os.version")); > > The copyright end year needs to be updated. Thanks for catching. - PR: https://git.openjdk.org/jdk/pull/9077
RFR: 8287917: System.loadLibrary does not work on Big Sur if JDK is built with macOS SDK 10.15 and earlier
Please review this PR. SDK 10.15 and earlier reports os.version as 10.16 on Big Sur. This fix will check if dynamic linker support, which is supported from Big Sur, is available or not on the OS even if os.version is reported as 10.16 instead of 11. The os.version 10.16 doesn't include the update version like y of 10.x.y. Hence the change only see if it is 10.16 or not. - Commit messages: - Remove trailing whtiespace - Move print-debug to the top - Print os.version for logging purpose - 8287917: System.loadLibrary does not work on Big Sur if JDK is built with macOS SDK 10.15 and earlier Changes: https://git.openjdk.org/jdk/pull/9077/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=9077=00 Issue: https://bugs.openjdk.org/browse/JDK-8287917 Stats: 5 lines in 2 files changed: 3 ins; 0 del; 2 mod Patch: https://git.openjdk.org/jdk/pull/9077.diff Fetch: git fetch https://git.openjdk.org/jdk pull/9077/head:pull/9077 PR: https://git.openjdk.org/jdk/pull/9077
Integrated: 8276536: Update TimeZoneNames files to follow the changes made by JDK-8275766
On Wed, 3 Nov 2021 07:05:08 GMT, Yoshiki Sato wrote: > Please review this PR to update the TimeZoneNames files. @naotoj @coffeys > > Some short name changes are not integrated to the JDK. It was detected by the > change made in JDK-8275766. > - Africa/Juba change was done by 2017c > - Africa/Windhoek changes were done by 2018e > - Antarctica/Macquarie change was done by 2017a This pull request has now been integrated. Changeset: ad3be04d Author:Yoshiki Sato Committer: Naoto Sato URL: https://git.openjdk.java.net/jdk/commit/ad3be04d2ac84836e393d696ff03ddfe72779094 Stats: 88 lines in 11 files changed: 22 ins; 0 del; 66 mod 8276536: Update TimeZoneNames files to follow the changes made by JDK-8275766 Reviewed-by: naoto, coffeys - PR: https://git.openjdk.java.net/jdk/pull/6226
RFR: 8276536: Update TimeZoneNames files to follow the changes made by JDK-8275766
Please review this PR to update the TimeZoneNames files. @naotoj @coffeys Some short name changes are not integrated to the JDK. It was detected by the change made in JDK-8275766. - Africa/Juba change was done by 2017c - Africa/Windhoek changes were done by 2018e - Antarctica/Macquarie change was done by 2017a - Commit messages: - Update affected long timezone names with its English name - Fix full timezone names of Antarctica/Macquarie - 8276536: Update TimeZoneNames files to follow the changes made by JDK-8275766 Changes: https://git.openjdk.java.net/jdk/pull/6226/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=6226=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8276536 Stats: 88 lines in 11 files changed: 22 ins; 0 del; 66 mod Patch: https://git.openjdk.java.net/jdk/pull/6226.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6226/head:pull/6226 PR: https://git.openjdk.java.net/jdk/pull/6226
Integrated: 8275766: (tz) Update Timezone Data to 2021e
On Thu, 28 Oct 2021 01:02:27 GMT, Yoshiki Sato wrote: > Please review the integration of tzdata2021e (including tzdata2021d) to the > JDK. > The fix has passed all relevant JTREG regression tests and JCK tests. > > 8275754: (tz) Update Timezone Data to 2021d > 8275849: TestZoneInfo310.java fails with tzdata2021e This pull request has now been integrated. Changeset: 5b4e3986 Author:Yoshiki Sato Committer: Sean Coffey URL: https://git.openjdk.java.net/jdk/commit/5b4e39863dbc0d61e91675261dd6887f704ab868 Stats: 52 lines in 6 files changed: 29 ins; 6 del; 17 mod 8275766: (tz) Update Timezone Data to 2021e 8275849: TestZoneInfo310.java fails with tzdata2021e Reviewed-by: naoto, iris, erikj, coffeys - PR: https://git.openjdk.java.net/jdk/pull/6144
Re: RFR: 8275766: (tz) Update Timezone Data to 2021e
On Thu, 28 Oct 2021 01:02:27 GMT, Yoshiki Sato wrote: > Please review the integration of tzdata2021e (including tzdata2021d) to the > JDK. > The fix has passed all relevant JTREG regression tests and JCK tests. > > 8275754: (tz) Update Timezone Data to 2021d > 8275849: TestZoneInfo310.java fails with tzdata2021e @coffeys please complete your review on this PR. - PR: https://git.openjdk.java.net/jdk/pull/6144
RFR: 8275766: (tz) Update Timezone Data to 2021e
Please review the integration of tzdata2021e (including tzdata2021d) to the JDK. The fix has passed all relevant JTREG regression tests and JCK tests. 8275754: (tz) Update Timezone Data to 2021d 8275849: TestZoneInfo310.java fails with tzdata2021e - Commit messages: - 8275754: (tz) Update Timezone Data to 2021d Changes: https://git.openjdk.java.net/jdk/pull/6144/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=6144=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8275766 Stats: 52 lines in 6 files changed: 29 ins; 6 del; 17 mod Patch: https://git.openjdk.java.net/jdk/pull/6144.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6144/head:pull/6144 PR: https://git.openjdk.java.net/jdk/pull/6144