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 ok with that.
The actual fix looks good to. You will still need a JDK reviewer to ok
it. Also, please notify me when you push this so that the closed version
of the configure script may also be regenerated.
/Erik
On 2013-03-13 13:18, Andrew Hughes wrote:
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 output
is:
checking if we should zip debug-info files...
with no result as $enable_zip_debug_info is unset.
This simple patch makes the option use the more standard AC_ARG_ENABLE form
used elsewhere and will
print the default ('yes') when the option is unspecified:
checking if we should zip debug-info files... yes
What actually took longer than the fix was updating the generated files. We
seem to have already settled
on autoconf 2.67 for generating the configure script, so my initial attempt
threw up a huge number of changes
as the system install is 2.69. I was able to get it down to something closer
to what is expected by installing
a local copy of 2.67 but it's still not perfect. I don't know why. I've never
been a fan of including generated
files for this reason.
So this script also updates autogen.sh to see if there is an autoconf-2.67
available and use that in preference
to autoconf if it is. I also added a little debug output so we can see which
autoconf is being used in autogen.sh.
If this is ok, can you please allocate it a bug ID and let me know which tree
to commit it to.
Thanks,
