apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Philip M. Gollucci" <pgollu...@p6m7g8.com>
Subject Re: [PATCH]: FreeBSD ports devel/apr
Date Wed, 09 Apr 2008 20:23:47 GMT
Joe Orton wrote:
 > On Tue, Apr 08, 2008 at 01:51:19PM -0400, Philip M. Gollucci wrote:
 >> I'm sending this patch upstream, apr is one of the projects I don't 
have a commit bit too, so please review commit, and MFC as needed.
 >
 > That code should be using LDFLAGS rather than LIBS, and it should be 
adding to, rather than overwriting, both the variables.  Can you try 
this instead?

via visual inspection, your patch looks much more correct, but I don't 
expect any differences, as dbm related things already compile/link 
correctly, so the result will be the same for FreeBSD's devel/apr; however,

I don't happen to be setup to do manual building atm...; even if I was, 
I'm not sure what I'd have to try without the patch to break it then 
apply and verify it fixes it.

It looks like back in 2006/04/06
adding
+APR_UTIL_CONF_ARGS+=   --with-gdbm=${LOCALBASE}
aka ./configure for apr-util
blew away the CFLAGS and was affecting subsequent linking aka berkely db.

As seen in the below log snippet, the default freebsd compile still 
works with this patch applied.

HTH

building apr-1.2.12 in directory /usr/home/space/8.0-FreeBSD
Looking for apr source in /work/a/ports/devel/apr/work/apr-1.2.12
Creating include/private/apu_config.h ...
Creating configure ...
Generating 'make' outputs ...
rebuilding rpm spec file
checking for ldap support...
checking for default DBM... sdbm (default)
checking libpq-fe.h usability... no
checking libpq-fe.h presence... no
checking for libpq-fe.h... no
checking sqlite3.h usability... no
checking sqlite3.h presence... no
checking for sqlite3.h... no
checking sqlite.h usability... no
checking sqlite.h presence... no
checking for sqlite.h... no
   setting LDFLAGS to "-L/usr/local/lib"
   adding "-I/usr/local/include" to CPPFLAGS
   setting APRUTIL_INCLUDES to "-I/usr/local/include"
   setting APRUTIL_LDFLAGS to "-L/usr/local/lib"
------------------------------------------------------------------------
Philip M. Gollucci (philip@ridecharge.com)
o:703.549.2050x206
Senior System Admin - Riderway, Inc.
http://riderway.com / http://ridecharge.com
1024D/EC88A0BF 0DE5 C55C 6BF3 B235 2DAB  B89E 1324 9B4F EC88 A0BF

Work like you don't need the money,
love like you'll never get hurt,
and dance like nobody's watching.


Mime
View raw message