Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-18 Thread Paul Sandoz
> On 18 Dec 2017, at 17:52, David Holmes wrote: > > On 19/12/2017 11:42 AM, Paul Sandoz wrote: >> Looks good to me. >> I am including HS dev for the class file version changes: >> http://cr.openjdk.java.net/~psandoz/jdk/JDK-8173382-classfile-version/webrev/ >>

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-18 Thread David Holmes
On 19/12/2017 11:42 AM, Paul Sandoz wrote: Looks good to me. I am including HS dev for the class file version changes: http://cr.openjdk.java.net/~psandoz/jdk/JDK-8173382-classfile-version/webrev/ http://cr.openjdk.java.net/~psandoz/jdk/JDK-8173382-classfile-version-build-changes/webrev/ We

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-18 Thread Joseph D. Darcy
Thanks Paul. I'm doing final checks and preparations of fixes to JDK-8173382: Add -source 11 and -target 11 to javac JDK-8193291: Add SourceVersion.RELEASE_11 and expect to push shortly. Cheers, -Joe On 12/18/2017 5:42 PM, Paul Sandoz wrote: Looks good to me. I am including HS dev

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-18 Thread Paul Sandoz
Looks good to me. I am including HS dev for the class file version changes: http://cr.openjdk.java.net/~psandoz/jdk/JDK-8173382-classfile-version/webrev/

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-15 Thread Erik Joelsson
That sounds good to me. /Erik On 2017-12-15 19:40, Paul Sandoz wrote: Hi Iris, Erik We should be able to automate all the following by referring to the latest release variable, yes? Nashorn build targets version 9 source https://bugs.openjdk.java.net/browse/JDK-8188012

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-15 Thread Paul Sandoz
Here are two webrevs (in order that can be merged with the other): the class file version changes; and the build changes related to that. http://cr.openjdk.java.net/~psandoz/jdk/JDK-8173382-classfile-version/webrev/

RE: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-15 Thread Iris Clark
Hi, Paul. > We should be able to automate all the following by referring to the latest > release variable, yes? [ ... ] > Update link to license in Docs.gmk > https://bugs.openjdk.java.net/browse/JDK-818999 > http://hg.openjdk.java.net/jdk10/master/rev/a6e591e12f12 Yes! iris

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-15 Thread Paul Sandoz
Hi Iris, Erik We should be able to automate all the following by referring to the latest release variable, yes? Nashorn build targets version 9 source https://bugs.openjdk.java.net/browse/JDK-8188012 http://hg.openjdk.java.net/jdk10/master/rev/0e67ab18b511 symbolgenerator targets jdk 9

RE: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-15 Thread Iris Clark
Hi. Another build target we need to update is LICENSE_URL in make/Docs.gmk. Here's the equivalent bug for 10: https://bugs.openjdk.java.net/browse/JDK-8189919 I've requested this link: http://www.oracle.com/technetwork/java/javase/terms/license/java11speclicense.html Thanks, iris

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-15 Thread Erik Joelsson
Looking at JDK-8188015, I was just reminded of JDK-8188012 and JDK-8188013 which indicates that we have more than one place where 10 needs to be bumped to 11 make/BuildNashorn.gmk make/langtools/src/classes/build/tools/symbolgenerator/TransitiveDependencies.java /Erik On 2017-12-12 16:36,

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-14 Thread Paul Sandoz
> On 13 Dec 2017, at 19:51, mark.reinh...@oracle.com wrote: > > 2017/12/13 15:49:53 -0800, paul.san...@oracle.com: >> On 13 Dec 2017, at 14:36, mark.reinh...@oracle.com wrote: >>> I understand that other incoming changes are waiting for the 11 version >>> bump, but can't we at least do the

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-14 Thread Paul Sandoz
> On 14 Dec 2017, at 03:58, Erik Joelsson wrote: > > And here it is: http://cr.openjdk.java.net/~erikj/8173382/webrev.01/ > > Feel free to incorporate this into whichever changeset you want, or I can > push this separately. > Thanks! that is exactly what i was

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-14 Thread Erik Joelsson
And here it is: http://cr.openjdk.java.net/~erikj/8173382/webrev.01/ Feel free to incorporate this into whichever changeset you want, or I can push this separately. /Erik On 2017-12-14 10:37, Erik Joelsson wrote: On 2017-12-14 05:40, David Holmes wrote: On 14/12/2017 1:51 PM,

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-14 Thread Erik Joelsson
On 2017-12-14 05:40, David Holmes wrote: On 14/12/2017 1:51 PM, mark.reinh...@oracle.com wrote: I just went through this for the versioning changes.  The attached patch (against d2a837cf9ff1) does what you want, I think. We can't do that.

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread David Holmes
On 14/12/2017 1:51 PM, mark.reinh...@oracle.com wrote: 2017/12/13 15:49:53 -0800, paul.san...@oracle.com: On 13 Dec 2017, at 14:36, mark.reinh...@oracle.com wrote: I understand that other incoming changes are waiting for the 11 version bump, but can't we at least do the straightforward

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread mark . reinhold
2017/12/13 15:49:53 -0800, paul.san...@oracle.com: > On 13 Dec 2017, at 14:36, mark.reinh...@oracle.com wrote: >> I understand that other incoming changes are waiting for the 11 version >> bump, but can't we at least do the straightforward parameterizations and >> introduce _CURRENT constants in

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread joe darcy
Hi David, On 12/13/2017 5:16 PM, David Holmes wrote: Hi Joe, On 14/12/2017 4:36 AM, joe darcy wrote: There are various changes associated with a JDK N to (N+1) update including (but not limited to): 1) New -source/-target value in javac 2) Make new -source/-target value the

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread David Holmes
Hi Joe, On 14/12/2017 4:36 AM, joe darcy wrote: There are various changes associated with a JDK N to (N+1) update including (but not limited to):     1) New -source/-target value in javac     2) Make new -source/-target value the default in javac     3) Change build of the JDK to use new

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread joe darcy
Hello, On 12/13/2017 1:18 PM, mark.reinh...@oracle.com wrote: 2017/12/13 8:44:33 -0800, paul.san...@oracle.com: On 12 Dec 2017, at 20:56, david.hol...@oracle.com wrote: [snip] Looking at Joe's webrev, many of the changes from 10 to 11 could instead be changed to

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread Paul Sandoz
> On 13 Dec 2017, at 14:36, mark.reinh...@oracle.com wrote: > > 2017/12/13 14:09:44 -0800, paul.san...@oracle.com: >>> On 13 Dec 2017, at 13:18, mark.reinh...@oracle.com wrote: >>> How much of this can we parameterize and/or automate? >> >> I suspect quite a bit, as you present below, and i

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread mark . reinhold
2017/12/13 14:09:44 -0800, paul.san...@oracle.com: >> On 13 Dec 2017, at 13:18, mark.reinh...@oracle.com wrote: >> How much of this can we parameterize and/or automate? > > I suspect quite a bit, as you present below, and i agree we should try > and automate as much as possible. At the moment

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread Brian Goetz
Don't forget to add the new version to the list of validated versions in multi-release JARs... On 12/13/2017 1:36 PM, joe darcy wrote: There are various changes associated with a JDK N to (N+1) update including (but not limited to):     1) New -source/-target value in javac     2) Make new

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread Paul Sandoz
> On 13 Dec 2017, at 13:18, mark.reinh...@oracle.com wrote: > > 2017/12/13 8:44:33 -0800, paul.san...@oracle.com: >>> On 12 Dec 2017, at 20:56, david.hol...@oracle.com wrote: >>> >>> Anyway, none of the proposed changes have any impact on hotspot >>> AFAICT. It is only when the actual version

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread mark . reinhold
2017/12/13 8:44:33 -0800, paul.san...@oracle.com: >> On 12 Dec 2017, at 20:56, david.hol...@oracle.com wrote: >> >> Anyway, none of the proposed changes have any impact on hotspot >> AFAICT. It is only when the actual version is updated to 11 that >> hotspot, and other entities will have to be

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread Jonathan Gibbons
On 12/13/17 10:36 AM, joe darcy wrote: Hi David, On 12/12/2017 8:56 PM, David Holmes wrote: Hi Joe, On 13/12/2017 9:20 AM, joe darcy wrote: Hi David, On 12/12/2017 1:32 PM, David Holmes wrote: [snip] For background, what we've done in the past is at the very start of a new JDK

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread joe darcy
Hi David, On 12/12/2017 8:56 PM, David Holmes wrote: Hi Joe, On 13/12/2017 9:20 AM, joe darcy wrote: Hi David, On 12/12/2017 1:32 PM, David Holmes wrote: [snip] For background, what we've done in the past is at the very start of a new JDK release, first create -source/-target

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread Paul Sandoz
Joe, Here's a patch on top of your patch: http://cr.openjdk.java.net/~psandoz/jdk/JDK-8173382-release-classfile-version/webrev/ Currently building/testing with both patches applied. Paul. > On 13 Dec

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-13 Thread Paul Sandoz
> On 12 Dec 2017, at 20:56, David Holmes wrote: > > Anyway, none of the proposed changes have any impact on hotspot AFAICT. It is > only when the actual version is updated to 11 that hotspot, and other > entities will have to be updated. I'm still unclear if someone

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread David Holmes
Hi Joe, On 13/12/2017 9:20 AM, joe darcy wrote: Hi David, On 12/12/2017 1:32 PM, David Holmes wrote: Hi Joe, On 13/12/2017 7:27 AM, joe darcy wrote: Hi David, On 12/12/2017 1:11 PM, David Holmes wrote: Hi Joe, On 12/12/2017 5:04 PM, joe darcy wrote: Hello, With the JDK 11 line of

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread joe darcy
Hi David, On 12/12/2017 1:32 PM, David Holmes wrote: Hi Joe, On 13/12/2017 7:27 AM, joe darcy wrote: Hi David, On 12/12/2017 1:11 PM, David Holmes wrote: Hi Joe, On 12/12/2017 5:04 PM, joe darcy wrote: Hello, With the JDK 11 line of development opening up in a few days, a few changes

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread David Holmes
Hi Joe, On 13/12/2017 7:27 AM, joe darcy wrote: Hi David, On 12/12/2017 1:11 PM, David Holmes wrote: Hi Joe, On 12/12/2017 5:04 PM, joe darcy wrote: Hello, With the JDK 11 line of development opening up in a few days, a few changes are needed to prepared to turn a JDK 10 code base into a

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread joe darcy
Hi David, On 12/12/2017 1:11 PM, David Holmes wrote: Hi Joe, On 12/12/2017 5:04 PM, joe darcy wrote: Hello, With the JDK 11 line of development opening up in a few days, a few changes are needed to prepared to turn a JDK 10 code base into a JDK 11 one including: JDK-8173382: Add

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread David Holmes
Hi Joe, On 12/12/2017 5:04 PM, joe darcy wrote: Hello, With the JDK 11 line of development opening up in a few days, a few changes are needed to prepared to turn a JDK 10 code base into a JDK 11 one including:     JDK-8173382: Add -source 11 and -target 11 to javac     JDK-8193291: Add

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread Paul Sandoz
> On 12 Dec 2017, at 10:03, Paul Sandoz wrote: > > > >> On 12 Dec 2017, at 09:52, joe darcy wrote: >> >> Hi Paul, >> >> There is sense in working in including the class file version update at the >> same time if the added complexity doesn't

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread Paul Sandoz
> On 12 Dec 2017, at 09:52, joe darcy wrote: > > Hi Paul, > > There is sense in working in including the class file version update at the > same time if the added complexity doesn't delay the changes. > I sense it should be so complex and delay since the additional

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread joe darcy
Hi Paul, There is sense in working in including the class file version update at the same time if the added complexity doesn't delay the changes. Did the patch for the increment to 54 require previous changes in the bundled ASM? Thanks, -Joe On 12/12/2017 9:31 AM, Paul Sandoz wrote: Hi

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread Paul Sandoz
Hi Joe, I would like to see the class file version increment [*] in the same patch (the unified repo makes this easier now). Then we have everything in one changeset (except for any ctsym changes?) that can be easily templated when the next increment is required, since we will be doing this

Re: Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-12 Thread Erik Joelsson
Build change looks good. /Erik On 2017-12-12 08:04, joe darcy wrote: Hello, With the JDK 11 line of development opening up in a few days, a few changes are needed to prepared to turn a JDK 10 code base into a JDK 11 one including:     JDK-8173382: Add -source 11 and -target 11 to javac   

Initial JDK 11 RFR of JDK-8173382: Add -source 11 and -target 11 to javac - Java Bug System & JDK-8193291: Add SourceVersion.RELEASE_11

2017-12-11 Thread joe darcy
Hello, With the JDK 11 line of development opening up in a few days, a few changes are needed to prepared to turn a JDK 10 code base into a JDK 11 one including: JDK-8173382: Add -source 11 and -target 11 to javac JDK-8193291: Add SourceVersion.RELEASE_11 Webev: