On Wed, Jan 20, 2010 at 10:53:06AM +0100, Fabian Greffrath wrote: >> The patch was against configure.ac. I don't know if the minimal >> debian/rules is sufficient to re-generate configure after applying this >> patch. >> If need be, add a line like "autoreconf -f" somewhere before calling >> configure. > Personally, instead of running autoreconf (or similar) during build, I'd > prefer a patch that contains all the changes to configure caused by the > changes to configure.ac. This way the package stays idempotent, i.e. it
I've also considered this. Find patch attached. Note that both versions of the patch can be dropped after the next upstream release. Cheerio -- mail: a...@thur.de http://adi.thur.de PGP/GPG: key via keyserver
--- configure 2010-01-16 20:38:40.000000000 +0100 +++ configure.new 2010-01-20 11:11:09.000000000 +0100 @@ -4321,6 +4321,8 @@ # Check for SSE optimization. if test "x$ac_sse" = "xyes"; then ac_sse_cflags="-msse -mfpmath=sse -ffast-math" + CFLAGS_OLD="$CFLAGS" + CPPFLAGS_OLD="$CPPFLAGS" CFLAGS="$ac_sse_cflags $CFLAGS" CPPFLAGS="$ac_sse_cflags $CPPFLAGS" { $as_echo "$as_me:$LINENO: checking for SSE optimization" >&5 @@ -4400,6 +4402,9 @@ ac_sse=$ac_cv_sse if test "x$ac_sse" = "xyes"; then ac_cflags="$ac_sse_cflags $ac_cflags" + else + CFLAGS="$CFLAGS_OLD" + CPPFLAGS="$CPPFLAGS_OLD" fi fi