<Beans Dev> JDK 9 RFR of JDK-8151763; Use more informative format for problem list

Mandy Chung mandy.chung at oracle.com
Tue Mar 29 19:31:21 UTC 2016


> On Mar 29, 2016, at 12:15 PM, joe darcy <joe.darcy at oracle.com> wrote:
> 
> Hi Mandy,
> 
> On 3/28/2016 8:48 PM, Mandy Chung wrote:
>>> On Mar 28, 2016, at 5:03 PM, Joseph D. Darcy <joe.darcy at oracle.com> wrote:
>>> 
>>> Hello,
>>> 
>>> New iteration of the webrev updated after the Jigsaw integration and incorporating the earlier feedback.
>>> 
>>>    http://cr.openjdk.java.net/~darcy/8151763.1
>>> 
>> # tools/jimage/JImageTest.java                                    linux-i586,windows-i586
>> 
>> Is this test accidentally removed?  Other than this, looks okay.
> 
> The "#" lines are comments so I was removing a commented out line. (I assumed, but did not verify, the line for this test was a leftover artifact of the recent Jigsaw merge.)

I missed “#” since this test should be excluded (some error might have been creeped in before the integration)

This test needs to be added back in the problem list.  I’ll create a changeset.

> 
>> 
>> Nit: it’d be good to have most of bug ids aligned in the same column start.
>> Here are a few ones:
>> 
>>  210 sun/security/krb5/auto/Unreachable.java     7164518 macosx-all no PortUnreachableException on Mac
>>  212 java/security/KeyPairGenerator/SolarisShortDSA.java 7041639 solaris-all Solaris DSA keypair generation bug
>>  213 sun/security/tools/keytool/standard.sh              7041639 solaris-all Solaris DSA keypair generation bug
>>  346 java/util/Arrays/ParallelPrefix.java 8080165,8085982 generic-all
>>  348 java/util/BitSet/BitSetStreamTest.java 8079538 generic-all
>>  360 sun/tools/jmap/heapconfig/JMapHeapConfigTest.java 8072131,8132452 generic-all
>>  370 sun/tools/jinfo/JInfoSanityTest.java                           8059035 generic-all
>> 
>> 
> 
> I was trying to avoid introducing lots of spacing changes in an attempt to make the patch easier to review, but I can look over these cases again.

That’d be good.  Thanks
Mandy


More information about the beans-dev mailing list