Good point, I will look into it.
/Erik
On 2018-02-08 02:37, Magnus Ihse Bursie wrote:
Erik,
Is it possible that you could address
https://bugs.openjdk.java.net/browse/JDK-8193016 at this time as well?
Apart from that, it looks good to me.
/Magnus
8 feb. 2018 kl. 02:49 skrev Tim Bell
Looks good, happy pushing without a sponsor!
/Erik
On 2018-02-08 06:42, Thomas Stüfe wrote:
Hi,
may I please have reviews for this tiny fix:
Bug: https://bugs.openjdk.java.net/browse/JDK-8196488
webrev:
The check for when to generate the generated configure script is not
working quite as expected. It currently only compares timestamps if the
local repository has any local changes in the make/autoconf directory.
This used to make sense when we had a committed generated script, but
now we
Hi Eric,
On Thu, Feb 8, 2018 at 6:09 PM, Erik Joelsson
wrote:
> Looks good, happy pushing without a sponsor!
>
> /Erik
>
>
Yes please, that would be nice!
..Thomas
>
>
> On 2018-02-08 06:42, Thomas Stüfe wrote:
>
>> Hi,
>>
>> may I please have reviews for this tiny
Erik:
The check for when to generate the generated configure script is not
working quite as expected. It currently only compares timestamps if the
local repository has any local changes in the make/autoconf directory.
This used to make sense when we had a committed generated script, but
now we
Thanks, updated in place.
/Erik
On 2018-02-08 11:55, Tim Bell wrote:
Erik:
make/devkit/Tools.gmk
line 553: one typo - 'too the' should be 'to the'
Looks good otherwise.
Tim
On 02/08/18 11:42, Erik Joelsson wrote:
Here is a new webrev with dtrace included. I have verified that it
builds
Here is a new webrev with dtrace included. I have verified that it
builds open linux-x64 successfully with dtrace enabled both on my Ubuntu
without any dtrace installed as well as in mach5.
http://cr.openjdk.java.net/~erikj/8196998/webrev.02/
/Erik
On 2018-02-08 02:37, Magnus Ihse Bursie
Erik:
make/devkit/Tools.gmk
line 553: one typo - 'too the' should be 'to the'
Looks good otherwise.
Tim
On 02/08/18 11:42, Erik Joelsson wrote:
Here is a new webrev with dtrace included. I have verified that it
builds open linux-x64 successfully with dtrace enabled both on my Ubuntu
without
Looks good to me.
/Magnus
> 8 feb. 2018 kl. 20:59 skrev Erik Joelsson :
>
> Thanks, updated in place.
>
> /Erik
>
>
>> On 2018-02-08 11:55, Tim Bell wrote:
>> Erik:
>>
>> make/devkit/Tools.gmk
>>
>> line 553: one typo - 'too the' should be 'to the'
>>
>> Looks
On 07/02/2018 16:57, Lance Andersen wrote:
Hi all,
I think we are at a point where we are ready to start reviewing the changes to
remove the Java EE and CORBA modules as JEP 320, JDK-8189188, has been
targeted to JDK 11.
The CSR for removing the modules has been approved:
Erik,
Is it possible that you could address
https://bugs.openjdk.java.net/browse/JDK-8193016 at this time as well?
Apart from that, it looks good to me.
/Magnus
> 8 feb. 2018 kl. 02:49 skrev Tim Bell :
>
> Erik:
>
>> For oracle internal builds, we need to construct a
Thanks!
On Thu, Feb 8, 2018 at 3:49 PM, Lindenmaier, Goetz <
goetz.lindenma...@sap.com> wrote:
> HI Thomas,
>
> looks good, thanks!
>
> Best regards,
> Goetz.
>
> > -Original Message-
> > From: ppc-aix-port-dev [mailto:ppc-aix-port-dev-
> > boun...@openjdk.java.net] On Behalf Of Thomas
Thanks Matthias!
On Thu, Feb 8, 2018 at 3:58 PM, Baesken, Matthias
wrote:
> Hi Thomas , looks good to me( not a Reviever however).
>
>
>
> Best Regards, Matthias
>
>
>
> *From:* ppc-aix-port-dev [mailto:ppc-aix-port-dev-boun...@openjdk.java.net]
> *On Behalf Of
On 2018-01-17 15:07, Adam Farley8 wrote:
If this is the consensus, then perhaps we should consider setting
--disable-warnings-as-errors by default (in the code), rather than
depending on the user using an option which is not part of the formal
build instructions.
I'm not sure why.
Because the
> On Feb 8, 2018, at 3:04 AM, Alan Bateman wrote:
>
> On 07/02/2018 16:57, Lance Andersen wrote:
>> Hi all,
>>
>> I think we are at a point where we are ready to start reviewing the changes
>> to remove the Java EE and CORBA modules as JEP 320, JDK-8189188, has been
Looks good to me.
/Magnus
> 7 feb. 2018 kl. 23:33 skrev Erik Joelsson :
>
> When building the jdk with GCC 7.3, a bunch of new warnings are triggered.
> These include:
>
> * implicit-fallthrough
> * shift-negative-value
> * misleading-indentation
> *
Hi,
may I please have reviews for this tiny fix:
Bug: https://bugs.openjdk.java.net/browse/JDK-8196488
webrev:
http://cr.openjdk.java.net/~stuefe/webrevs/8196488-aix-toc-overflow-in-fastdebug/webrev.00/webrev/
Thanks and Kind Regards, Thomas
HI Thomas,
looks good, thanks!
Best regards,
Goetz.
> -Original Message-
> From: ppc-aix-port-dev [mailto:ppc-aix-port-dev-
> boun...@openjdk.java.net] On Behalf Of Thomas Stüfe
> Sent: Donnerstag, 8. Februar 2018 15:42
> To: ppc-aix-port-...@openjdk.java.net; build-dev
Hi Thomas , looks good to me( not a Reviever however).
Best Regards, Matthias
From: ppc-aix-port-dev [mailto:ppc-aix-port-dev-boun...@openjdk.java.net] On
Behalf Of Thomas Stüfe
Sent: Donnerstag, 8. Februar 2018 15:42
To: ppc-aix-port-...@openjdk.java.net; build-dev
19 matches
Mail list logo