[Flac-dev] patches for flac build

Josh Coalson xflac at yahoo.com
Wed Jul 18 17:22:54 PDT 2001


> Speaking of CFLAGS, it looks like configure.in clobbers any existing
> CFLAGS in
> the environment.  This can be a pain when trying to compile with a
> modified
> CFLAGS, as the usual idiom is:
> 
> CFLAGS=... ./configure ...
> 
> thus propagating CFLAGS to all generated makefiles.  The right thing
> to do is
> probably to prepend to CFLAGS instead of replacing it, so the user's
> settings
> will come later on the command line and override the defaults.

OK, I checked in a fix... does this snippet look alright?

SAVE_CFLAGS="$CFLAGS"
CFLAGS='-I./include -I $(top_srcdir)/include -Wall -W'
if test x$debug = xtrue; then
    CFLAGS="$CFLAGS -g -O0 -DDEBUG"
else
    CFLAGS="$CFLAGS -O3 -DNDEBUG"
    if test x$GCC = xyes; then
        CFLAGS="$CFLAGS -fomit-frame-pointer -funroll-loops \
        -finline-functions -Winline -DFLAC__INLINE=__inline__"
    fi
fi
CFLAGS="$CFLAGS $SAVE_CFLAGS"


__________________________________________________
Do You Yahoo!?
Get personalized email addresses from Yahoo! Mail
http://personal.mail.yahoo.com/




More information about the Flac-dev mailing list