Re: RFR: 8287917: System.loadLibrary does not work on Big Sur if JDK is built with macOS SDK 10.15 and earlier [v2]

2022-06-13 Thread Yoshiki Sato
> 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

2022-06-13 Thread Yoshiki Sato
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

2022-06-09 Thread Yoshiki Sato
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

2021-11-10 Thread Yoshiki Sato
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

2021-11-09 Thread Yoshiki Sato
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

2021-11-02 Thread Yoshiki Sato
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

2021-11-01 Thread Yoshiki Sato
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

2021-10-28 Thread Yoshiki Sato
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