Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Received-SPF: pass (athena.apache.org: local policy)
Message-ID: <495F4229.9060700@kippdata.de>
Date: Sat, 03 Jan 2009 11:47:05 +0100
From: Rainer Jung <rainer.jung@kippdata.de>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; de;
 rv:1.9.1b3pre) Gecko/20081204 Thunderbird/3.0b1
MIME-Version: 1.0
To: dev@httpd.apache.org
Subject: Re: svn commit: r730882 - /httpd/httpd/trunk/configure.in
References: <5c902b9e0901022352r5f1d6668xce6e27751428fe97@mail.gmail.com>
In-Reply-To: <5c902b9e0901022352r5f1d6668xce6e27751428fe97@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-15; format=flowed
Content-Transfer-Encoding: 7bit

On 03.01.2009 08:52, Justin Erenkrantz wrote:
> [ It's odd as I didn't get the email for this commit...anyway... ]
>
>> Author: rjung
>> Date: Fri Jan  2 17:01:56 2009
>> New Revision: 730882
>>
>> URL: http://svn.apache.org/viewvc?rev=730882&view=rev
>> Log:
>> Only link libhttpd against pcre.
>>
>> Modified:
>>     httpd/httpd/trunk/configure.in
>>
>> Modified: httpd/httpd/trunk/configure.in
>> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/configure.in?rev=730882&r1=730881&r2=730882&view=diff
>> ==============================================================================
>> --- httpd/httpd/trunk/configure.in (original)
>> +++ httpd/httpd/trunk/configure.in Fri Jan  2 17:01:56 2009
>> @@ -182,7 +182,8 @@
>>    fi
>>    AC_MSG_NOTICE([Using external PCRE library from $PCRE_CONFIG])
>>    APR_ADDTO(CFLAGS, [`$PCRE_CONFIG --cflags`])
>> -  APR_ADDTO(LIBS, [`$PCRE_CONFIG --libs`])
>> +  APR_ADDTO(LTLIBRARY_LIBADD, [`$PCRE_CONFIG --libs`])
>> +  APACHE_SUBST(LTLIBRARY_LIBADD)
>> else
>>    AC_MSG_ERROR([pcre-config for libpcre not found. PCRE is required and available from http://pcre.org/])
>> fi
>
> This change busts my build because it tries to link a dynamic library
> (pcre) against a static library (libmain).  You can't do that.
> (jlibtool barfs on this.)  It is only legal to link static libraries
> against other static libraries.  Please revert.  -- justin


There was no problem with usual libtool. I think it simply records the 
dependency in the *.la file (but didn't check that).

Does the following change fix it for MacOS?

Index: configure.in
===================================================================
--- configure.in        (revision 730882)
+++ configure.in        (working copy)
@@ -182,8 +182,7 @@
    fi
    AC_MSG_NOTICE([Using external PCRE library from $PCRE_CONFIG])
    APR_ADDTO(CFLAGS, [`$PCRE_CONFIG --cflags`])
-  APR_ADDTO(LTLIBRARY_LIBADD, [`$PCRE_CONFIG --libs`])
-  APACHE_SUBST(LTLIBRARY_LIBADD)
+  APR_ADDTO(HTTPD_LDFLAGS, [`$PCRE_CONFIG --libs`])
  else
    AC_MSG_ERROR([pcre-config for libpcre not found. PCRE is required 
and available from http://pcre.org/])
  fi

If so, that's better for several reasons.

Regards,

Rainer