From goetz.lindenmaier at sap.com Wed Jun 8 09:17:57 2016 From: goetz.lindenmaier at sap.com (Lindenmaier, Goetz) Date: Wed, 8 Jun 2016 09:17:57 +0000 Subject: RFR(S): 7901696: os.simpleArch is x64 for linux-ppc64le in @require context Message-ID: <07227dd93c3445e0a014d8cf2cace8b2@DEWDFE13DE09.global.corp.sap> Hi Equivalent to https://bugs.openjdk.java.net/browse/CODETOOLS-7901514 I need to test for ppc64le. We also run jtreg test with SAP JVM on zArch/s390, so I would like to fix this, too. When building, I found the doku wrong. Please review this fix. Do I need a sponsor, or may I push myself? http://cr.openjdk.java.net/~goetz/wr16/7901696-jtregFixRequires/ Best regards, Goetz. From volker.simonis at gmail.com Wed Jun 8 13:42:34 2016 From: volker.simonis at gmail.com (Volker Simonis) Date: Wed, 8 Jun 2016 15:42:34 +0200 Subject: RFR(S): 7901696: os.simpleArch is x64 for linux-ppc64le in @require context In-Reply-To: <07227dd93c3445e0a014d8cf2cace8b2@DEWDFE13DE09.global.corp.sap> References: <07227dd93c3445e0a014d8cf2cace8b2@DEWDFE13DE09.global.corp.sap> Message-ID: Hi Goetz, your change looks good, but I'm not a formal reviewer. code-tools is an independent project with its own committers and reviewers (and unfortunately not many of them :( CC-ing Aleksey, John and Jonathan with the kind request for sponsorship :) Regards, Volker [1] http://openjdk.java.net/census#code-tools On Wed, Jun 8, 2016 at 11:17 AM, Lindenmaier, Goetz wrote: > Hi > > Equivalent to https://bugs.openjdk.java.net/browse/CODETOOLS-7901514 > I need to test for ppc64le. > We also run jtreg test with SAP JVM on zArch/s390, so I would like to fix this, > too. When building, I found the doku wrong. > > Please review this fix. Do I need a sponsor, or may I push myself? > http://cr.openjdk.java.net/~goetz/wr16/7901696-jtregFixRequires/ > > Best regards, > Goetz. > > From jonathan.gibbons at oracle.com Wed Jun 8 15:38:07 2016 From: jonathan.gibbons at oracle.com (Jonathan Gibbons) Date: Wed, 08 Jun 2016 08:38:07 -0700 Subject: RFR(S): 7901696: os.simpleArch is x64 for linux-ppc64le in @require context In-Reply-To: References: <07227dd93c3445e0a014d8cf2cace8b2@DEWDFE13DE09.global.corp.sap> Message-ID: <57583BDF.2010006@oracle.com> I'm just back from vacation; I will get to this shortly. -- Jon On 06/08/2016 06:42 AM, Volker Simonis wrote: > Hi Goetz, > > your change looks good, but I'm not a formal reviewer. code-tools is > an independent project with its own committers and reviewers (and > unfortunately not many of them :( > > CC-ing Aleksey, John and Jonathan with the kind request for sponsorship :) > > Regards, > Volker > > [1] http://openjdk.java.net/census#code-tools > > On Wed, Jun 8, 2016 at 11:17 AM, Lindenmaier, Goetz > wrote: >> Hi >> >> Equivalent to https://bugs.openjdk.java.net/browse/CODETOOLS-7901514 >> I need to test for ppc64le. >> We also run jtreg test with SAP JVM on zArch/s390, so I would like to fix this, >> too. When building, I found the doku wrong. >> >> Please review this fix. Do I need a sponsor, or may I push myself? >> http://cr.openjdk.java.net/~goetz/wr16/7901696-jtregFixRequires/ >> >> Best regards, >> Goetz. >> >> From jonathan.gibbons at oracle.com Wed Jun 8 23:11:20 2016 From: jonathan.gibbons at oracle.com (Jonathan Gibbons) Date: Wed, 08 Jun 2016 16:11:20 -0700 Subject: RFR(S): 7901696: os.simpleArch is x64 for linux-ppc64le in @require context In-Reply-To: References: <07227dd93c3445e0a014d8cf2cace8b2@DEWDFE13DE09.global.corp.sap> Message-ID: <5758A618.3090107@oracle.com> Reviewed, sponsored and pushed. -- Jon On 06/08/2016 06:42 AM, Volker Simonis wrote: > Hi Goetz, > > your change looks good, but I'm not a formal reviewer. code-tools is > an independent project with its own committers and reviewers (and > unfortunately not many of them :( > > CC-ing Aleksey, John and Jonathan with the kind request for sponsorship :) > > Regards, > Volker > > [1] http://openjdk.java.net/census#code-tools > > On Wed, Jun 8, 2016 at 11:17 AM, Lindenmaier, Goetz > wrote: >> Hi >> >> Equivalent to https://bugs.openjdk.java.net/browse/CODETOOLS-7901514 >> I need to test for ppc64le. >> We also run jtreg test with SAP JVM on zArch/s390, so I would like to fix this, >> too. When building, I found the doku wrong. >> >> Please review this fix. Do I need a sponsor, or may I push myself? >> http://cr.openjdk.java.net/~goetz/wr16/7901696-jtregFixRequires/ >> >> Best regards, >> Goetz. >> >> From volker.simonis at gmail.com Thu Jun 9 06:08:10 2016 From: volker.simonis at gmail.com (Volker Simonis) Date: Thu, 9 Jun 2016 08:08:10 +0200 Subject: RFR(S): 7901696: os.simpleArch is x64 for linux-ppc64le in @require context In-Reply-To: <5758A618.3090107@oracle.com> References: <07227dd93c3445e0a014d8cf2cace8b2@DEWDFE13DE09.global.corp.sap> <5758A618.3090107@oracle.com> Message-ID: Thanks a lot Jonathan! That was really fast :) Regards, Volker On Thu, Jun 9, 2016 at 1:11 AM, Jonathan Gibbons wrote: > Reviewed, sponsored and pushed. > > -- Jon > > On 06/08/2016 06:42 AM, Volker Simonis wrote: >> >> Hi Goetz, >> >> your change looks good, but I'm not a formal reviewer. code-tools is >> an independent project with its own committers and reviewers (and >> unfortunately not many of them :( >> >> CC-ing Aleksey, John and Jonathan with the kind request for sponsorship :) >> >> Regards, >> Volker >> >> [1] http://openjdk.java.net/census#code-tools >> >> On Wed, Jun 8, 2016 at 11:17 AM, Lindenmaier, Goetz >> wrote: >>> >>> Hi >>> >>> Equivalent to https://bugs.openjdk.java.net/browse/CODETOOLS-7901514 >>> I need to test for ppc64le. >>> We also run jtreg test with SAP JVM on zArch/s390, so I would like to fix >>> this, >>> too. When building, I found the doku wrong. >>> >>> Please review this fix. Do I need a sponsor, or may I push myself? >>> http://cr.openjdk.java.net/~goetz/wr16/7901696-jtregFixRequires/ >>> >>> Best regards, >>> Goetz. >>> >>> > From goetz.lindenmaier at sap.com Thu Jun 9 06:24:42 2016 From: goetz.lindenmaier at sap.com (Lindenmaier, Goetz) Date: Thu, 9 Jun 2016 06:24:42 +0000 Subject: RFR(S): 7901696: os.simpleArch is x64 for linux-ppc64le in @require context In-Reply-To: <5758A618.3090107@oracle.com> References: <07227dd93c3445e0a014d8cf2cace8b2@DEWDFE13DE09.global.corp.sap> <5758A618.3090107@oracle.com> Message-ID: Hi Jon, thanks a lot, that was fast! Best regards, Goetz. > -----Original Message----- > From: Jonathan Gibbons [mailto:jonathan.gibbons at oracle.com] > Sent: Donnerstag, 9. Juni 2016 01:11 > To: Volker Simonis ; Lindenmaier, Goetz > > Cc: code-tools-dev at openjdk.java.net; Aleksey Shipilev > ; John Coomes > Subject: Re: RFR(S): 7901696: os.simpleArch is x64 for linux-ppc64le in > @require context > > Reviewed, sponsored and pushed. > > -- Jon > > On 06/08/2016 06:42 AM, Volker Simonis wrote: > > Hi Goetz, > > > > your change looks good, but I'm not a formal reviewer. code-tools is > > an independent project with its own committers and reviewers (and > > unfortunately not many of them :( > > > > CC-ing Aleksey, John and Jonathan with the kind request for sponsorship :) > > > > Regards, > > Volker > > > > [1] http://openjdk.java.net/census#code-tools > > > > On Wed, Jun 8, 2016 at 11:17 AM, Lindenmaier, Goetz > > wrote: > >> Hi > >> > >> Equivalent to https://bugs.openjdk.java.net/browse/CODETOOLS- > 7901514 > >> I need to test for ppc64le. > >> We also run jtreg test with SAP JVM on zArch/s390, so I would like to fix > this, > >> too. When building, I found the doku wrong. > >> > >> Please review this fix. Do I need a sponsor, or may I push myself? > >> http://cr.openjdk.java.net/~goetz/wr16/7901696-jtregFixRequires/ > >> > >> Best regards, > >> Goetz. > >> > >>