BTW why compile time dependency should't be transitive?

On Mon, 16 Dec 2019 at 20:12, Maxim Solodovnik <solomax...@gmail.com> wrote:
>
> I can create PR in a minute (based on OPENJPA-2798-java11-friendly branch)
> Was planning to add "Automatic-Module-Name"s to manifest
>
> On Mon, 16 Dec 2019 at 20:03, Romain Manni-Bucau <rmannibu...@gmail.com> 
> wrote:
> >
> > Nobody uses it but adding them breaks apps ;), yeah.
> > Anyway openjpa shouldnt have jpa api transitive anyway IMHO so not a big
> > deal while we dont break OSGi case in the feature.xml.
> >
> > Romain Manni-Bucau
> > @rmannibucau <https://twitter.com/rmannibucau> |  Blog
> > <https://rmannibucau.metawerx.net/> | Old Blog
> > <http://rmannibucau.wordpress.com> | Github 
> > <https://github.com/rmannibucau> |
> > LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book
> > <https://www.packtpub.com/application-development/java-ee-8-high-performance>
> >
> >
> > Le lun. 16 déc. 2019 à 13:52, Mark Struberg <strub...@yahoo.de.invalid> a
> > écrit :
> >
> > > I'd go with fixing our module info in geronimo-jpa.
> > >
> > > But otoh nobody is using modules anyway it seems ...
> > >
> > > LieGrue,
> > > strub
> > >
> > > > Am 11.12.2019 um 06:39 schrieb Romain Manni-Bucau <rmannibu...@gmail.com
> > > >:
> > > >
> > > > Yes this one
> > > >
> > > > For openjpa the challenge will likely to not have module-info in main
> > > > artifacts - guess we can duplicate artifacts with a classifier jpms or
> > > so -
> > > > to avoid to break some servers and env.
> > > >
> > > > Last thing to take care: all osgi integration, i didnt check jakarta jar
> > > > supports it as well as our spec jar.
> > > >
> > > > So likely some validation work to do but yes it sounds like the plan.
> > > >
> > > > Hope it helps even if not a straight "yes".
> > > >
> > > > Le mer. 11 déc. 2019 à 03:57, Maxim Solodovnik <solomax...@gmail.com> a
> > > > écrit :
> > > >
> > > >> Do you mean this one:
> > > >>
> > > >> <dependency>
> > > >>    <groupId>org.eclipse.persistence</groupId>
> > > >>    <artifactId>jakarta.persistence</artifactId>
> > > >>    <version>2.2.2</version>
> > > >> </dependency>
> > > >>
> > > >> Looks good to me!
> > > >> Shall I create PR? (the task seems to be doable for me :)))
> > > >>
> > > >> On Wed, 11 Dec 2019 at 01:03, Romain Manni-Bucau <rmannibu...@gmail.com
> > > >
> > > >> wrote:
> > > >>
> > > >>> Hi
> > > >>>
> > > >>> I proposed some time ago to rename them all but since jakarta is at
> > > >> eclipse
> > > >>> now, i guess well more move to their bundles.
> > > >>>
> > > >>> Wdyt?
> > > >>>
> > > >>> Le mar. 10 déc. 2019 à 18:18, Maxim Solodovnik <solomax...@gmail.com>
> > > a
> > > >>> écrit :
> > > >>>
> > > >>>> Hello All,
> > > >>>>
> > > >>>> not sure which list should I ask, so will start here
> > > >>>>
> > > >>>> I have started to add module-info to our project
> > > >>>> The issue I found is:
> > > >>>> openjpa depends on "Apache Geronimo JPA Spec 2.2"
> > > >>>> geronimo jpa doesn't have automatic module name and automatically
> > > being
> > > >>>> resolved as
> > > >>>> geronimo.jpa.2.2.spec
> > > >>>> Which is invalid java identifier :(((
> > > >>>> Maybe it would be possible to release same artifact with some correct
> > > >>>> module name for ex. `geronimo.jpa_2_2.spec` and with incremented 
> > > >>>> micro
> > > >>>> version?
> > > >>>>
> > > >>>> Or maybe there are other options I\, not aware of?
> > > >>>>
> > > >>>> --
> > > >>>> WBR
> > > >>>> Maxim aka solomax
> > > >>>>
> > > >>>
> > > >>
> > > >>
> > > >> --
> > > >> WBR
> > > >> Maxim aka solomax
> > > >>
> > >
> > >
>
>
>
> --
> WBR
> Maxim aka solomax



-- 
WBR
Maxim aka solomax

Reply via email to