How to increment JDK_MINOR_VERSION for JDK 9? -- request for review

Joe Darcy joe.darcy at oracle.com
Fri Dec 20 04:17:35 UTC 2013


On 12/19/2013 11:40 AM, Bradford Wetmore wrote:
> This is blocking JDK-8030823 (JCE prebuilt jar files), so I looked at 
> it also.
>
> Ditto the comments from Erik.  The plan is these @ignores will be 
> pulled out when 8030656 is fixed?

Exactly; thanks,

-Joe

>
> Brad
>
>
> On 12/19/2013 1:38 AM, Erik Joelsson wrote:
>> Build change looks good to me. I'm not familiar with the tests so can't
>> comment on those.
>>
>> /Erik
>>
>>
>> On 2013-12-18 19:19, Joe Darcy wrote:
>>> On 12/16/2013 02:14 PM, Joe Darcy wrote:
>>>> On 12/15/2013 09:50 PM, Joe Darcy wrote:
>>>>> On 12/10/2013 03:51 PM, David Katleman wrote:
>>>>>>
>>>>>> On 12/10/2013 3:33 PM, Joseph Darcy wrote:
>>>>>>> Hello,
>>>>>>>
>>>>>>> Looking ahead to JDK 9, a build 01 change that is needed is
>>>>>>> updating the JDK_MINOR_VERSION setting in the build from "8" to 
>>>>>>> "9":
>>>>>>>
>>>>>>>         JDK-8000962 Update JDK_MINOR_VERSION for JDK 9
>>>>>>> https://bugs.openjdk.java.net/browse/JDK-8000962
>>>>>>>
>>>>>>> I poked around a bit, but didn't see where this needs to be set.
>>>>>>> The value of this variable gets used in numerous places throughout
>>>>>>> the JDK, including in system properties.
>>>>>>>
>>>>>>> How can this be updating in the new build?
>>>>>>
>>>>>> That would be in common/autoconf/version-numbers
>>>>>>
>>>>>>         Dave
>>>>>>
>>>>>
>>>>> An update, I've done a build after bumping up that value, but I ran
>>>>> into testing issue. It seems that the jtreg needs a minor update to
>>>>> allow for the successful running of TestNG tests after the minor
>>>>> version is incremented; I've contacted the jtreg team and a fix is
>>>>> in progress.
>>>>>
>>>>> Thanks,
>>>>>
>>>>> -Joe
>>>>
>>>> An update on the update: a new jtreg build resolves the TestNG issue.
>>>> After a new build of jtreg is promoted, I'll go ahead with the
>>>> JDK_MINOR_VERSIO update.
>>>>
>>>> Cheers,
>>>>
>>>> -Joe
>>>
>>> Please review this increment in common/autoconf/version-numbers along
>>> with test updates to work around a HotSpot bug:
>>>
>>>     http://cr.openjdk.java.net/~darcy/8000962.0/
>>>
>>> Patch below.
>>>
>>> Thanks,
>>>
>>> -Joe
>>>
>>> --- old/common/autoconf/version-numbers    2013-12-18
>>> 09:12:06.000000000 -0800
>>> +++ new/common/autoconf/version-numbers    2013-12-18
>>> 09:12:06.000000000 -0800
>>> @@ -24,7 +24,7 @@
>>>  #
>>>
>>>  JDK_MAJOR_VERSION=1
>>> -JDK_MINOR_VERSION=8
>>> +JDK_MINOR_VERSION=9
>>>  JDK_MICRO_VERSION=0
>>>  JDK_UPDATE_VERSION=
>>>  LAUNCHER_NAME=openjdk
>>> ---
>>> old/langtools/test/tools/javac/MethodParameters/AnnotationTest.java
>>> 2013-12-18 09:12:07.000000000 -0800
>>> +++
>>> new/langtools/test/tools/javac/MethodParameters/AnnotationTest.java
>>> 2013-12-18 09:12:07.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters AnnotationTest.java
>>> ---
>>> old/langtools/test/tools/javac/MethodParameters/AnonymousClass.java
>>> 2013-12-18 09:12:07.000000000 -0800
>>> +++
>>> new/langtools/test/tools/javac/MethodParameters/AnonymousClass.java
>>> 2013-12-18 09:12:07.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters AnonymousClass.java
>>> --- old/langtools/test/tools/javac/MethodParameters/CaptureTest.java
>>> 2013-12-18 09:12:07.000000000 -0800
>>> +++ new/langtools/test/tools/javac/MethodParameters/CaptureTest.java
>>> 2013-12-18 09:12:07.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8015701
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary Test method parameter attribute generation with captured
>>> locals.
>>>   * @compile -parameters CaptureTest.java
>>>   * @run main CaptureTest
>>> --- old/langtools/test/tools/javac/MethodParameters/Constructors.java
>>> 2013-12-18 09:12:08.000000000 -0800
>>> +++ new/langtools/test/tools/javac/MethodParameters/Constructors.java
>>> 2013-12-18 09:12:08.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters Constructors.java
>>> --- old/langtools/test/tools/javac/MethodParameters/EnumTest.java
>>> 2013-12-18 09:12:08.000000000 -0800
>>> +++ new/langtools/test/tools/javac/MethodParameters/EnumTest.java
>>> 2013-12-18 09:12:08.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582 8008658
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters EnumTest.java
>>> ---
>>> old/langtools/test/tools/javac/MethodParameters/InstanceMethods.java
>>> 2013-12-18 09:12:09.000000000 -0800
>>> +++
>>> new/langtools/test/tools/javac/MethodParameters/InstanceMethods.java
>>> 2013-12-18 09:12:09.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters InstanceMethods.java
>>> --- old/langtools/test/tools/javac/MethodParameters/LambdaTest.java
>>> 2013-12-18 09:12:09.000000000 -0800
>>> +++ new/langtools/test/tools/javac/MethodParameters/LambdaTest.java
>>> 2013-12-18 09:12:09.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters LambdaTest.java
>>> ---
>>> old/langtools/test/tools/javac/MethodParameters/LocalClassTest.java
>>> 2013-12-18 09:12:09.000000000 -0800
>>> +++
>>> new/langtools/test/tools/javac/MethodParameters/LocalClassTest.java
>>> 2013-12-18 09:12:09.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582 8008658
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters LocalClassTest.java
>>> ---
>>> old/langtools/test/tools/javac/MethodParameters/MemberClassTest.java
>>> 2013-12-18 09:12:10.000000000 -0800
>>> +++
>>> new/langtools/test/tools/javac/MethodParameters/MemberClassTest.java
>>> 2013-12-18 09:12:10.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582 8008658
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters MemberClassTest.java
>>> --- old/langtools/test/tools/javac/MethodParameters/StaticMethods.java
>>> 2013-12-18 09:12:10.000000000 -0800
>>> +++ new/langtools/test/tools/javac/MethodParameters/StaticMethods.java
>>> 2013-12-18 09:12:10.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters StaticMethods.java
>>> ---
>>> old/langtools/test/tools/javac/MethodParameters/UncommonParamNames.java 
>>> 2013-12-18
>>> 09:12:10.000000000 -0800
>>> +++
>>> new/langtools/test/tools/javac/MethodParameters/UncommonParamNames.java 
>>> 2013-12-18
>>> 09:12:10.000000000 -0800
>>> @@ -24,6 +24,7 @@
>>>  /*
>>>   * @test
>>>   * @bug 8006582
>>> + * @ignore 8030656 Bad version check for parameter information in
>>> src/share/vm/classfile/javaClasses.cpp
>>>   * @summary javac should generate method parameters correctly.
>>>   * @build Tester
>>>   * @compile -parameters UncommonParamNames.java
>>> --- old/jdk/test/ProblemList.txt    2013-12-18 09:12:11.000000000 -0800
>>> +++ new/jdk/test/ProblemList.txt    2013-12-18 09:12:11.000000000 -0800
>>> @@ -123,6 +123,11 @@
>>>  # 8029415
>>>  java/lang/reflect/Method/invoke/TestPrivateInterfaceMethodReflect.java 
>>> generic-all
>>>
>>>
>>> +# 8030656
>>> +java/lang/reflect/Parameter/GetAnnotatedTypeTest.java generic-all
>>> +java/lang/reflect/Parameter/WithParameters.java generic-all
>>> +java/lang/reflect/Parameter/BadClassFiles.java generic-all
>>> +
>>>  ############################################################################ 
>>>
>>>
>>>
>>>  # jdk_management
>>>
>>




More information about the build-dev mailing list