PING: Re: Code Review Request: Simple new build system fix

2013-03-25 Thread Andrew Hughes
- Original Message - > > > On 2013-03-13 15:01, Andrew Hughes wrote: > > - Original Message - > >> Hello, > >> > >> I created a bug for you: > >> > >> 8009988: build-infra: Fix configure output for zip debuginfo check > >> > >> As David says, we haven't decided on 2.67, but I woul

Re: Code Review Request: Simple new build system fix

2013-03-13 Thread David Holmes
On 13/03/2013 11:58 PM, Andrew Hughes wrote: - Original Message - Andrew, FWIW we are not tied to autoconf 2.67. There have already been a number of pushes using 2.68 and now 2.69 is also showing up. Yes, Omair was kind enough to point me to this changeset which indeed uses 2.68: ht

Re: Code Review Request: Simple new build system fix

2013-03-13 Thread Erik Joelsson
On 2013-03-13 15:01, Andrew Hughes wrote: - Original Message - Hello, I created a bug for you: 8009988: build-infra: Fix configure output for zip debuginfo check As David says, we haven't decided on 2.67, but I would guess that a majority of the commits have been with that version.

Re: Code Review Request: Simple new build system fix

2013-03-13 Thread Andrew Hughes
- Original Message - > Hello, > > I created a bug for you: > > 8009988: build-infra: Fix configure output for zip debuginfo check > > As David says, we haven't decided on 2.67, but I would guess that a > majority of the commits have been with that version. This change is a > first step t

Re: Code Review Request: Simple new build system fix

2013-03-13 Thread Andrew Hughes
- Original Message - > Andrew, > > FWIW we are not tied to autoconf 2.67. There have already been a > number > of pushes using 2.68 and now 2.69 is also showing up. > Yes, Omair was kind enough to point me to this changeset which indeed uses 2.68: http://hg.openjdk.java.net/jdk8/build/r

Re: Code Review Request: Simple new build system fix

2013-03-13 Thread Erik Joelsson
Hello, I created a bug for you: 8009988: build-infra: Fix configure output for zip debuginfo check As David says, we haven't decided on 2.67, but I would guess that a majority of the commits have been with that version. This change is a first step towards enforcing a specific version and I'm

Re: Code Review Request: Simple new build system fix

2013-03-13 Thread David Holmes
Andrew, FWIW we are not tied to autoconf 2.67. There have already been a number of pushes using 2.68 and now 2.69 is also showing up. David On 13/03/2013 10:18 PM, Andrew Hughes wrote: I've finally found time to look at the new build system (well, there seems to no longer be any choice ;) a

Code Review Request: Simple new build system fix

2013-03-13 Thread Andrew Hughes
I've finally found time to look at the new build system (well, there seems to no longer be any choice ;) and so thought I start out with a simple fix. http://cr.openjdk.java.net/~andrew/build/zip_debug_info/webrev.01/ At the moment, if disable-zip-debug-info is not specified, the configure outpu