Re: RFR: 8277422: tools/jar/JarEntryTime.java fails with modified time mismatch

2021-12-02 Thread Naoto Sato
On Wed, 1 Dec 2021 18:33:44 GMT, Lance Andersen  wrote:

> Hi all,
> 
> Please review this patch to address a failure at DST->STD offset transition.  
> The fix mirrors what was done for JDK-8190748
> 
> 
> Best,
> Lance

Looks good, Lance. I'd add `noreg-self` label to the JIRA issue.

-

Marked as reviewed by naoto (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/6648


Re: RFR: 8277422: tools/jar/JarEntryTime.java fails with modified time mismatch

2021-12-02 Thread Mark Sheppard
On Wed, 1 Dec 2021 18:33:44 GMT, Lance Andersen  wrote:

> Hi all,
> 
> Please review this patch to address a failure at DST->STD offset transition.  
> The fix mirrors what was done for JDK-8190748
> 
> 
> Best,
> Lance

Marked as reviewed by msheppar (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/6648


Re: RFR: 8277422: tools/jar/JarEntryTime.java fails with modified time mismatch

2021-12-01 Thread Joe Wang
On Wed, 1 Dec 2021 18:33:44 GMT, Lance Andersen  wrote:

> Hi all,
> 
> Please review this patch to address a failure at DST->STD offset transition.  
> The fix mirrors what was done for JDK-8190748
> 
> 
> Best,
> Lance

Marked as reviewed by joehw (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/6648


RFR: 8277422: tools/jar/JarEntryTime.java fails with modified time mismatch

2021-12-01 Thread Lance Andersen
Hi all,

Please review this patch to address a failure at DST->STD offset transition.  
The fix mirrors what was done for JDK-8190748


Best,
Lance

-

Commit messages:
 - Patch for JDK-8277422

Changes: https://git.openjdk.java.net/jdk/pull/6648/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=6648=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8277422
  Stats: 11 lines in 1 file changed: 9 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6648.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6648/head:pull/6648

PR: https://git.openjdk.java.net/jdk/pull/6648