Source: libvidstab
Version: 1.1.0-1
Severity: serious
Justification: baseline violation on i386
Tags: patch

libvidstab fails to cross build from source when built on amd64, because
it passes -msse2 and e.g. an armel cross compiler does not like that
flag. Looking at /proc/cpuinfo is not a good way to check whether we can
use -msse2. In practise, this will also cause passing -msse2 for i386
which is considered a baseline violation and thus an rc bug. The
attached patch fixes that by making the choice explicit. Please consider
applying it.

Helmut
diff --minimal -Nru libvidstab-1.1.0/debian/changelog 
libvidstab-1.1.0/debian/changelog
--- libvidstab-1.1.0/debian/changelog   2018-08-01 08:39:07.000000000 +0200
+++ libvidstab-1.1.0/debian/changelog   2018-10-02 19:01:09.000000000 +0200
@@ -1,3 +1,10 @@
+libvidstab (1.1.0-1.1) UNRELEASED; urgency=medium
+
+  * Non-maintainer upload.
+  * Explicitly choose where to use -msse2. (Closes: #-1)
+
+ -- Helmut Grohne <hel...@subdivi.de>  Tue, 02 Oct 2018 19:01:09 +0200
+
 libvidstab (1.1.0-1) unstable; urgency=low
 
   * Upstream release 1.1.0 (Closes: #709193)
diff --minimal -Nru libvidstab-1.1.0/debian/rules libvidstab-1.1.0/debian/rules
--- libvidstab-1.1.0/debian/rules       2018-08-01 08:39:07.000000000 +0200
+++ libvidstab-1.1.0/debian/rules       2018-10-02 19:01:07.000000000 +0200
@@ -1,3 +1,13 @@
 #!/usr/bin/make -f
+include /usr/share/dpkg/architecture.mk
+ifeq ($(DEB_HOST_ARCH_CPU),amd64)
+HAVE_SSE2=ON
+else
+HAVE_SSE2=OFF
+endif
+
 %:
        dh $@
+
+override_dh_auto_configure:
+       dh_auto_configure -- -DSSE2_FOUND=$(HAVE_SSE2)

Reply via email to