Package: pixelmed
Version: 20120508-1
Severity: normal
User: ubuntu-de...@lists.ubuntu.com
Usertags: openjdk-7-transition

Hi,

Your package FTBFS with Java7 because of (at least) two reasons:
 * missing -source 1.5 parameter for javac
 * use of com.sun class that disappeared in Java7

I have an example patch to solve the first issue, but not the later:

"""
javac -O -target 1.5 -source 1.5 -encoding "UTF8" -Xlint:deprecation -classpath 
../../..:/usr/share/java/excalibur-bzip2-1.0.jar:/usr/share/java/commons-codec.jar:/usr/share/java/vecmath.jar
 \
                -sourcepath ../../.. AttributeListTableBrowser.java
warning: [options] bootstrap class path not set in conjunction with -source 1.5
../../../com/pixelmed/display/ApplicationFrame.java:166: error: package 
com.sun.image.codec.jpeg does not exist
                        
com.sun.image.codec.jpeg.JPEGCodec.createJPEGEncoder(new 
FileOutputStream(snapShotFile)).encode(snapShotImage);
                                                ^
"""

You can reproduce this by using default-jdk from experimental.

We are trying to evaluate whether to migrate to Java7 as default in
Wheezy; we know it is very late in the cycle, but the reason is
#675495.

~Niels
diff -Nru pixelmed-20120508/debian/changelog pixelmed-20120508/debian/changelog
--- pixelmed-20120508/debian/changelog	2012-05-31 14:04:56.000000000 +0200
+++ pixelmed-20120508/debian/changelog	2012-06-21 12:38:14.000000000 +0200
@@ -1,3 +1,10 @@
+pixelmed (20120508-1.1) UNRELEASED; urgency=low
+
+  * Non-maintainer upload.
+  * Pass -source 1.5 to javac to fix FTBFS with Java7.
+
+ -- Niels Thykier <ni...@thykier.net>  Thu, 21 Jun 2012 12:38:03 +0200
+
 pixelmed (20120508-1) unstable; urgency=low
 
   * New upstream
diff -Nru pixelmed-20120508/debian/patches/javac-options.diff pixelmed-20120508/debian/patches/javac-options.diff
--- pixelmed-20120508/debian/patches/javac-options.diff	1970-01-01 01:00:00.000000000 +0100
+++ pixelmed-20120508/debian/patches/javac-options.diff	2012-06-21 12:38:49.000000000 +0200
@@ -0,0 +1,14 @@
+Description: Add -source 1.5 to javac options to fix FTBFS with Java7
+Author: Niels Thykier <ni...@thykier.net>
+
+--- pixelmed-20120508.orig/Makefile.common.mk
++++ pixelmed-20120508/Makefile.common.mk
+@@ -44,7 +44,7 @@ SERVERADDITIONALJARS = ${VIEWERADDITIONA
+ .SUFFIXES:	.class .java .ico .png
+ 
+ #JAVACOPTIONS = -O -target 1.5 -encoding "UTF8" -Xlint:deprecation -Xlint:unchecked
+-JAVACOPTIONS = -O -target 1.5 -encoding "UTF8" -Xlint:deprecation
++JAVACOPTIONS = -O -target 1.5 -source 1.5 -encoding "UTF8" -Xlint:deprecation
+ #JAVACOPTIONS = -g -O -target 1.5 -encoding "UTF8" -Xlint:deprecation
+ 
+ .java.class:
diff -Nru pixelmed-20120508/debian/patches/series pixelmed-20120508/debian/patches/series
--- pixelmed-20120508/debian/patches/series	2012-05-08 15:25:37.000000000 +0200
+++ pixelmed-20120508/debian/patches/series	2012-06-21 12:37:59.000000000 +0200
@@ -8,3 +8,4 @@
 addmanifest.patch
 fixfonts.patch
 removetest.patch
+javac-options.diff

Reply via email to