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 <cjwat...@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 [cjwat...@debian.org]
--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
