Charles Plessy <[EMAIL PROTECTED]> writes: > Would the following patch make sense? > > diff --git a/configure.ac b/configure.ac > index ad2f1e6..4f2993a 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -23,10 +23,7 @@ case "${host_cpu}-${host_os}" in > AC_COMPILE_IFELSE([AC_LANG_PROGRAM], [ext_CFLAGS="-m64"], > []);; > esac;; > *) > - AC_MSG_CHECKING([if gcc accepts -m64]) > - CFLAGS="-m64" > - AC_COMPILE_IFELSE([AC_LANG_PROGRAM], [ext_CFLAGS="-m64"; > AC_MSG_RESULT([yes])], > - > [ext_CFLAGS="-D_FILE_OFFSET_BITS=64"; AC_MSG_RESULT([no])]);; > + ext_CFLAGS="-D_FILE_OFFSET_BITS=64";; > esac > AC_ARG_ENABLE(experimental, [ --enable-experimental enable experimental > features], > [ext_CFLAGS="${ext_CFLAGS} > -DMAQ_SHOW_EXPERIMENTAL"], [])
The above change is probably fine, but the ideal solution would be for upstream to stop trying to code this directly and instead use the Autoconf macro provided for this purpose. AC_SYS_LARGEFILE will add to CFLAGS whatever flags are needed to enable large files. (Unfortunately, since upstream is using ext_CFLAGS instead of CFLAGS, you may not be able to just delete upstream's Autoconf code and add that macro.) -- Russ Allbery ([EMAIL PROTECTED]) <http://www.eyrie.org/~eagle/> -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]