You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by wr...@apache.org on 2006/03/24 20:15:52 UTC

svn commit: r388605 - /apr/apr-util/trunk/xml/expat/buildconf.sh

Author: wrowe
Date: Fri Mar 24 11:15:49 2006
New Revision: 388605

URL: http://svn.apache.org/viewcvs?rev=388605&view=rev
Log:

  Apply the same cache pre-clean/post-clean m4's version collision avoidance
  that we use in aprutil itself, to the bundled expat.



Modified:
    apr/apr-util/trunk/xml/expat/buildconf.sh

Modified: apr/apr-util/trunk/xml/expat/buildconf.sh
URL: http://svn.apache.org/viewcvs/apr/apr-util/trunk/xml/expat/buildconf.sh?rev=388605&r1=388604&r2=388605&view=diff
==============================================================================
--- apr/apr-util/trunk/xml/expat/buildconf.sh (original)
+++ apr/apr-util/trunk/xml/expat/buildconf.sh Fri Mar 24 11:15:49 2006
@@ -50,6 +50,9 @@
 
 cross_compile_warning="warning: AC_TRY_RUN called without default to allow cross compiling"
 
+# Remove aclocal.m4 as it'll break some builds...
+rm -rf aclocal.m4 autom4te*.cache
+
 #
 # Generate the autoconf header template (config.h.in) and ./configure
 #



Re: svn commit: r388605 - /apr/apr-util/trunk/xml/expat/buildconf.sh

Posted by "William A. Rowe, Jr." <wr...@rowe-clan.net>.
Justin Erenkrantz wrote:
> On Fri, Mar 24, 2006 at 07:15:52PM -0000, wrowe@apache.org wrote:
> 
>>Author: wrowe
>>Date: Fri Mar 24 11:15:49 2006
>>New Revision: 388605
>>
>>URL: http://svn.apache.org/viewcvs?rev=388605&view=rev
>>Log:
>>
>>  Apply the same cache pre-clean/post-clean m4's version collision avoidance
>>  that we use in aprutil itself, to the bundled expat.
> 
> Uh.  No.  Thanks for more build breakage.

Anytime :)  Sorry I haven't been more active previously, work and RL have
conspired to keep me absent from APR for far too long.  Anyways, I'm back :)

>>+# Remove aclocal.m4 as it'll break some builds...
>>+rm -rf aclocal.m4 autom4te*.cache
> 
> This removes the aclocal.m4 created by libtoolize.  This means that
> configure doesn't get created properly as the AC_PROG_LIBTOOL* macros
> aren't available when autoconf is run.  So, this busts the build.

Ah HA.  Thank you I was running this question down myself; it's the only
remaining reason there's no new 0.9.11 packages yet.  There will be, shortly
if this solves that quirk!

> There's no reason to remove these files 

Ok, I'll delete those lines in apr/buildconf apr-util/buildconf when I have
a few minutes to break folks other than yourself (after all everyone deserves
their turn :0)

> and it's certainly wrong to do so at this point.  -- justin

With -that- I will agree, thank you for explaining my own observations, I'll
roll this change up prior to libtoolize.  Thank you!

Bill

Re: svn commit: r388605 - /apr/apr-util/trunk/xml/expat/buildconf.sh

Posted by Justin Erenkrantz <ju...@erenkrantz.com>.
On Fri, Mar 24, 2006 at 07:15:52PM -0000, wrowe@apache.org wrote:
> Author: wrowe
> Date: Fri Mar 24 11:15:49 2006
> New Revision: 388605
> 
> URL: http://svn.apache.org/viewcvs?rev=388605&view=rev
> Log:
> 
>   Apply the same cache pre-clean/post-clean m4's version collision avoidance
>   that we use in aprutil itself, to the bundled expat.

Uh.  No.  Thanks for more build breakage.

>  cross_compile_warning="warning: AC_TRY_RUN called without default to allow cross compiling"
>  
> +# Remove aclocal.m4 as it'll break some builds...
> +rm -rf aclocal.m4 autom4te*.cache
> +

This removes the aclocal.m4 created by libtoolize.  This means that
configure doesn't get created properly as the AC_PROG_LIBTOOL* macros
aren't available when autoconf is run.  So, this busts the build.

There's no reason to remove these files and it's certainly wrong to do so
at this point.  -- justin