[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Problems building trunk (and probably 1.8.x) on FreeBSD 9.0

From: Daniel Shahaf <danielsh_at_elego.de>
Date: Sat, 11 May 2013 01:30:57 +0300

Ben Reser wrote on Fri, May 10, 2013 at 13:55:16 -0700:
> The problem is that FreeBSD's `apu-1-config --includes` has
> -I/usr/local/include included in the output (which is where
> sqlite3ext.h is installed). I'm not sure what the precise reason for
> this is but I'm assuming there's some legit reason.
>

I said it on IRC but I'll repeat here for posterity:

It seems that another fix would be for the FreeBSD APR port to install
its headers in e.g. /usr/local/include/apr-1/apr-1/apr_pools.h, rather
than /usr/local/include/apr-1/apr_pools.h, so that `apr-1-config --includes`
doesn't need to list -I/usr/local/include, which may pull in other
libraries' headers as well.

-

I'll note that some of infra's primary services, which run on FreeBSD,
run into similar problems with the system libapr and our self-compiled
(not-via-ports) libapr. The solution we use is:

    % sudo gzip /usr/local/lib/libapr*
    % make && sudo make install # uses our self-compiled libapr
    % sudo gunzip /usr/local/lib/libapr*

Not pretty, but works.

> Putting SVN_SQLITE_INCLUDES earlier in the INCLUDES list solves the problem:
> [[[
> Index: Makefile.in
> ===================================================================
> --- Makefile.in (revision 1481158)
> +++ Makefile.in (working copy)
> @@ -121,10 +121,11 @@
> LT_CXX_LIBADD = @LT_CXX_LIBADD@
>
> INCLUDES = -I$(top_srcdir)/subversion/include -I$(top_builddir)/subversion \
> + @SVN_SQLITE_INCLUDES@ \
> @SVN_APR_INCLUDES@ @SVN_APRUTIL_INCLUDES@
> @SVN_APR_MEMCACHE_INCLUDES@ \
> @SVN_DB_INCLUDES@ @SVN_GNOME_KEYRING_INCLUDES@ \
> @SVN_KWALLET_INCLUDES@ @SVN_MAGIC_INCLUDES@ \
> - @SVN_SASL_INCLUDES@ @SVN_SERF_INCLUDES@ @SVN_SQLITE_INCLUDES@ \
> + @SVN_SASL_INCLUDES@ @SVN_SERF_INCLUDES@ \
> @SVN_XML_INCLUDES@ @SVN_ZLIB_INCLUDES@
> ]]]
>
> Does that seem like a reasonable change for us to make?
Received on 2013-05-11 00:31:31 CEST

This is an archived mail posted to the Subversion Dev mailing list.