[ 
https://issues.apache.org/jira/browse/OFBIZ-9226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15972217#comment-15972217
 ] 

Jacques Le Roux commented on OFBIZ-9226:
----------------------------------------

We had a discussion with Taher on HipChat. Here is the most relevant part

This article explains things in detail
[With Java 9 comes Project Jigsaw - a modularization of the JDK - which will 
break existing code. An overview over the planned changes lets you see whether 
yours is 
affected.|http://blog.codefx.org/java/dev/how-java-9-and-project-jigsaw-may-break-your-code/]
but as mentionned Taher 
bq. Unless we depend on some old sun.* in our framework we'll be fine 
I checked:
{code}
Mostly related to email handling
BlogRssServices
EmailServices
RomeEventHandler
Indirectly OFBizActiveDirectoryAuthenticationHandler & 
OFBizLdapAuthenticationHandler (used in context)

To be checked because I'm unsure after reading 
http://blog.codefx.org/java/dev/how-java-9-and-project-jigsaw-may-break-your-code/#Internal-APIs
 (I did not check  @jdk.Exported annotation in "sun" imported packages) 
{code}

Taher said
bq. Well I don't think any of the dependencies in the classes I saw are 
internal. com.sun.mail  is hardly internal.

I agreed
{code}
Yes, I think so, same for com.sun.syndication
for com.sun.jndi.ldap in ldap component, it's only indirect
/code 
env.put(Context.INITIAL_CONTEXT_FACTORY,"com.sun.jndi.ldap.LdapCtxFactory");
Anyway we just need to try
{code}
Still on the radar but not a priority...

> Check that OFBiz runs and compile with Oracle JDK 9 (Java 9)
> ------------------------------------------------------------
>
>                 Key: OFBIZ-9226
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-9226
>             Project: OFBiz
>          Issue Type: Test
>          Components: ALL COMPONENTS
>    Affects Versions: Trunk
>            Reporter: Jacques Le Roux
>            Priority: Trivial
>             Fix For: Upcoming Release
>
>
> After reading https://docs.oracle.com/javase/9/migrate/ I believe we should 
> not cross too much problems



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to