Re: RFR: jdk9: 8029997: [infra] remove Solaris ISA directories and the links
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
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
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
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