Hi Andreas,

On 08/14/2014 02:24 PM, Andreas Metzler wrote:
> this patch looks strange. Can you show how to reproduce this issue?

I wrote:
> I'll try to reproduce/understand it again and ping back.

Got it; it was due to building the package as a privileged user.
(thus passing setup.py's is_system_wide_install() test, due to
write-access on /usr/lib/python*/dist-packages)

The package builds fine on a normal/unprivileged sbuild build,
just with the test-suite patch.

I see you already made a uploaded w/ that patch for delayed/10.
Thanks, that will make it.

In case you'd prefer to put in a patch w/ DEP-3 headers and stuff,
I'm attaching one for your convenience.

Thanks again, and best regards.

--
Mauricio Faria de Oliveira
IBM Linux Technology Center
diff -Nru libprelude-1.0.0/debian/changelog libprelude-1.0.0/debian/changelog
--- libprelude-1.0.0/debian/changelog   2014-08-05 17:29:12.000000000 +0000
+++ libprelude-1.0.0/debian/changelog   2014-08-15 16:29:33.000000000 +0000
@@ -1,3 +1,9 @@
+libprelude (1.0.0-11.3ppc64el1) UNRELEASED; urgency=medium
+
+  * Fix libmissing/ test-suite on ppc64el (ppc64el-fix-gnulib-tests.patch) 
(Closes: #751822)
+
+ -- Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>  Thu, 14 Aug 2014 
18:06:04 -0300
+
 libprelude (1.0.0-11.3) unstable; urgency=low
 
   * Non-maintainer upload.
diff -Nru libprelude-1.0.0/debian/patches/ppc64el-fix-gnulib-tests.patch 
libprelude-1.0.0/debian/patches/ppc64el-fix-gnulib-tests.patch
--- libprelude-1.0.0/debian/patches/ppc64el-fix-gnulib-tests.patch      
1970-01-01 00:00:00.000000000 +0000
+++ libprelude-1.0.0/debian/patches/ppc64el-fix-gnulib-tests.patch      
2014-08-15 16:28:09.000000000 +0000
@@ -0,0 +1,55 @@
+Subject: [PATCH] isfinite, isinf, isnan tests: fix for little-endian PowerPC
+Origin: backport, 
http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=commit;h=e962c07c0c8bb18f9a5969cbf1ddb01d0fe4f55c
+Forwarded: https://www.prelude-ids.org/issues/556
+Author: Mauricio Faria de Oliveira <mauri...@linux.vnet.ibm.com>
+Last-Update: 2014-08-14
+
+This patch is a trivial backport (subset) of the upstream patch; it just 
removes
+the files that don't exist in libprelude (not required).
+
+ curl 
'http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=patch;h=e962c07c0c8bb18f9a5969cbf1ddb01d0fe4f55c'
 \
+  | filterdiff \
+      --strip=1 \
+      --addoldprefix='a/libmissing/' \
+      --addnewprefix='b/libmissing/' \
+      -i '*/tests/test-isnanl.h' \
+      -i '*/tests/test-signbit.c' \
+  > ppc64el-fix-gnulib-tests.patch
+
+--- a/libmissing/tests/test-isnanl.h
++++ b/libmissing/tests/test-isnanl.h
+@@ -51,6 +51,15 @@ main ()
+   /* A bit pattern that is different from a Quiet NaN.  With a bit of luck,
+      it's a Signalling NaN.  */
+   {
++#if defined __powerpc__ && LDBL_MANT_DIG == 106
++    /* This is PowerPC "double double", a pair of two doubles.  Inf and Nan 
are
++       represented as the corresponding 64-bit IEEE values in the first 
double;
++       the second is ignored.  Manipulate only the first double.  */
++    #undef NWORDS
++    #define NWORDS \
++      ((sizeof (double) + sizeof (unsigned int) - 1) / sizeof (unsigned int))
++#endif
++
+     memory_long_double m;
+     m.value = NaNl ();
+ # if LDBL_EXPBIT0_BIT > 0
+--- a/libmissing/tests/test-signbit.c
++++ b/libmissing/tests/test-signbit.c
+@@ -151,6 +151,16 @@ test_signbitl ()
+     #define NWORDS \
+       ((sizeof (long double) + sizeof (unsigned int) - 1) / sizeof (unsigned 
int))
+     typedef union { long double value; unsigned int word[NWORDS]; } 
memory_long_double;
++
++#if defined __powerpc__ && LDBL_MANT_DIG == 106
++    /* This is PowerPC "double double", a pair of two doubles.  Inf and Nan 
are
++       represented as the corresponding 64-bit IEEE values in the first 
double;
++       the second is ignored.  Manipulate only the first double.  */
++    #undef NWORDS
++    #define NWORDS \
++      ((sizeof (double) + sizeof (unsigned int) - 1) / sizeof (unsigned int))
++#endif
++
+     memory_long_double m;
+     m.value = zerol / zerol;
+ # if LDBL_EXPBIT0_BIT > 0
diff -Nru libprelude-1.0.0/debian/patches/series 
libprelude-1.0.0/debian/patches/series
--- libprelude-1.0.0/debian/patches/series      2014-05-25 13:58:53.000000000 
+0000
+++ libprelude-1.0.0/debian/patches/series      2014-08-15 16:28:09.000000000 
+0000
@@ -5,3 +5,4 @@
 request_REUSEADDR_before_bind.patch
 test-poll_add_sleep_after_write.patch
 keep-atfork-symbol.patch
+ppc64el-fix-gnulib-tests.patch

Reply via email to