On Sat, Aug 30, 2014 at 11:58:08AM -0700, Lucas Nussbaum wrote:
> >   [javadoc] 
> > /«PKGBUILDDIR»/lib/src/jsip/src/gov/nist/javax/sip/header/AuthenticationHeader.java:443:
> >  error: unmappable character for encoding ASCII
> >   [javadoc]      * Set the nonce count pakrameter. Bug fix sent in by 
> > Andreas Bystr???m
[...]

I tried to fix this with this patch:

  * Set encoding for javadoc to fix errors with Java 7 (closes: #759835).

diff -Nru jitsi-2.4.4997/debian/patches/encoding.patch 
jitsi-2.4.4997/debian/patches/encoding.patch
--- jitsi-2.4.4997/debian/patches/encoding.patch        1970-01-01 
01:00:00.000000000 +0100
+++ jitsi-2.4.4997/debian/patches/encoding.patch        2014-09-05 
13:53:59.000000000 +0100
@@ -0,0 +1,33 @@
+Description: Set encoding for javadoc to fix errors with Java 7
+Author: Colin Watson <[email protected]>
+Bug-Debian: https://bugs.debian.org/759835
+Forwarded: no
+Last-Update: 2014-09-05
+
+Index: b/lib/src/jsip/build.xml
+===================================================================
+--- a/lib/src/jsip/build.xml
++++ b/lib/src/jsip/build.xml
+@@ -167,7 +167,7 @@
+             <fileset dir="src/javax/sip" includes="**/*.html" />
+         </copy>
+ 
+-        <javadoc 
packagenames="javax.sip.*,javax.sip.header.*,javax.sip.message.*,javax.sip.address.*,gov.nist.javax.sip.*,gov.nist.javax.sip.address.*,gov.nist.javax.sip.message.*,gov.nist.javax.sip.header.*,gov.nist.javas.sip.header.ims.*,gov.nist.javax.sip.stack.*,gov.nist.core.net.*,gov.nist.javax.sip.parser.*,javax.sdp.*,gov.nist.javax.sdp.*,gov.nist.javax.sdp.fields.*"
 sourcepath="${srcroot}" overview="${docdir}/overview.html" destdir="javadoc" 
use="true" splitindex="true" windowtitle="The JAIN-SIP-1.2 RI For the People !" 
doctitle="NIST-SIP: The Reference Implementation for JAIN-SIP 1.2" 
public="true" author="true" version="true" defaultexcludes="yes" 
additionalparam="-breakiterator">
++        <javadoc 
packagenames="javax.sip.*,javax.sip.header.*,javax.sip.message.*,javax.sip.address.*,gov.nist.javax.sip.*,gov.nist.javax.sip.address.*,gov.nist.javax.sip.message.*,gov.nist.javax.sip.header.*,gov.nist.javas.sip.header.ims.*,gov.nist.javax.sip.stack.*,gov.nist.core.net.*,gov.nist.javax.sip.parser.*,javax.sdp.*,gov.nist.javax.sdp.*,gov.nist.javax.sdp.fields.*"
 sourcepath="${srcroot}" overview="${docdir}/overview.html" destdir="javadoc" 
use="true" splitindex="true" windowtitle="The JAIN-SIP-1.2 RI For the People !" 
doctitle="NIST-SIP: The Reference Implementation for JAIN-SIP 1.2" 
public="true" author="true" version="true" defaultexcludes="yes" 
additionalparam="-breakiterator" encoding="iso-8859-1">
+             <classpath>
+                 <pathelement location="${log4j_jar}" />
+                 <pathelement location="${jain-sip-api-jar}" />
+Index: b/lib/src/swingworker/build.xml
+===================================================================
+--- a/lib/src/swingworker/build.xml
++++ b/lib/src/swingworker/build.xml
+@@ -63,7 +63,8 @@
+         <javadoc access="protected" link="" 
+             packagenames="org.jdesktop.swingworker" author="true" 
destdir="${javadoc.dir}" 
+             nodeprecated="false" nodeprecatedlist="false" noindex="false" 
+-            nonavbar="false" notree="false" use="true" version="true">
++            nonavbar="false" notree="false" use="true" version="true"
++            encoding="iso-8859-1">
+             <link href="${external.doc}" />
+             <sourcepath>
+                 <pathelement path="${src.dir}" />
diff -Nru jitsi-2.4.4997/debian/patches/series 
jitsi-2.4.4997/debian/patches/series
--- jitsi-2.4.4997/debian/patches/series        2014-05-20 20:29:22.000000000 
+0100
+++ jitsi-2.4.4997/debian/patches/series        2014-09-05 13:34:00.000000000 
+0100
@@ -3,3 +3,4 @@
 guava-update
 libav10.patch
 json.patch
+encoding.patch

This is probably incomplete, and there are concerning warnings such as
the following which indicate that a better approach would be to work
with upstream to get everything converted over to be consistently UTF-8
and then set encoding="utf-8" everywhere, or possibly force
LC_ALL=C.UTF-8 in debian/rules:

> > compiletck:
> > [jain-javac] /«PKGBUILDDIR»/lib/src/jsip/build.xml:360: warning: 
> > 'includeantruntime' was not set, defaulting to build.sysclasspath=last; set 
> > to false for repeatable builds
> > [jain-javac] Compiling 69 source files to 
> > /«PKGBUILDDIR»/lib/src/jsip/classes
> > [jain-javac] warning: [options] bootstrap class path not set in conjunction 
> > with -source 1.5
> > [jain-javac] 
> > /«PKGBUILDDIR»/lib/src/jsip/src/test/tck/factory/MessageFactoryTest.java:484:
> >  warning: unmappable character for encoding ASCII
> > [jain-javac]                byte[] content = "??".getBytes( "ISO-8859-1" );
> > [jain-javac]                                  ^
> > [jain-javac] 
> > /«PKGBUILDDIR»/lib/src/jsip/src/test/tck/factory/MessageFactoryTest.java:484:
> >  warning: unmappable character for encoding ASCII
> > [jain-javac]                byte[] content = "??".getBytes( "ISO-8859-1" );
> > [jain-javac]                                   ^

However, even after fixing all that, this remains a fatal error:

> > build.core.client.meta:
> >     [mkdir] Created dir: 
> > /«PKGBUILDDIR»/lib/src/gdata-java-client/java/genfiles/meta/client/META-INF/gdata/kinds
> >       [apt] 
> > /«PKGBUILDDIR»/lib/src/gdata-java-client/java/build-src/core.xml:310: 
> > warning: 'includeantruntime' was not set, defaulting to 
> > build.sysclasspath=last; set to false for repeatable builds
> >       [apt] Since compiler setting isn't classic or modern, ignoring fork 
> > setting.
> >       [apt] Since compiler setting isn't classic or modern, ignoring fork 
> > setting.
> >       [apt] Compiling 60 source files to 
> > /«PKGBUILDDIR»/lib/src/gdata-java-client/java/genfiles/meta/client/META-INF/gdata/kinds
> >       [apt] Since compiler setting isn't classic or modern, ignoring fork 
> > setting.
> >       [apt] E: Command line option 'd' [from -d] is not known.

This is because Java's annotation processing tool has been removed, and
it's now trying to call /usr/bin/apt from Debian's apt package, which
obviously doesn't work so well.  Resolving this will require somebody
who actually speaks Java.

-- 
Colin Watson                                       [[email protected]]


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to