Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Received-SPF: pass (nike.apache.org: domain of rainer.jung@kippdata.de
 designates 195.227.30.149 as permitted sender)
Message-ID: <541C7249.7020406@kippdata.de>
Date: Fri, 19 Sep 2014 20:13:29 +0200
From: Rainer Jung <rainer.jung@kippdata.de>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64;
 rv:31.0) Gecko/20100101 Thunderbird/31.1.1
MIME-Version: 1.0
To: APR Developer List <dev@apr.apache.org>
Subject: Re: [VOTE] Release APR-util 1.5.4
References: 
 <20140919093715.ec908e91c20de17e6e448089a4bc3ed2.c10d75d66c.mailapi@email11.secureserver.net>
In-Reply-To: 
 <20140919093715.ec908e91c20de17e6e448089a4bc3ed2.c10d75d66c.mailapi@email11.secureserver.net>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit

Am 19.09.2014 um 18:37 schrieb wrowe@rowe-clan.net:
> Aren't most of these issues resolved by correctly using pkg-config where
> it is available?

pkg-config doesn't help for

- crypto configure for OpenSSL expected to fail at least on Solaris,
    because when linking against the libssl we need
    the additional flags "-ldl -lsocket -lnsl". Currently
    there's no way to fix this apart from hacking configure.
    For Linux I'm not sure, but likely you'll need "-ldl".

- configure fails for Berkeley DB in non-standard path,
    because it doesn't add an rpath to conftest and then tries
    to run the compiled conftest binary.

- no rpath for ldap, mysql and crypto dso extensions. So no
    way to run the result without LD_LIBRARY_PATH.
    Interesting: the m4 file does add an rpath only for oracle.
    Since there's no platform independent way to add rpath,
    we might want to remove it everywhere (and users would need
    to use the respective LDADD variables).

- no rpath for ldap, mysql and crypto dependency libs when building
   without dso support, e.g. when there's only one big apr util library.

Using pkg-config does enter the right rpath to the extension libs for 
expat, sqlite, apr and oracle, so no LDADD needed there.

> In any case, that isn't a showstopper against 1.5.4 IMHO, since these
> don't represent regressions AFAIK.

Agreed, that's why I voted +1.

Rainer