You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by jf...@apache.org on 2008/06/05 10:37:38 UTC
svn commit: r663499 - /apr/apr-util/trunk/build/dso.m4
Author: jfclere
Date: Thu Jun 5 01:37:38 2008
New Revision: 663499
URL: http://svn.apache.org/viewvc?rev=663499&view=rev
Log:
Otherwise install-modules-yes in Makefile and the shell will complains on
syntax error because APU_MODULES is only a space (on Solaris for example).
Modified:
apr/apr-util/trunk/build/dso.m4
Modified: apr/apr-util/trunk/build/dso.m4
URL: http://svn.apache.org/viewvc/apr/apr-util/trunk/build/dso.m4?rev=663499&r1=663498&r2=663499&view=diff
==============================================================================
--- apr/apr-util/trunk/build/dso.m4 (original)
+++ apr/apr-util/trunk/build/dso.m4 Thu Jun 5 01:37:38 2008
@@ -69,6 +69,8 @@
test $apu_have_freetds = 1 && dsos="$dsos dbd/apr_dbd_freetds.la"
test $apu_has_ldap = 1 && dsos="$dsos ldap/apr_ldap.la"
- APU_MODULES="$APU_MODULES $dsos"
+ if test -n "$dsos"; then
+ APU_MODULES="$APU_MODULES $dsos"
+ fi
fi
])
Re: svn commit: r663499 - /apr/apr-util/trunk/build/dso.m4
Posted by jean-frederic clere <jf...@gmail.com>.
jfclere@apache.org wrote:
> Author: jfclere
> Date: Thu Jun 5 01:37:38 2008
> New Revision: 663499
>
> URL: http://svn.apache.org/viewvc?rev=663499&view=rev
> Log:
> Otherwise install-modules-yes in Makefile and the shell will complains on
> syntax error because APU_MODULES is only a space (on Solaris for example).
>
> Modified:
> apr/apr-util/trunk/build/dso.m4
>
> Modified: apr/apr-util/trunk/build/dso.m4
> URL: http://svn.apache.org/viewvc/apr/apr-util/trunk/build/dso.m4?rev=663499&r1=663498&r2=663499&view=diff
> ==============================================================================
> --- apr/apr-util/trunk/build/dso.m4 (original)
> +++ apr/apr-util/trunk/build/dso.m4 Thu Jun 5 01:37:38 2008
> @@ -69,6 +69,8 @@
> test $apu_have_freetds = 1 && dsos="$dsos dbd/apr_dbd_freetds.la"
> test $apu_has_ldap = 1 && dsos="$dsos ldap/apr_ldap.la"
>
> - APU_MODULES="$APU_MODULES $dsos"
> + if test -n "$dsos"; then
> + APU_MODULES="$APU_MODULES $dsos"
> + fi
> fi
> ])
>
>
>
It seems 1.3.0 is buggy and doesn't build on my test machine for the
same reasons ;-(
Cheers
Jean-Frederic