[Debian-med-packaging] Bug#768769: plastimatch: FTBFS in jessie: Tests failures

[Colin Watson]

On Tue, Nov 11, 2014 at 07:20:02AM +0100, Andreas Tille wrote:
> On Mon, Nov 10, 2014 at 12:01:48PM -0800, Gregory Sharp wrote:
> > This is a bug in upstream, where the regression test
> > is missing a dependency.
> 
> I'm not sure what this really means but if it is a missing dependency do
> you want to express that we can fix this by just another Build-Depends
> and if yes which one?  Are you able to commit a patch to SVN?

I think he means cmake dependencies for the tests in question; they're
failing in parallel builds.  Here's a patch that fixes things for me:

  * Add rect-10-roi-1 and rect-11-roi dependencies to plm-union test; add
    plm-convert-dicom-a dependency to plm-convert-cxt test (closes:
    #768769).

diff -Nru plastimatch-1.5.16+dfsg/debian/patches/series plastimatch-1.5.16+dfsg/debian/patches/series
--- plastimatch-1.5.16+dfsg/debian/patches/series	1970-01-01 01:00:00.000000000 +0100
+++ plastimatch-1.5.16+dfsg/debian/patches/series	2014-11-11 16:17:06.000000000 +0000
@@ -0,0 +1 @@
+test-depends.patch
diff -Nru plastimatch-1.5.16+dfsg/debian/patches/test-depends.patch plastimatch-1.5.16+dfsg/debian/patches/test-depends.patch
--- plastimatch-1.5.16+dfsg/debian/patches/test-depends.patch	1970-01-01 01:00:00.000000000 +0100
+++ plastimatch-1.5.16+dfsg/debian/patches/test-depends.patch	2014-11-11 16:18:50.000000000 +0000
@@ -0,0 +1,29 @@
+Description: Fix test dependencies to support parallel testing
+Author: Colin Watson <cjwatson at ubuntu.com>
+Bug-Debian: https://bugs.debian.org/768769
+Forwarded: no
+Last-Update: 2014-11-11
+
+Index: b/Testing/CMakeLists.txt
+===================================================================
+--- a/Testing/CMakeLists.txt
++++ b/Testing/CMakeLists.txt
+@@ -1775,7 +1775,8 @@
+   "1648003"
+   "1648003"
+   )
+-set_tests_properties (plm-convert-cxt PROPERTIES DEPENDS plm-convert-dicom-c)
++set_tests_properties (plm-convert-cxt PROPERTIES
++  DEPENDS "plm-convert-dicom-a;plm-convert-dicom-c")
+ set_tests_properties (plm-convert-cxt-stats PROPERTIES DEPENDS plm-convert-cxt)
+ set_tests_properties (plm-convert-cxt-check PROPERTIES 
+   DEPENDS plm-convert-cxt-stats)
+@@ -3290,7 +3291,7 @@
+   "1271"
+   )
+ set_tests_properties (plm-union 
+-  PROPERTIES DEPENDS "rect-10-mask-1;rect-11-mask")
++  PROPERTIES DEPENDS "rect-10-mask-1;rect-10-roi-1;rect-11-mask;rect-11-roi")
+ set_tests_properties (plm-union-stats PROPERTIES DEPENDS plm-union)
+ set_tests_properties (plm-union-check PROPERTIES DEPENDS plm-union-stats)
+ 

Thanks,

-- 
Colin Watson                                       [cjwatson at debian.org]