Hi,

I spotted this bug and thought I might be able to help.  Attached is a
patch that adds execution of the CFFI test suite to the package build.
You may notice I have also separated out building of the test C
libraries, they seem to be the main pain point during a failure.
Unfortunately the tests swallow the GCC and Make stdio so without
building them as a separate step it is difficult to debug.

I have tested the build on i386 and amd64.  The C libraries that built
during testing seem to target both 64 & 32 bit architectures, so
gcc-multilib is required for amd64 but I am not entirely sure about
other 64 bit architectures.  If you suspect this may cause a problem, I
can setup qemubuilder and test the builds on some of the other debian
architectures.

Thanks,
Russell

commit 5ec485dd3659a396558773990ceae7d2f06e21e2
Author: Russell Sim <russell....@gmail.com>
Date:   Fri Mar 30 17:57:00 2012 +1100

    added initial testing code to rules

diff --git a/debian/control b/debian/control
index f4c2ae9..0f1b79b 100644
--- a/debian/control
+++ b/debian/control
@@ -2,7 +2,7 @@ Source: cffi
 Section: lisp
 Priority: optional
 Build-Depends: debhelper (>= 7)
-Build-Depends-Indep: dh-lisp, texinfo, a2ps, sbcl, texlive-extra-utils, texlive, texlive-generic-recommended
+Build-Depends-Indep: dh-lisp, texinfo, a2ps, sbcl, texlive-extra-utils, texlive, texlive-generic-recommended, cl-rt, cl-alexandria, cl-trivial-features, cl-babel, gcc-multilib [amd64]
 Maintainer: Debian Common Lisp Team <pkg-common-lisp-de...@lists.alioth.debian.org>
 Uploaders: Peter Van Eynde <pvane...@debian.org>
 Standards-Version: 3.8.3
diff --git a/debian/rules b/debian/rules
index 94b5229..d1b6f2d 100755
--- a/debian/rules
+++ b/debian/rules
@@ -36,6 +36,10 @@ binary-indep: build
 	dh_installchangelogs 
 	dh_compress
 	dh_fixperms
+	cd tests
+	make
+	cd ..
+	sbcl --disable-debugger --no-userinit --eval "(require 'asdf)" --eval "(asdf:oos 'asdf:test-op :cffi-tests)" --eval '(quit)'
 	dh_installdeb
 	dh_gencontrol
 	dh_md5sums

Reply via email to