Bug#961581: pbcopper: ftbfs on ppc64el

2020-05-27 Thread Andreas Tille
On Wed, May 27, 2020 at 09:53:44AM +1200, Michael Hudson-Doyle wrote:
> 
> Thanks. I think a fixed simde has been uploaded now, so my patch might not
> be needed at all...

OK, while pbcopper is doing another cycle in new queue I commited

pbcopper(master) $ git diff
diff --git a/debian/changelog b/debian/changelog
index f7f77e3..16462c8 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+pbcopper (1.6.0+dfsg-3) UNRELEASED; urgency=medium
+
+  * ppc64el patch is not needed for libsimde-dev >= 0.0.0.git.20200526
+
+ -- Andreas Tille   Wed, 27 May 2020 08:21:28 +0200
+
 pbcopper (1.6.0+dfsg-2) unstable; urgency=medium
 
   * Use d-shlibs to make sure SOVERSION is correct
diff --git a/debian/control b/debian/control
index 9bc83ec..fbf1215 100644
--- a/debian/control
+++ b/debian/control
@@ -11,7 +11,7 @@ Build-Depends: debhelper-compat (= 12),
libboost-dev,
googletest,
libgtest-dev,
-   libsimde-dev
+   libsimde-dev (>= 0.0.0.git.20200526)
 Standards-Version: 4.5.0
 Vcs-Browser: https://salsa.debian.org/med-team/pbcopper
 Vcs-Git: https://salsa.debian.org/med-team/pbcopper.git
diff --git a/debian/patches/series b/debian/patches/series
index 69204c0..88405bb 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -2,5 +2,5 @@
 skip_test_relying_on_clock.patch
 meson.patch
 simde.patch
-simde-ppc64el.patch
+#simde-ppc64el.patch
 static+shared_lib.patch
diff --git a/debian/patches/simde-ppc64el.patch 
b/debian/patches/simde-ppc64el.patch
index 19e4d63..4b31e19 100644
--- a/debian/patches/simde-ppc64el.patch
+++ b/debian/patches/simde-ppc64el.patch
@@ -2,6 +2,7 @@ From: Michael Hudson-Doyle 
 Date: Tue, 26 May 2020 22:00:48 +1200
 Bug-Debian: https://bugs.debian.org/961581
 Description: Fix simde build for ppc64el
+  Not needed for libsimde-dev >= 0.0.0.git.20200526-1
 
 --- a/src/align/cssw/ssw.c
 +++ b/src/align/cssw/ssw.c



Thanks for the hint

  Andreas.

-- 
http://fam-tille.de



Bug#961581: pbcopper: ftbfs on ppc64el

2020-05-26 Thread Michael Hudson-Doyle
On Tue, 26 May 2020 at 23:43, Andreas Tille  wrote:

> thanks for the patch.  I've uploaded pbcopper to new since
> it needed a soname bump and thus a new binary package name.
>

Thanks. I think a fixed simde has been uploaded now, so my patch might not
be needed at all...


Bug#961581: pbcopper: ftbfs on ppc64el

2020-05-26 Thread Andreas Tille
Control: tags -1 pending

Hi Michael,

thanks for the patch.  I've uploaded pbcopper to new since
it needed a soname bump and thus a new binary package name.

Kind regards

  Andreas.

-- 
http://fam-tille.de



Bug#961581: pbcopper: ftbfs on ppc64el

2020-05-26 Thread Michael Hudson-Doyle
Source: pbcopper
Version: 1.6.0+dfsg-1
Severity: serious
Tags: patch ftbfs
Justification: fails to build from source (but built successfully in the past)

Dear Maintainer,

As can be seen on 

https://buildd.debian.org/status/logs.php?pkg=pbcopper=ppc64el

pbcopper currently ftbfs on pcp64el. I think this is a bug in simde:

https://github.com/nemequ/simde/issues/325

but anyway, I'm attaching the workaround I just uploaded to Ubuntu.

Cheers,
mwh


-- System Information:
Debian Release: bullseye/sid
  APT prefers focal-updates
  APT policy: (500, 'focal-updates'), (500, 'focal-security'), (500, 'focal'), 
(400, 'focal-proposed'), (100, 'focal-backports')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 5.4.0-31-generic (SMP w/8 CPU cores)
Kernel taint flags: TAINT_PROPRIETARY_MODULE, TAINT_OOT_MODULE
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled
--- a/src/align/cssw/ssw.c
+++ b/src/align/cssw/ssw.c
@@ -35,6 +35,9 @@
  *
  */
 
+#ifdef __powerpc64__
+#define SIMDE_NO_NATIVE
+#endif
 #define SIMDE_ENABLE_NATIVE_ALIASES
 #include "simde/x86/sse2.h"
 #include 
--- a/src/align/cssw/ssw.h
+++ b/src/align/cssw/ssw.h
@@ -11,6 +11,9 @@
 #ifndef SSW_H
 #define SSW_H
 
+#ifdef __powerpc64__
+#define SIMDE_NO_NATIVE
+#endif
 #define SIMDE_ENABLE_NATIVE_ALIASES
 #include "simde/x86/sse2.h"
 #include