Re: RFR: jdk9: 8029997: [infra] remove Solaris ISA directories and the links

2013-12-23 Thread Kumar Srinivasan


On 12/21/2013 12:08 AM, Alan Bateman wrote:

On 20/12/2013 23:03, Kumar Srinivasan wrote:

Hello,

Please review the removal of  ISA (Instruction Specific Architecture) 
directories namely sparcv9, amd64
and the symlinks in these directories,   this was provided to aid 
transition to jdk8, where solaris 32-bit was

removed, and the 32-bit binaries were replaced with 64-bit versions.

http://cr.openjdk.java.net/~ksrini/8029997/webrev.0/
This looks good to me too (I assumed there would be several tests that 
needed adjustment, it seems not).



There were :),  I  nailed those when I removed 32-bit  in jdk8.

Thanks Tim and Alan.

Kumar



-Alan.




Re: RFR: jdk9: 8029997: [infra] remove Solaris ISA directories and the links

2013-12-21 Thread Alan Bateman

On 20/12/2013 23:03, Kumar Srinivasan wrote:

Hello,

Please review the removal of  ISA (Instruction Specific Architecture) 
directories namely sparcv9, amd64
and the symlinks in these directories,   this was provided to aid 
transition to jdk8, where solaris 32-bit was

removed, and the 32-bit binaries were replaced with 64-bit versions.

http://cr.openjdk.java.net/~ksrini/8029997/webrev.0/
This looks good to me too (I assumed there would be several tests that 
needed adjustment, it seems not).


-Alan.


Re: RFR: jdk9: 8029997: [infra] remove Solaris ISA directories and the links

2013-12-20 Thread Tim Bell

Hi Kumar:

Please review the removal of  ISA (Instruction Specific Architecture) 
directories namely sparcv9, amd64
and the symlinks in these directories,   this was provided to aid 
transition to jdk8, where solaris 32-bit was

removed, and the 32-bit binaries were replaced with 64-bit versions.

http://cr.openjdk.java.net/~ksrini/8029997/webrev.0/


Three cheers for code deletion!  Looks good to me.

Tim



RFR: jdk9: 8029997: [infra] remove Solaris ISA directories and the links

2013-12-20 Thread Kumar Srinivasan

Hello,

Please review the removal of  ISA (Instruction Specific Architecture) 
directories namely sparcv9, amd64
and the symlinks in these directories,   this was provided to aid 
transition to jdk8, where solaris 32-bit was

removed, and the 32-bit binaries were replaced with 64-bit versions.

http://cr.openjdk.java.net/~ksrini/8029997/webrev.0/

Thanks
Kumar