[Debian-med-packaging] Bug#893409: pixelmed FTBFS with openjdk-9

[Andreas Tille]

Hi,

On Wed, Dec 19, 2018 at 10:39:55AM +0100, Olivier Sallou wrote:
> > The missing class is in libjaxb-api-java. Just make sure it's on the
> > CLASSPATH.
> 
> yeap, an other issue at java migration to openjdk 10/11
> 
> Adding this lib to deps should fix the pb

Thanks for the quick help.  I tried:


diff --git a/debian/control b/debian/control
index e0548ee..3f8b86c 100644
--- a/debian/control
+++ b/debian/control
@@ -14,7 +14,8 @@ Build-Depends-Indep: default-jdk,
                      libjmdns-java,
                      libvecmath-java,
                      libpixelmed-codec-java,
-                     libjsonp-java
+                     libjsonp-java,
+                     libjaxb-api-java
 Standards-Version: 4.2.1
 Vcs-Browser: https://salsa.debian.org/med-team/pixelmed
 Vcs-Git: https://salsa.debian.org/med-team/pixelmed.git
diff --git a/debian/patches/fixdoc.patch b/debian/patches/fixdoc.patch
index 752b78c..1737f5f 100644
--- a/debian/patches/fixdoc.patch
+++ b/debian/patches/fixdoc.patch
@@ -11,7 +11,7 @@ Index: pixelmed-20140326/Makefile
        rm -rf docs/javadoc
        javadoc \
 -              -classpath .:lib/additional/excalibur-bzip2-1.0.jar:lib/additional/hsqldb.jar:lib/additional/vecmath1.2-1.14.jar:lib/additional/commons-codec-1.3.jar:lib/additional/commons-net2.jar:lib/additional/jmdns.jar:lib/additional/jpedalSTD.jar:lib/junit/junit-4.8.1.jar \
-+              -classpath .:/usr/share/java/excalibur-bzip2-1.0.jar:/usr/share/java/hsqldb.jar:/usr/share/java/vecmath.jar:/usr/share/java/commons-codec.jar:/usr/share/java/commons-net2.jar:/usr/share/java/jmdns.jar:/usr/share/java/jpedalSTD.jar:/usr/share/java/junit4.jar \
++              -classpath .:/usr/share/java/excalibur-bzip2-1.0.jar:/usr/share/java/hsqldb.jar:/usr/share/java/vecmath.jar:/usr/share/java/commons-codec.jar:/usr/share/java/commons-net2.jar:/usr/share/java/jmdns.jar:/usr/share/java/jpedalSTD.jar:/usr/share/java/junit4.jar:/usr/share/java/jaxb-api.jar \
                -link http://download.oracle.com/javase/1.5.0/docs/api/ \
                -link http://jpedal.org/javadoc/ \
                -link http://www.hsqldb.org/doc/src/ \
diff --git a/debian/patches/jaxb-api.patch b/debian/patches/jaxb-api.patch
new file mode 100644
index 0000000..85760dd
--- /dev/null
+++ b/debian/patches/jaxb-api.patch
@@ -0,0 +1,16 @@
+Description: For OpenJDK 11 jaxb needs to be in classpath
+Bug-Debian: https://bugs.debian.org/893409
+Author: Andreas Tille <tille at debian.org>
+Last-Update: Wed, 19 Dec 2018 08:53:43 +0100
+
+--- a/Makefile.common.mk
++++ b/Makefile.common.mk
+@@ -65,7 +65,7 @@ JAVACOPTIONS = -O ${JAVACTARGETOPTIONS}
+ 
+ .java.class:
+       export JAVAVERSIONTARGETJARFILE=${JAVA_HOME}/jre/lib/rt.jar; javac ${JAVACOPTIONS} \
+-              -classpath ${PATHTOROOT}:${DICOMADDITIONALJARS}:${VIEWERADDITIONALJARS}:${FTPADDITIONALJARS}:${JUNITJAR} \
++              -classpath ${PATHTOROOT}:${DICOMADDITIONALJARS}:${VIEWERADDITIONALJARS}:${FTPADDITIONALJARS}:${JUNITJAR}:/usr/share/java/jaxb-api.jar \
+               -sourcepath ${PATHTOROOT} $<
+ 
+ .png.ico:
diff --git a/debian/patches/series b/debian/patches/series
index 10286d4..6abfc45 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -16,3 +16,4 @@ set_java_home.patch
 imageio.patch
 no_Xdiags_verbose.patch
 do_not_set_bootclasspath.patch
+jaxb-api.patch



I've pushed these changes but unfortunately it does not help. :-(

Any further hints?

Kind regards

      Andreas.


-- 
http://fam-tille.de