Skip to contentSkip to conversation

package/pixman: disable tests

PATCH New Yann E. MORIN 2 participants 2 messages Dec 5, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/pixman: disable tests

Tests are broken on some architectures because of an incomplete fenv.

Since the tests are not really useful in the Buildroot environment, just disable them.

Fixes a lot of build failures:

    http://autobuild.buildroot.org/results/cd7/cd722620832c28d22eacfc43336f6952b82305ff/
    http://autobuild.buildroot.org/results/3f4/3f4f1e88988d66b052432885b4f044a0eddcee2c/
    ...

Pixman was already autoreconfed (but that was not needed). Since we now really need to autoreconf it, add a comment stating why we do.

---
 package/pixman/0001-Disable-tests.patch | 29 +++++++++++++++++++++++++++++
 package/pixman/pixman.mk                |  2 ++
 2 files changed, 31 insertions(+)
 create mode 100644 package/pixman/0001-Disable-tests.patch
package/pixman/0001-Disable-tests.patch new file +29 -0
new file mode 100644
index 0000000..dfadd69
@@ -0,0 +1,29 @@
+From 9b8132738c364fc3c886e81e7d383aaff80dc867 Mon Sep 17 00:00:00 2001
+From: "Yann E. MORIN" <[email protected]>
+Date: Sat, 5 Dec 2015 12:00:53 +0100
+Subject: [PATCH] Disable tests
+
+Tests are causing build failures on some architectures that are missing
+a proper fenv.h, so just disable them.
+
+Signed-off-by: "Yann E. MORIN" <[email protected]>
+
+---
+Status: Buildroot specific, not suitable for upstream in this state.
+---
+ Makefile.am | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/Makefile.am b/Makefile.am
+index 5137c9e..eae79fd 100644
+--- a/Makefile.am
++++ b/Makefile.am
+@@ -1,4 +1,4 @@
+-SUBDIRS = pixman demos test
++SUBDIRS = pixman demos
+ 
+ pkgconfigdir=$(libdir)/pkgconfig
+ pkgconfig_DATA=pixman-1.pc
+-- 
+1.9.1
+
package/pixman/pixman.mk +2 -0
index cbf57fe..8281773 100644
@@ -12,6 +12,8 @@ PIXMAN_LICENSE_FILES = COPYING
 
 PIXMAN_INSTALL_STAGING = YES
 PIXMAN_DEPENDENCIES = host-pkgconf
+
+# For 0001-Disable-tests.patch
 PIXMAN_AUTORECONF = YES
 
 # don't build gtk based demos
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/1 package/pixman: disable tests

Quoted 11 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Yann" == Yann E MORIN <[email protected]> writes:

Tests are broken on some architectures because of an incomplete fenv. Since the tests are not really useful in the Buildroot environment, just disable them.

Fixes a lot of build failures:

    http://autobuild.buildroot.org/results/cd7/cd722620832c28d22eacfc43336f6952b82305ff/
    http://autobuild.buildroot.org/results/3f4/3f4f1e88988d66b052432885b4f044a0eddcee2c/
    ...
Quoted 5 lines

Pixman was already autoreconfed (but that was not needed). Since we now really need to autoreconf it, add a comment stating why we do.

Committed, thanks.

Quoted 2 lines

+---
+Status: Buildroot specific, not suitable for upstream in this state.

Indeed. It would be good to work on an upstreamable solution (E.G. probably a --disable-tests option) and send it upstream.

Signature · 1 line
Bye, Peter Korsgaard