Recent changes to 114: Please be compatible with java9 modules

#114 Please be compatible with java9 modules

Mickael Guessant — Mon, 29 Oct 2018 15:44:57 -0000

Group: Next Release (example) --> Backlog

#114 Please be compatible with java9 modules

Mickael Guessant — Mon, 29 Oct 2018 15:44:51 -0000

Well, looks like Alexandre decided to move forward and include JNDI classes directly in DavMail tree. This may be the only practical way to handle this.

#114 Please be compatible with java9 modules

niol — Thu, 18 Oct 2018 13:42:42 -0000

What do you think about this? https://salsa.debian.org/debian/davmail/blob/master/debian/patches/embed-private-jdk-apis

#114 Please be compatible with java9 modules

Mickael Guessant — Wed, 27 Jun 2018 14:40:25 -0000

Well, the Sun classes are working fine :-)

Anyway, I also tried to move those classes under another name space, only need to move 5 classes.

#114 Please be compatible with java9 modules

niol — Wed, 27 Jun 2018 13:03:12 -0000

Thanks a lot. Is it possible to migrate to another BER encoding/decoding library?

#114 Please be compatible with java9 modules

Mickael Guessant — Tue, 26 Jun 2018 06:13:34 -0000

status: open --> accepted

Please be compatible with java9 modules

niol — Mon, 25 Jun 2018 12:57:27 -0000

java9 introduces modules which make some jdk apis private. For instance, com.sun.jndi.ldap which is in the module java.naming is not exported and thus not available for applications (Ber is internal proprietary API and has been removed).

The following patch works around the issue.

Maybe davmail should migrate to another BER encoding/decoding library.

--- davmail.git.orig/build.xml  2018-06-25 12:53:18.000000000 +0200
+++ davmail.git/build.xml       2018-06-25 14:45:50.902961330 +0200
@@ -88,10 +88,12 @@

     <target name="compile" depends="init">
         <mkdir dir="target/classes"/>
-        <javac srcdir="src/java" destdir="target/classes" source="1.6" target="1.6" debug="on" encoding="UTF-8"
+        <javac srcdir="src/java" destdir="target/classes" source="9" debug="on" encoding="UTF-8"
                includeantruntime="false">
+            <compilerarg value="--add-exports" />
+            <compilerarg value="java.naming/com.sun.jndi.ldap=ALL-UNNAMED" />
             <classpath>
                 <path refid="classpath"/>
             </classpath>

Please be compatible with java9 modules

niol — Mon, 25 Jun 2018 12:57:27 -0000

Ticket 114 has been modified: Please be compatible with java9 modules
Edited By: Mickael Guessant (mguessan)
Status updated: u'open' => u'accepted'

Please be compatible with java9 modules

niol — Mon, 25 Jun 2018 12:57:27 -0000

Ticket 114 has been modified: Please be compatible with java9 modules
Edited By: Mickael Guessant (mguessan)
_milestone updated: u'Next Release (example)' => u'Backlog'