Re: [jdk21] RFR: 8313312: Add missing classpath exception copyright header

2023-08-17 Thread Roger Riggs
On Tue, 8 Aug 2023 15:55:10 GMT, Roger Riggs wrote: > The same copyright updates are needed in JDK21. Too late for jdk21 RC. Will backport to jdk21u. - PR Comment: https://git.openjdk.org/jdk21/pull/166#issuecomment-1682340185

Re: [jdk21] RFR: 8313312: Add missing classpath exception copyright header

2023-08-15 Thread Alan Bateman
On Tue, 8 Aug 2023 15:55:10 GMT, Roger Riggs wrote: > The same copyright updates are needed in JDK21. JDK 21 is in the RC phase so I assume this should re-targeted to jdk21u. - PR Comment: https://git.openjdk.org/jdk21/pull/166#issuecomment-1678567910

Re: [jdk21] RFR: 8313312: Add missing classpath exception copyright header

2023-08-09 Thread Adam Sotona
On Tue, 8 Aug 2023 15:55:10 GMT, Roger Riggs wrote: > The same copyright updates are needed in JDK21. Marked as reviewed by asotona (Reviewer). - PR Review: https://git.openjdk.org/jdk21/pull/166#pullrequestreview-1569051344

Re: [jdk21] RFR: 8313312: Add missing classpath exception copyright header

2023-08-08 Thread Naoto Sato
On Tue, 8 Aug 2023 15:55:10 GMT, Roger Riggs wrote: > The same copyright updates are needed in JDK21. Marked as reviewed by naoto (Reviewer). - PR Review: https://git.openjdk.org/jdk21/pull/166#pullrequestreview-1567795009

Re: [jdk21] RFR: 8313312: Add missing classpath exception copyright header

2023-08-08 Thread Brian Burkhalter
On Tue, 8 Aug 2023 15:55:10 GMT, Roger Riggs wrote: > The same copyright updates are needed in JDK21. +1 - Marked as reviewed by bpb (Reviewer). PR Review: https://git.openjdk.org/jdk21/pull/166#pullrequestreview-1567674852

[jdk21] RFR: 8313312: Add missing classpath exception copyright header

2023-08-08 Thread Roger Riggs
The same copyright updates are needed in JDK21. - Commit messages: - Backport 53ca75b18ea419d469758475fac8352bf915b484 Changes: https://git.openjdk.org/jdk21/pull/166/files Webrev: https://webrevs.openjdk.org/?repo=jdk21&pr=166&range=00 Issue: https://bugs.openjdk.org/browse/JDK-

Re: RFR: 8313312: Add missing classpath exception copyright header [v2]

2023-08-02 Thread Amit Kumar
On Fri, 28 Jul 2023 17:39:18 GMT, Amit Kumar wrote: >> Adds `classpath exception` in multiple files, affects copyright header only. > > Amit Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > revert java.lang.foreign changes Thanks for

Re: RFR: 8313312: Add missing classpath exception copyright header [v2]

2023-08-02 Thread Roger Riggs
On Fri, 28 Jul 2023 17:39:18 GMT, Amit Kumar wrote: >> Adds `classpath exception` in multiple files, affects copyright header only. > > Amit Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > revert java.lang.foreign changes Marked as r

Re: RFR: 8313312: Add missing classpath exception copyright header [v2]

2023-08-02 Thread Adam Sotona
On Fri, 28 Jul 2023 17:39:18 GMT, Amit Kumar wrote: >> Adds `classpath exception` in multiple files, affects copyright header only. > > Amit Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > revert java.lang.foreign changes Marked as r

Re: RFR: 8313312: Add missing classpath exception copyright header [v2]

2023-08-02 Thread Roger Riggs
On Fri, 28 Jul 2023 17:39:18 GMT, Amit Kumar wrote: >> Adds `classpath exception` in multiple files, affects copyright header only. > > Amit Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > revert java.lang.foreign changes /Reviewers

Re: RFR: 8313312: Add missing classpath exception copyright header [v2]

2023-08-02 Thread Roger Riggs
On Fri, 28 Jul 2023 17:39:18 GMT, Amit Kumar wrote: >> Adds `classpath exception` in multiple files, affects copyright header only. > > Amit Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > revert java.lang.foreign changes The changes

Re: RFR: 8313312: Add missing classpath exception copyright header [v2]

2023-08-01 Thread Amit Kumar
On Mon, 31 Jul 2023 18:13:07 GMT, Roger Riggs wrote: >Please change the bug priority to P2; that is needed to be considered. > Add a comment as described headlined with "Fix Request" and the described > rationale and information. Added comment and set priority to P2. This is low risk code chan

Re: RFR: 8313312: Add missing classpath exception copyright header [v2]

2023-07-31 Thread Roger Riggs
On Fri, 28 Jul 2023 17:39:18 GMT, Amit Kumar wrote: >> Adds `classpath exception` in multiple files, affects copyright header only. > > Amit Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > revert java.lang.foreign changes Since we're

Re: RFR: 8313312: Add missing classpath exception copyright header [v2]

2023-07-29 Thread David Holmes
On Fri, 28 Jul 2023 17:39:18 GMT, Amit Kumar wrote: >> Adds `classpath exception` in multiple files, affects copyright header only. > > Amit Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > revert java.lang.foreign changes > Please le

Re: RFR: 8313312: Add missing classpath exception copyright header [v2]

2023-07-28 Thread Amit Kumar
> Adds `classpath exception` in multiple files, affects copyright header only. Amit Kumar has updated the pull request incrementally with one additional commit since the last revision: revert java.lang.foreign changes - Changes: - all: https://git.openjdk.org/jdk/pull/15070/fil

Re: RFR: 8313312: Add missing classpath exception copyright header

2023-07-28 Thread Maurizio Cimadamore
On Fri, 28 Jul 2023 13:02:20 GMT, Amit Kumar wrote: > Adds `classpath exception` in multiple files, affects copyright header only. Please leave the java.lang.foreign changes out of this. We already have another PR for those: https://github.com/openjdk/jdk/pull/15042 - PR Comment:

RFR: 8313312: Add missing classpath exception copyright header

2023-07-28 Thread Amit Kumar
Adds `classpath exception` in multiple files, affects copyright header only. - Commit messages: - adds classpath exception Changes: https://git.openjdk.org/jdk/pull/15070/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=15070&range=00 Issue: https://bugs.openjdk.org/browse/