-1 on this change. Please back out.

Expat might not be located in xml/expat. It is entirely possible/reasonable
that we found Expat already installed on the system.

Look at apr-util/build/apu-conf.m4 for more information. Specifically: we
should have already put the expat libs into the LIBS symbol. If it isn't
making it into the LIBS symbol in test/Makefile.in, then there is another
problem -- and your patch is not solving that problem.

Cheers,
-g

On Wed, Aug 22, 2001 at 09:32:14PM -0000, [EMAIL PROTECTED] wrote:
> ianh        01/08/22 14:32:14
> 
>   Modified:    test     Makefile.in
>   Log:
>   add some the expat libs to the link line, and change it from LIB to ALL_LIB 
> so that
>   it can link.
>   
>   Revision  Changes    Path
>   1.14      +7 -7      apr-util/test/Makefile.in
>   
>   Index: Makefile.in
>   ===================================================================
>   RCS file: /home/cvs/apr-util/test/Makefile.in,v
>   retrieving revision 1.13
>   retrieving revision 1.14
>   diff -u -r1.13 -r1.14
>   --- Makefile.in     2001/08/07 23:02:40     1.13
>   +++ Makefile.in     2001/08/22 21:32:14     1.14
>   @@ -9,21 +9,21 @@
>    test: $(PROGRAMS)
>    
>    testdbm_OBJECTS = testdbm.lo
>   -testdbm_LDADD =  ../libaprutil.la @APR_SOURCE_DIR@/libapr.la
>   +testdbm_LDADD =  ../libaprutil.la @APR_SOURCE_DIR@/libapr.la 
> ../xml/expat/lib/libexpat.la
>    testdbm: $(testdbm_OBJECTS) $(testdbm_LDADD)
>   -   $(LINK) $(testdbm_OBJECTS) $(testdbm_LDADD) $(LIBS)
>   +   $(LINK) $(testdbm_OBJECTS) $(testdbm_LDADD) $(ALL_LIBS)
>    
>    testdate_OBJECTS = testdate.lo
>   -testdate_LDADD =  ../libaprutil.la @APR_SOURCE_DIR@/libapr.la
>   +testdate_LDADD =  ../libaprutil.la @APR_SOURCE_DIR@/libapr.la 
> ../xml/expat/lib/libexpat.la
>    testdate: $(testdate_OBJECTS) $(testdate_LDADD)
>   -   $(LINK) $(testdate_OBJECTS) $(testdate_LDADD) $(LIBS)
>   +   $(LINK) $(testdate_OBJECTS) $(testdate_LDADD) $(ALL_LIBS)
>    
>    testxml_OBJECTS = testxml.lo
>    testxml_LDADD =  ../libaprutil.la @APR_SOURCE_DIR@/libapr.la 
> ../xml/expat/lib/libexpat.la
>    testxml: $(testxml_OBJECTS) $(testxml_LDADD)
>   -   $(LINK) $(testxml_OBJECTS) $(testxml_LDADD) $(LIBS)
>   +   $(LINK) $(testxml_OBJECTS) $(testxml_LDADD) $(ALL_LIBS)
>    
>    testmd4_OBJECTS = testmd4.lo
>   -testmd4_LDADD =  ../libaprutil.la @APR_SOURCE_DIR@/libapr.la
>   +testmd4_LDADD =  ../libaprutil.la @APR_SOURCE_DIR@/libapr.la 
> ../xml/expat/lib/libexpat.la
>    testmd4: $(testmd4_OBJECTS) $(testmd4_LDADD)
>   -   $(LINK) $(testmd4_OBJECTS) $(testmd4_LDADD) $(LIBS)
>   +   $(LINK) $(testmd4_OBJECTS) $(testmd4_LDADD) $(ALL_LIBS)
>   
>   
>   

-- 
Greg Stein, http://www.lyra.org/

Reply via email to