Mailing-List: contact dev-help@couchdb.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@couchdb.apache.org
Received-SPF: pass (athena.apache.org: domain of bchesneau@gmail.com
 designates 209.85.214.180 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :content-type;
        b=sVTnEWqH50dXaf7dSeUVFFKTk4G2Eim2hnHNFu/vW+S+qSkRy0NYYGw5G96l8BECEu
         2R6oWaTXJ2vdNhmJyHtgdCi+zgWNeEDQ9Zd7uGl+3rI6rDPa+1GuzzBSlzmO2VcZVAMt
         La9yk8yvmIi7azh8HcTOiyKatrlauR3u8ihXw=
MIME-Version: 1.0
In-Reply-To: <AANLkTimuiLPEVYrbvOWjG_oMXbcn+EZARx7QdWP_gq-2@mail.gmail.com>
References: <AANLkTimuiLPEVYrbvOWjG_oMXbcn+EZARx7QdWP_gq-2@mail.gmail.com>
Date: Wed, 2 Feb 2011 12:59:08 +0100
Message-ID: <AANLkTikbPAwp_2vvKBgT3NjuJYnqgb6v8LXoqVDSS+p3@mail.gmail.com>
Subject: Re: Compile and Link Flags
From: Benoit Chesneau <bchesneau@gmail.com>
To: dev@couchdb.apache.org
Content-Type: text/plain; charset=ISO-8859-1

On Wed, Feb 2, 2011 at 5:36 AM, Randall Leeds <randall.leeds@gmail.com> wrote:
> Hey devs,
>
> After mucking around in the build system the past few weeks my
> compulsions got the best of me and I did things.
> I think it's an improvement. I apologize for digging this up. It's not
> glamorous.
>
> I put a branch up on github based on current trunk[1].
>
> I started by trying to make the build system find libmozjs and jsapi.h
> from a xulrunner development installation's pkg-config files.
> I also wanted to make libcurl a soft dependency. Paul committed this,
> but it's got an error fixed on this branch[2].
>
> After staring at configure.ac this long, I got twitchy.
> In particular, the following sorts of things really irked me:
>
> 1) "# manually linking libm is requred for FreeBSD 7.0"
>
> I think this was needed by the icu driver, which is why icu-config
> --ldflags includes -lm on my system.
> But we weren't using icu-config --ldflags, we were just using the
> search paths. Why? ICU wants other stuff for me.
> In particular, AC_CHECK_ICU sets ICU_CFLAGS and ICU_LIBS for us but we
> were ignoring them. That's craziness.
>
> 2) CFLAGS="-O2"
> But only on the Windows build?
> There's one place where we differentiate between windows/nix in
> configure.ac but each case contained mostly the same stuff!
> Only a -D CFLAG for SpiderMonkey needs to be different.
> This CFLAG was also dropped into the generic FLAGS variable, along
> with ERLANG_FLAGS, which gets duplicated in CPPFLAGS and LDFLAGS!
> LTFLAGS was set to CFLAGS, but only on windows. I believe this is
> default for libtool anyway.
>
> Anyway, once I broke all this stuff down more explicitly and passed
> only what was needed everywhere and let ICU tell us how it should be
> linked I found that my ICU install says it should be compiled with
> -ansi, so I fixed the comments in the icu driver to be C style (/*
> */).
>
> Everything passes here and builds nicely.
>
> Combining it all we get:
> A configure.ac that is easier on the eyes.
> Less platform specific stuff strew about the build system.
> No need for --with-js-* on systems with a system-wide xulrunner.
> A (correct this time) soft dependency on libcurl.
>
> So, is this stuff that should go in trunk? Or was this all for naught?
> Again, sorry for the non-functional changes. I promise I'll submit
> patches that actually improve functionality soon.
>
> [1] https://github.com/tilgovi/couchdb/tree/flagmadness
> [2] https://github.com/tilgovi/couchdb/commit/0185faa2150f072a8c71b03325424c20e24c3eab
>

Looks good for me. Will try this branch in afternoon on my bsd systems.

- benoit