On Wed, Jan 20, 2016 at 5:01 PM, Michael Musgrove <mmusgrov@redhat.com> wrote:


On Wed, Jan 20, 2016 at 4:45 PM, Tomasz Adamski <tadamski@redhat.com> wrote:
> A follow up question is, if we add javax.orb.api as module dependency (in
> our wildfly transactions subsystem) then at runtime will we be using the
> classes provided by openjdk-orb?

If we use openjdk-orb in jboss-modules environment we are sure that openjdk-orb is used as orb classes are not exported as system packages.

Okay so we are covered in wildfly and EAP which is good to know.
 

> Is it the case that we cannot avoid this warning but
> that we are safe provided at runtime we use the classes in the openjdk-orb
> jar?

In standalone scenario it should work correctly too. I will look wheter the error is caused by jdeps tool behaviour or do we indeed have some unwanted runtime references to JDK.

In a standalone scenario we are not using modules so the JVM would use rt.jar unless the user updates the bootclasspath. I do not see how it would be possible to avoid the jdeps warnings.
 

> We probably should rename the packages to avoid confusion. What do you think
> Tomasz?

This can be done but I see one problem: openjdk-orb is a hg repository and we have possibility to synchronize it with JDK head if there are some changes in ORB code. I would have to script it somehow to preserve such possibility. But I agree that it will be nice to do.

I agree it would be nice (but I guess not a must have).

On second thoughts, for standalone usage it is a pain to have to tell the user that he must start the JVM with -Xbootclasspath/p:${openjdk-orb.jar} which might be a barrier to adoption for some users.

Mike

Mike 

--
Tomasz Adamski
Software Engineer
JBoss by Red Hat

----- Oryginalna wiadomość -----
> Od: "Jason Greene" <jason.greene@redhat.com>
> Do: "Michael Musgrove" <mmusgrov@redhat.com>
> DW: "WildFly Dev" <wildfly-dev@lists.jboss.org>, "Tomasz Adamski" <tadamski@redhat.com>
> Wysłane: środa, 20 styczeń 2016 17:13:04
> Temat: Re: [wildfly-dev] How to avoid depending on corba classes in rt.jar
>
> We probably should rename the packages to avoid confusion. What do you think
> Tomasz?
>
>
> > On Jan 20, 2016, at 10:07 AM, Michael Musgrove <mmusgrov@redhat.com> wrote:
> >
> > The narayana jts jar has dependencies on internal jdk orb classes. We test
> > that it can work with the openjdk-orb by adding the openjdk-orb jar to the
> > bootclasspath. However, jdeps -jdkinternals reports that we depend on
> > internal JDK APIs. Is it the case that we cannot avoid this warning but
> > that we are safe provided at runtime we use the classes in the openjdk-orb
> > jar?
> >
> > A follow up question is, if we add javax.orb.api as module dependency (in
> > our wildfly transactions subsystem) then at runtime will we be using the
> > classes provided by openjdk-orb?
> >
> > Thanks,
> > Mike
> >
> > --
> > Michael Musgrove
> > Transactions Team
> > e: mmusgrov@redhat.com <mailto:mmusgrov@redhat.com>
> > t: +44 191 243 0870
> >
> > Registered in England and Wales under Company Registration No. 03798903
> > Directors: Michael Cunningham (US), Paul Hickey (Ireland), Matt Parson
> > (US), Charles Peters (US)
> >
> > Michael Cunningham (US), Charles Peters (US), Matt Parson (US), Michael
> > O'Neill(Ireland)
> > _______________________________________________
> > wildfly-dev mailing list
> > wildfly-dev@lists.jboss.org
> > https://lists.jboss.org/mailman/listinfo/wildfly-dev
>
> --
> Jason T. Greene
> WildFly Lead / JBoss EAP Platform Architect
> JBoss, a division of Red Hat
>
>



--
Michael Musgrove
Transactions Team

Registered in England and Wales under Company Registration No. 03798903
Directors: Michael Cunningham (US), Paul Hickey (Ireland), Matt Parson
(US), Charles Peters (US)

Michael Cunningham (US), Charles Peters (US), Matt Parson (US), Michael O'Neill(Ireland)



--
Michael Musgrove
Transactions Team
e: mmusgrov@redhat.com
t: +44 191 243 0870

Registered in England and Wales under Company Registration No. 03798903
Directors: Michael Cunningham (US), Paul Hickey (Ireland), Matt Parson
(US), Charles Peters (US)

Michael Cunningham (US), Charles Peters (US), Matt Parson (US), Michael O'Neill(Ireland)