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
 
_______________________________________________
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers

Reply via email to