You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomee.apache.org by Romain Manni-Bucau <rm...@gmail.com> on 2018/11/15 10:40:26 UTC
Fwd: tomee git commit: TOMEE-2277 Java 11 - define modules for TomEE
At the moment, they are tight to maven artifact but we can improve later.
Hey JL, did you check these names are valid? Think the - can be an issue
(jdk.internal.module.Checks#requireModuleName)
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>
---------- Forwarded message ---------
From: <jl...@apache.org>
Date: jeu. 15 nov. 2018 à 11:32
Subject: tomee git commit: TOMEE-2277 Java 11 - define modules for TomEE At
the moment, they are tight to maven artifact but we can improve later.
To: <co...@tomee.apache.org>
Repository: tomee
Updated Branches:
refs/heads/master b470fc8ce -> 127207255
TOMEE-2277 Java 11 - define modules for TomEE
At the moment, they are tight to maven artifact but we can improve later.
Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/12720725
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/12720725
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/12720725
Branch: refs/heads/master
Commit: 12720725544cd731f7192d7d9d4483974c62ac06
Parents: b470fc8
Author: Jean-Louis Monteiro <je...@gmail.com>
Authored: Thu Nov 15 11:32:12 2018 +0100
Committer: Jean-Louis Monteiro <je...@gmail.com>
Committed: Thu Nov 15 11:32:12 2018 +0100
----------------------------------------------------------------------
container/openejb-core/pom.xml | 3 ++-
container/openejb-javaagent/pom.xml | 3 ++-
itests/openejb-itests-client/pom.xml | 5 ++++-
pom.xml | 8 +++++++-
server/openejb-client/pom.xml | 5 ++++-
tomee/apache-tomee/pom.xml | 5 +++++
tomee/tomee-embedded/pom.xml | 5 ++++-
tomee/tomee-webapp/pom.xml | 5 +++++
8 files changed, 33 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/container/openejb-core/pom.xml
----------------------------------------------------------------------
diff --git a/container/openejb-core/pom.xml b/container/openejb-core/pom.xml
index bcd6dd6..7aadde4 100644
--- a/container/openejb-core/pom.xml
+++ b/container/openejb-core/pom.xml
@@ -392,12 +392,13 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
- <archive>
+ <archive combine.children="append">
<manifest>
<mainClass>org.apache.openejb.cli.Bootstrap</mainClass>
<!--<addClasspath>true</addClasspath>-->
</manifest>
<manifestEntries>
+ <Automatic-Module-Name>${tomee.build.name
}</Automatic-Module-Name>
<Class-Path>openejb-loader-${project.version}.jar
openejb-client-${project.version}.jar
xbean-finder-shaded-${xbeanVersion}.jar
xbean-asm7-shaded-${xbeanVersion}.jar
</Class-Path>
http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/container/openejb-javaagent/pom.xml
----------------------------------------------------------------------
diff --git a/container/openejb-javaagent/pom.xml
b/container/openejb-javaagent/pom.xml
index 5e69a63..4405551 100644
--- a/container/openejb-javaagent/pom.xml
+++ b/container/openejb-javaagent/pom.xml
@@ -32,8 +32,9 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
- <archive>
+ <archive combine.children="append">
<manifestEntries>
+ <Automatic-Module-Name>${tomee.build.name
}</Automatic-Module-Name>
<Premain-Class>org.apache.openejb.javaagent.Agent</Premain-Class>
<Agent-Class>org.apache.openejb.javaagent.Agent</Agent-Class>
<Can-Redefine-Classes>true</Can-Redefine-Classes>
http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/itests/openejb-itests-client/pom.xml
----------------------------------------------------------------------
diff --git a/itests/openejb-itests-client/pom.xml
b/itests/openejb-itests-client/pom.xml
index 626a74d..fe3b24b 100644
--- a/itests/openejb-itests-client/pom.xml
+++ b/itests/openejb-itests-client/pom.xml
@@ -45,11 +45,14 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
- <archive>
+ <archive combine.children="append">
<manifest>
<mainClass>org.apache.openejb.test.Main</mainClass>
<addClasspath>true</addClasspath>
</manifest>
+ <manifestEntries>
+ <Automatic-Module-Name>${tomee.build.name
}</Automatic-Module-Name>
+ </manifestEntries>
</archive>
</configuration>
</plugin>
http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index 537ec3a..404521e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -99,6 +99,9 @@
<maven.compiler.target>1.8</maven.compiler.target>
<surefire.version>2.21.0</surefire.version>
+ <!-- for the default name of the module -->
+ <tomee.build.name>${project.groupId}.${project.artifactId}</
tomee.build.name>
+
<!-- To easily change the javaee api version -->
<version.javaee-api>8.0</version.javaee-api>
@@ -455,8 +458,11 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
- <archive>
+ <archive combine.children="append">
<manifestFile>${project.build.outputDirectory}/META-INF/MANIFEST.MF</manifestFile>
+ <manifestEntries>
+ <Automatic-Module-Name>${tomee.build.name
}</Automatic-Module-Name>
+ </manifestEntries>
</archive>
</configuration>
</plugin>
http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/server/openejb-client/pom.xml
----------------------------------------------------------------------
diff --git a/server/openejb-client/pom.xml b/server/openejb-client/pom.xml
index c40d60c..5618ec4 100644
--- a/server/openejb-client/pom.xml
+++ b/server/openejb-client/pom.xml
@@ -80,10 +80,13 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
- <archive>
+ <archive combine.children="append">
<manifest>
<mainClass>org.apache.openejb.client.Main</mainClass>
</manifest>
+ <manifestEntries>
+ <Automatic-Module-Name>${tomee.build.name
}</Automatic-Module-Name>
+ </manifestEntries>
</archive>
</configuration>
</plugin>
http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/apache-tomee/pom.xml
----------------------------------------------------------------------
diff --git a/tomee/apache-tomee/pom.xml b/tomee/apache-tomee/pom.xml
index 574c562..d02a785 100644
--- a/tomee/apache-tomee/pom.xml
+++ b/tomee/apache-tomee/pom.xml
@@ -140,6 +140,11 @@
<include>**/RemoteTomEEEJBContainer*.class</include>
<include>**/META-INF/services/javax.ejb.spi.EJBContainerProvider</include>
</includes>
+ <archive combine.children="append">
+ <manifestEntries>
+ <Automatic-Module-Name>${tomee.build.name
}</Automatic-Module-Name>
+ </manifestEntries>
+ </archive>
</configuration>
</plugin>
<plugin>
http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/tomee-embedded/pom.xml
----------------------------------------------------------------------
diff --git a/tomee/tomee-embedded/pom.xml b/tomee/tomee-embedded/pom.xml
index 659fa1c..bcb6c72 100644
--- a/tomee/tomee-embedded/pom.xml
+++ b/tomee/tomee-embedded/pom.xml
@@ -44,11 +44,14 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
- <archive>
+ <archive combine.children="append">
<manifest>
<Main-Class>org.apache.tomee.embedded.Main</Main-Class>
<addClasspath>true</addClasspath>
</manifest>
+ <manifestEntries>
+ <Automatic-Module-Name>${tomee.build.name
}</Automatic-Module-Name>
+ </manifestEntries>
</archive>
</configuration>
</plugin>
http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/tomee-webapp/pom.xml
----------------------------------------------------------------------
diff --git a/tomee/tomee-webapp/pom.xml b/tomee/tomee-webapp/pom.xml
index c538123..9ba180e 100644
--- a/tomee/tomee-webapp/pom.xml
+++ b/tomee/tomee-webapp/pom.xml
@@ -76,6 +76,11 @@
<include>META-INF/**/*</include>
<include>juli.properties</include>
</includes>
+ <archive combine.children="append">
+ <manifestEntries>
+ <Automatic-Module-Name>${tomee.build.name
}</Automatic-Module-Name>
+ </manifestEntries>
+ </archive>
</configuration>
</plugin>
<plugin>
Re: tomee git commit: TOMEE-2277 Java 11 - define modules for TomEE
At the moment, they are tight to maven artifact but we can improve later.
Posted by Jean-Louis Monteiro <jl...@tomitribe.com>.
I'll do now
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com
On Thu, Nov 15, 2018 at 12:11 PM Romain Manni-Bucau <rm...@gmail.com>
wrote:
> The full tomee build does not rely on modules even ran with java11 so it
> does not surprises me much.
>
> Try to create a new project and require openejb-core module for instance.
> Will be enough to check
>
> 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 jeu. 15 nov. 2018 à 12:09, Jean-Louis Monteiro <
> jlmonteiro@tomitribe.com>
> a écrit :
>
> > I built and ran TomEE with some opens and exports and did not get any
> > exception.
> > But I'm happy to do a second round and override the default name
> >
> > --
> > Jean-Louis Monteiro
> > http://twitter.com/jlouismonteiro
> > http://www.tomitribe.com
> >
> >
> > On Thu, Nov 15, 2018 at 11:40 AM Romain Manni-Bucau <
> rmannibucau@gmail.com
> > >
> > wrote:
> >
> > > Hey JL, did you check these names are valid? Think the - can be an
> issue
> > > (jdk.internal.module.Checks#requireModuleName)
> > >
> > > 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
> > > >
> > >
> > >
> > > ---------- Forwarded message ---------
> > > From: <jl...@apache.org>
> > > Date: jeu. 15 nov. 2018 à 11:32
> > > Subject: tomee git commit: TOMEE-2277 Java 11 - define modules for
> TomEE
> > At
> > > the moment, they are tight to maven artifact but we can improve later.
> > > To: <co...@tomee.apache.org>
> > >
> > >
> > > Repository: tomee
> > > Updated Branches:
> > > refs/heads/master b470fc8ce -> 127207255
> > >
> > >
> > > TOMEE-2277 Java 11 - define modules for TomEE
> > > At the moment, they are tight to maven artifact but we can improve
> later.
> > >
> > >
> > > Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
> > > Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/12720725
> > > Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/12720725
> > > Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/12720725
> > >
> > > Branch: refs/heads/master
> > > Commit: 12720725544cd731f7192d7d9d4483974c62ac06
> > > Parents: b470fc8
> > > Author: Jean-Louis Monteiro <je...@gmail.com>
> > > Authored: Thu Nov 15 11:32:12 2018 +0100
> > > Committer: Jean-Louis Monteiro <je...@gmail.com>
> > > Committed: Thu Nov 15 11:32:12 2018 +0100
> > >
> > > ----------------------------------------------------------------------
> > > container/openejb-core/pom.xml | 3 ++-
> > > container/openejb-javaagent/pom.xml | 3 ++-
> > > itests/openejb-itests-client/pom.xml | 5 ++++-
> > > pom.xml | 8 +++++++-
> > > server/openejb-client/pom.xml | 5 ++++-
> > > tomee/apache-tomee/pom.xml | 5 +++++
> > > tomee/tomee-embedded/pom.xml | 5 ++++-
> > > tomee/tomee-webapp/pom.xml | 5 +++++
> > > 8 files changed, 33 insertions(+), 6 deletions(-)
> > > ----------------------------------------------------------------------
> > >
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/container/openejb-core/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/container/openejb-core/pom.xml
> > > b/container/openejb-core/pom.xml
> > > index bcd6dd6..7aadde4 100644
> > > --- a/container/openejb-core/pom.xml
> > > +++ b/container/openejb-core/pom.xml
> > > @@ -392,12 +392,13 @@
> > > <groupId>org.apache.maven.plugins</groupId>
> > > <artifactId>maven-jar-plugin</artifactId>
> > > <configuration>
> > > - <archive>
> > > + <archive combine.children="append">
> > > <manifest>
> > > <mainClass>org.apache.openejb.cli.Bootstrap</mainClass>
> > > <!--<addClasspath>true</addClasspath>-->
> > > </manifest>
> > > <manifestEntries>
> > > + <Automatic-Module-Name>${tomee.build.name
> > > }</Automatic-Module-Name>
> > > <Class-Path>openejb-loader-${project.version}.jar
> > > openejb-client-${project.version}.jar
> > > xbean-finder-shaded-${xbeanVersion}.jar
> > > xbean-asm7-shaded-${xbeanVersion}.jar
> > > </Class-Path>
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/container/openejb-javaagent/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/container/openejb-javaagent/pom.xml
> > > b/container/openejb-javaagent/pom.xml
> > > index 5e69a63..4405551 100644
> > > --- a/container/openejb-javaagent/pom.xml
> > > +++ b/container/openejb-javaagent/pom.xml
> > > @@ -32,8 +32,9 @@
> > > <groupId>org.apache.maven.plugins</groupId>
> > > <artifactId>maven-jar-plugin</artifactId>
> > > <configuration>
> > > - <archive>
> > > + <archive combine.children="append">
> > > <manifestEntries>
> > > + <Automatic-Module-Name>${tomee.build.name
> > > }</Automatic-Module-Name>
> > >
> > > <Premain-Class>org.apache.openejb.javaagent.Agent</Premain-Class>
> > >
> > > <Agent-Class>org.apache.openejb.javaagent.Agent</Agent-Class>
> > > <Can-Redefine-Classes>true</Can-Redefine-Classes>
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/itests/openejb-itests-client/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/itests/openejb-itests-client/pom.xml
> > > b/itests/openejb-itests-client/pom.xml
> > > index 626a74d..fe3b24b 100644
> > > --- a/itests/openejb-itests-client/pom.xml
> > > +++ b/itests/openejb-itests-client/pom.xml
> > > @@ -45,11 +45,14 @@
> > > <groupId>org.apache.maven.plugins</groupId>
> > > <artifactId>maven-jar-plugin</artifactId>
> > > <configuration>
> > > - <archive>
> > > + <archive combine.children="append">
> > > <manifest>
> > > <mainClass>org.apache.openejb.test.Main</mainClass>
> > > <addClasspath>true</addClasspath>
> > > </manifest>
> > > + <manifestEntries>
> > > + <Automatic-Module-Name>${tomee.build.name
> > > }</Automatic-Module-Name>
> > > + </manifestEntries>
> > > </archive>
> > > </configuration>
> > > </plugin>
> > >
> > > http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/pom.xml b/pom.xml
> > > index 537ec3a..404521e 100644
> > > --- a/pom.xml
> > > +++ b/pom.xml
> > > @@ -99,6 +99,9 @@
> > > <maven.compiler.target>1.8</maven.compiler.target>
> > > <surefire.version>2.21.0</surefire.version>
> > >
> > > + <!-- for the default name of the module -->
> > > + <tomee.build.name>${project.groupId}.${project.artifactId}</
> > > tomee.build.name>
> > > +
> > > <!-- To easily change the javaee api version -->
> > > <version.javaee-api>8.0</version.javaee-api>
> > >
> > > @@ -455,8 +458,11 @@
> > > <groupId>org.apache.maven.plugins</groupId>
> > > <artifactId>maven-jar-plugin</artifactId>
> > > <configuration>
> > > - <archive>
> > > + <archive combine.children="append">
> > >
> > >
> > >
> >
> <manifestFile>${project.build.outputDirectory}/META-INF/MANIFEST.MF</manifestFile>
> > > + <manifestEntries>
> > > + <Automatic-Module-Name>${tomee.build.name
> > > }</Automatic-Module-Name>
> > > + </manifestEntries>
> > > </archive>
> > > </configuration>
> > > </plugin>
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/server/openejb-client/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/server/openejb-client/pom.xml
> > b/server/openejb-client/pom.xml
> > > index c40d60c..5618ec4 100644
> > > --- a/server/openejb-client/pom.xml
> > > +++ b/server/openejb-client/pom.xml
> > > @@ -80,10 +80,13 @@
> > > <groupId>org.apache.maven.plugins</groupId>
> > > <artifactId>maven-jar-plugin</artifactId>
> > > <configuration>
> > > - <archive>
> > > + <archive combine.children="append">
> > > <manifest>
> > > <mainClass>org.apache.openejb.client.Main</mainClass>
> > > </manifest>
> > > + <manifestEntries>
> > > + <Automatic-Module-Name>${tomee.build.name
> > > }</Automatic-Module-Name>
> > > + </manifestEntries>
> > > </archive>
> > > </configuration>
> > > </plugin>
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/apache-tomee/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/tomee/apache-tomee/pom.xml b/tomee/apache-tomee/pom.xml
> > > index 574c562..d02a785 100644
> > > --- a/tomee/apache-tomee/pom.xml
> > > +++ b/tomee/apache-tomee/pom.xml
> > > @@ -140,6 +140,11 @@
> > > <include>**/RemoteTomEEEJBContainer*.class</include>
> > >
> > >
> >
> <include>**/META-INF/services/javax.ejb.spi.EJBContainerProvider</include>
> > > </includes>
> > > + <archive combine.children="append">
> > > + <manifestEntries>
> > > + <Automatic-Module-Name>${tomee.build.name
> > > }</Automatic-Module-Name>
> > > + </manifestEntries>
> > > + </archive>
> > > </configuration>
> > > </plugin>
> > > <plugin>
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/tomee-embedded/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/tomee/tomee-embedded/pom.xml
> b/tomee/tomee-embedded/pom.xml
> > > index 659fa1c..bcb6c72 100644
> > > --- a/tomee/tomee-embedded/pom.xml
> > > +++ b/tomee/tomee-embedded/pom.xml
> > > @@ -44,11 +44,14 @@
> > > <groupId>org.apache.maven.plugins</groupId>
> > > <artifactId>maven-jar-plugin</artifactId>
> > > <configuration>
> > > - <archive>
> > > + <archive combine.children="append">
> > > <manifest>
> > > <Main-Class>org.apache.tomee.embedded.Main</Main-Class>
> > > <addClasspath>true</addClasspath>
> > > </manifest>
> > > + <manifestEntries>
> > > + <Automatic-Module-Name>${tomee.build.name
> > > }</Automatic-Module-Name>
> > > + </manifestEntries>
> > > </archive>
> > > </configuration>
> > > </plugin>
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/tomee-webapp/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/tomee/tomee-webapp/pom.xml b/tomee/tomee-webapp/pom.xml
> > > index c538123..9ba180e 100644
> > > --- a/tomee/tomee-webapp/pom.xml
> > > +++ b/tomee/tomee-webapp/pom.xml
> > > @@ -76,6 +76,11 @@
> > > <include>META-INF/**/*</include>
> > > <include>juli.properties</include>
> > > </includes>
> > > + <archive combine.children="append">
> > > + <manifestEntries>
> > > + <Automatic-Module-Name>${tomee.build.name
> > > }</Automatic-Module-Name>
> > > + </manifestEntries>
> > > + </archive>
> > > </configuration>
> > > </plugin>
> > > <plugin>
> > >
> >
>
Re: tomee git commit: TOMEE-2277 Java 11 - define modules for TomEE
At the moment, they are tight to maven artifact but we can improve later.
Posted by Romain Manni-Bucau <rm...@gmail.com>.
The full tomee build does not rely on modules even ran with java11 so it
does not surprises me much.
Try to create a new project and require openejb-core module for instance.
Will be enough to check
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 jeu. 15 nov. 2018 à 12:09, Jean-Louis Monteiro <jl...@tomitribe.com>
a écrit :
> I built and ran TomEE with some opens and exports and did not get any
> exception.
> But I'm happy to do a second round and override the default name
>
> --
> Jean-Louis Monteiro
> http://twitter.com/jlouismonteiro
> http://www.tomitribe.com
>
>
> On Thu, Nov 15, 2018 at 11:40 AM Romain Manni-Bucau <rmannibucau@gmail.com
> >
> wrote:
>
> > Hey JL, did you check these names are valid? Think the - can be an issue
> > (jdk.internal.module.Checks#requireModuleName)
> >
> > 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
> > >
> >
> >
> > ---------- Forwarded message ---------
> > From: <jl...@apache.org>
> > Date: jeu. 15 nov. 2018 à 11:32
> > Subject: tomee git commit: TOMEE-2277 Java 11 - define modules for TomEE
> At
> > the moment, they are tight to maven artifact but we can improve later.
> > To: <co...@tomee.apache.org>
> >
> >
> > Repository: tomee
> > Updated Branches:
> > refs/heads/master b470fc8ce -> 127207255
> >
> >
> > TOMEE-2277 Java 11 - define modules for TomEE
> > At the moment, they are tight to maven artifact but we can improve later.
> >
> >
> > Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
> > Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/12720725
> > Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/12720725
> > Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/12720725
> >
> > Branch: refs/heads/master
> > Commit: 12720725544cd731f7192d7d9d4483974c62ac06
> > Parents: b470fc8
> > Author: Jean-Louis Monteiro <je...@gmail.com>
> > Authored: Thu Nov 15 11:32:12 2018 +0100
> > Committer: Jean-Louis Monteiro <je...@gmail.com>
> > Committed: Thu Nov 15 11:32:12 2018 +0100
> >
> > ----------------------------------------------------------------------
> > container/openejb-core/pom.xml | 3 ++-
> > container/openejb-javaagent/pom.xml | 3 ++-
> > itests/openejb-itests-client/pom.xml | 5 ++++-
> > pom.xml | 8 +++++++-
> > server/openejb-client/pom.xml | 5 ++++-
> > tomee/apache-tomee/pom.xml | 5 +++++
> > tomee/tomee-embedded/pom.xml | 5 ++++-
> > tomee/tomee-webapp/pom.xml | 5 +++++
> > 8 files changed, 33 insertions(+), 6 deletions(-)
> > ----------------------------------------------------------------------
> >
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/container/openejb-core/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/container/openejb-core/pom.xml
> > b/container/openejb-core/pom.xml
> > index bcd6dd6..7aadde4 100644
> > --- a/container/openejb-core/pom.xml
> > +++ b/container/openejb-core/pom.xml
> > @@ -392,12 +392,13 @@
> > <groupId>org.apache.maven.plugins</groupId>
> > <artifactId>maven-jar-plugin</artifactId>
> > <configuration>
> > - <archive>
> > + <archive combine.children="append">
> > <manifest>
> > <mainClass>org.apache.openejb.cli.Bootstrap</mainClass>
> > <!--<addClasspath>true</addClasspath>-->
> > </manifest>
> > <manifestEntries>
> > + <Automatic-Module-Name>${tomee.build.name
> > }</Automatic-Module-Name>
> > <Class-Path>openejb-loader-${project.version}.jar
> > openejb-client-${project.version}.jar
> > xbean-finder-shaded-${xbeanVersion}.jar
> > xbean-asm7-shaded-${xbeanVersion}.jar
> > </Class-Path>
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/container/openejb-javaagent/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/container/openejb-javaagent/pom.xml
> > b/container/openejb-javaagent/pom.xml
> > index 5e69a63..4405551 100644
> > --- a/container/openejb-javaagent/pom.xml
> > +++ b/container/openejb-javaagent/pom.xml
> > @@ -32,8 +32,9 @@
> > <groupId>org.apache.maven.plugins</groupId>
> > <artifactId>maven-jar-plugin</artifactId>
> > <configuration>
> > - <archive>
> > + <archive combine.children="append">
> > <manifestEntries>
> > + <Automatic-Module-Name>${tomee.build.name
> > }</Automatic-Module-Name>
> >
> > <Premain-Class>org.apache.openejb.javaagent.Agent</Premain-Class>
> >
> > <Agent-Class>org.apache.openejb.javaagent.Agent</Agent-Class>
> > <Can-Redefine-Classes>true</Can-Redefine-Classes>
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/itests/openejb-itests-client/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/itests/openejb-itests-client/pom.xml
> > b/itests/openejb-itests-client/pom.xml
> > index 626a74d..fe3b24b 100644
> > --- a/itests/openejb-itests-client/pom.xml
> > +++ b/itests/openejb-itests-client/pom.xml
> > @@ -45,11 +45,14 @@
> > <groupId>org.apache.maven.plugins</groupId>
> > <artifactId>maven-jar-plugin</artifactId>
> > <configuration>
> > - <archive>
> > + <archive combine.children="append">
> > <manifest>
> > <mainClass>org.apache.openejb.test.Main</mainClass>
> > <addClasspath>true</addClasspath>
> > </manifest>
> > + <manifestEntries>
> > + <Automatic-Module-Name>${tomee.build.name
> > }</Automatic-Module-Name>
> > + </manifestEntries>
> > </archive>
> > </configuration>
> > </plugin>
> >
> > http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/pom.xml b/pom.xml
> > index 537ec3a..404521e 100644
> > --- a/pom.xml
> > +++ b/pom.xml
> > @@ -99,6 +99,9 @@
> > <maven.compiler.target>1.8</maven.compiler.target>
> > <surefire.version>2.21.0</surefire.version>
> >
> > + <!-- for the default name of the module -->
> > + <tomee.build.name>${project.groupId}.${project.artifactId}</
> > tomee.build.name>
> > +
> > <!-- To easily change the javaee api version -->
> > <version.javaee-api>8.0</version.javaee-api>
> >
> > @@ -455,8 +458,11 @@
> > <groupId>org.apache.maven.plugins</groupId>
> > <artifactId>maven-jar-plugin</artifactId>
> > <configuration>
> > - <archive>
> > + <archive combine.children="append">
> >
> >
> >
> <manifestFile>${project.build.outputDirectory}/META-INF/MANIFEST.MF</manifestFile>
> > + <manifestEntries>
> > + <Automatic-Module-Name>${tomee.build.name
> > }</Automatic-Module-Name>
> > + </manifestEntries>
> > </archive>
> > </configuration>
> > </plugin>
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/server/openejb-client/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/server/openejb-client/pom.xml
> b/server/openejb-client/pom.xml
> > index c40d60c..5618ec4 100644
> > --- a/server/openejb-client/pom.xml
> > +++ b/server/openejb-client/pom.xml
> > @@ -80,10 +80,13 @@
> > <groupId>org.apache.maven.plugins</groupId>
> > <artifactId>maven-jar-plugin</artifactId>
> > <configuration>
> > - <archive>
> > + <archive combine.children="append">
> > <manifest>
> > <mainClass>org.apache.openejb.client.Main</mainClass>
> > </manifest>
> > + <manifestEntries>
> > + <Automatic-Module-Name>${tomee.build.name
> > }</Automatic-Module-Name>
> > + </manifestEntries>
> > </archive>
> > </configuration>
> > </plugin>
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/apache-tomee/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/tomee/apache-tomee/pom.xml b/tomee/apache-tomee/pom.xml
> > index 574c562..d02a785 100644
> > --- a/tomee/apache-tomee/pom.xml
> > +++ b/tomee/apache-tomee/pom.xml
> > @@ -140,6 +140,11 @@
> > <include>**/RemoteTomEEEJBContainer*.class</include>
> >
> >
> <include>**/META-INF/services/javax.ejb.spi.EJBContainerProvider</include>
> > </includes>
> > + <archive combine.children="append">
> > + <manifestEntries>
> > + <Automatic-Module-Name>${tomee.build.name
> > }</Automatic-Module-Name>
> > + </manifestEntries>
> > + </archive>
> > </configuration>
> > </plugin>
> > <plugin>
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/tomee-embedded/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/tomee/tomee-embedded/pom.xml b/tomee/tomee-embedded/pom.xml
> > index 659fa1c..bcb6c72 100644
> > --- a/tomee/tomee-embedded/pom.xml
> > +++ b/tomee/tomee-embedded/pom.xml
> > @@ -44,11 +44,14 @@
> > <groupId>org.apache.maven.plugins</groupId>
> > <artifactId>maven-jar-plugin</artifactId>
> > <configuration>
> > - <archive>
> > + <archive combine.children="append">
> > <manifest>
> > <Main-Class>org.apache.tomee.embedded.Main</Main-Class>
> > <addClasspath>true</addClasspath>
> > </manifest>
> > + <manifestEntries>
> > + <Automatic-Module-Name>${tomee.build.name
> > }</Automatic-Module-Name>
> > + </manifestEntries>
> > </archive>
> > </configuration>
> > </plugin>
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/tomee-webapp/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/tomee/tomee-webapp/pom.xml b/tomee/tomee-webapp/pom.xml
> > index c538123..9ba180e 100644
> > --- a/tomee/tomee-webapp/pom.xml
> > +++ b/tomee/tomee-webapp/pom.xml
> > @@ -76,6 +76,11 @@
> > <include>META-INF/**/*</include>
> > <include>juli.properties</include>
> > </includes>
> > + <archive combine.children="append">
> > + <manifestEntries>
> > + <Automatic-Module-Name>${tomee.build.name
> > }</Automatic-Module-Name>
> > + </manifestEntries>
> > + </archive>
> > </configuration>
> > </plugin>
> > <plugin>
> >
>
Re: tomee git commit: TOMEE-2277 Java 11 - define modules for TomEE
At the moment, they are tight to maven artifact but we can improve later.
Posted by Jean-Louis Monteiro <jl...@tomitribe.com>.
I built and ran TomEE with some opens and exports and did not get any
exception.
But I'm happy to do a second round and override the default name
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com
On Thu, Nov 15, 2018 at 11:40 AM Romain Manni-Bucau <rm...@gmail.com>
wrote:
> Hey JL, did you check these names are valid? Think the - can be an issue
> (jdk.internal.module.Checks#requireModuleName)
>
> 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
> >
>
>
> ---------- Forwarded message ---------
> From: <jl...@apache.org>
> Date: jeu. 15 nov. 2018 à 11:32
> Subject: tomee git commit: TOMEE-2277 Java 11 - define modules for TomEE At
> the moment, they are tight to maven artifact but we can improve later.
> To: <co...@tomee.apache.org>
>
>
> Repository: tomee
> Updated Branches:
> refs/heads/master b470fc8ce -> 127207255
>
>
> TOMEE-2277 Java 11 - define modules for TomEE
> At the moment, they are tight to maven artifact but we can improve later.
>
>
> Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
> Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/12720725
> Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/12720725
> Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/12720725
>
> Branch: refs/heads/master
> Commit: 12720725544cd731f7192d7d9d4483974c62ac06
> Parents: b470fc8
> Author: Jean-Louis Monteiro <je...@gmail.com>
> Authored: Thu Nov 15 11:32:12 2018 +0100
> Committer: Jean-Louis Monteiro <je...@gmail.com>
> Committed: Thu Nov 15 11:32:12 2018 +0100
>
> ----------------------------------------------------------------------
> container/openejb-core/pom.xml | 3 ++-
> container/openejb-javaagent/pom.xml | 3 ++-
> itests/openejb-itests-client/pom.xml | 5 ++++-
> pom.xml | 8 +++++++-
> server/openejb-client/pom.xml | 5 ++++-
> tomee/apache-tomee/pom.xml | 5 +++++
> tomee/tomee-embedded/pom.xml | 5 ++++-
> tomee/tomee-webapp/pom.xml | 5 +++++
> 8 files changed, 33 insertions(+), 6 deletions(-)
> ----------------------------------------------------------------------
>
>
>
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/container/openejb-core/pom.xml
> ----------------------------------------------------------------------
> diff --git a/container/openejb-core/pom.xml
> b/container/openejb-core/pom.xml
> index bcd6dd6..7aadde4 100644
> --- a/container/openejb-core/pom.xml
> +++ b/container/openejb-core/pom.xml
> @@ -392,12 +392,13 @@
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-jar-plugin</artifactId>
> <configuration>
> - <archive>
> + <archive combine.children="append">
> <manifest>
> <mainClass>org.apache.openejb.cli.Bootstrap</mainClass>
> <!--<addClasspath>true</addClasspath>-->
> </manifest>
> <manifestEntries>
> + <Automatic-Module-Name>${tomee.build.name
> }</Automatic-Module-Name>
> <Class-Path>openejb-loader-${project.version}.jar
> openejb-client-${project.version}.jar
> xbean-finder-shaded-${xbeanVersion}.jar
> xbean-asm7-shaded-${xbeanVersion}.jar
> </Class-Path>
>
>
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/container/openejb-javaagent/pom.xml
> ----------------------------------------------------------------------
> diff --git a/container/openejb-javaagent/pom.xml
> b/container/openejb-javaagent/pom.xml
> index 5e69a63..4405551 100644
> --- a/container/openejb-javaagent/pom.xml
> +++ b/container/openejb-javaagent/pom.xml
> @@ -32,8 +32,9 @@
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-jar-plugin</artifactId>
> <configuration>
> - <archive>
> + <archive combine.children="append">
> <manifestEntries>
> + <Automatic-Module-Name>${tomee.build.name
> }</Automatic-Module-Name>
>
> <Premain-Class>org.apache.openejb.javaagent.Agent</Premain-Class>
>
> <Agent-Class>org.apache.openejb.javaagent.Agent</Agent-Class>
> <Can-Redefine-Classes>true</Can-Redefine-Classes>
>
>
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/itests/openejb-itests-client/pom.xml
> ----------------------------------------------------------------------
> diff --git a/itests/openejb-itests-client/pom.xml
> b/itests/openejb-itests-client/pom.xml
> index 626a74d..fe3b24b 100644
> --- a/itests/openejb-itests-client/pom.xml
> +++ b/itests/openejb-itests-client/pom.xml
> @@ -45,11 +45,14 @@
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-jar-plugin</artifactId>
> <configuration>
> - <archive>
> + <archive combine.children="append">
> <manifest>
> <mainClass>org.apache.openejb.test.Main</mainClass>
> <addClasspath>true</addClasspath>
> </manifest>
> + <manifestEntries>
> + <Automatic-Module-Name>${tomee.build.name
> }</Automatic-Module-Name>
> + </manifestEntries>
> </archive>
> </configuration>
> </plugin>
>
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/pom.xml
> ----------------------------------------------------------------------
> diff --git a/pom.xml b/pom.xml
> index 537ec3a..404521e 100644
> --- a/pom.xml
> +++ b/pom.xml
> @@ -99,6 +99,9 @@
> <maven.compiler.target>1.8</maven.compiler.target>
> <surefire.version>2.21.0</surefire.version>
>
> + <!-- for the default name of the module -->
> + <tomee.build.name>${project.groupId}.${project.artifactId}</
> tomee.build.name>
> +
> <!-- To easily change the javaee api version -->
> <version.javaee-api>8.0</version.javaee-api>
>
> @@ -455,8 +458,11 @@
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-jar-plugin</artifactId>
> <configuration>
> - <archive>
> + <archive combine.children="append">
>
>
> <manifestFile>${project.build.outputDirectory}/META-INF/MANIFEST.MF</manifestFile>
> + <manifestEntries>
> + <Automatic-Module-Name>${tomee.build.name
> }</Automatic-Module-Name>
> + </manifestEntries>
> </archive>
> </configuration>
> </plugin>
>
>
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/server/openejb-client/pom.xml
> ----------------------------------------------------------------------
> diff --git a/server/openejb-client/pom.xml b/server/openejb-client/pom.xml
> index c40d60c..5618ec4 100644
> --- a/server/openejb-client/pom.xml
> +++ b/server/openejb-client/pom.xml
> @@ -80,10 +80,13 @@
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-jar-plugin</artifactId>
> <configuration>
> - <archive>
> + <archive combine.children="append">
> <manifest>
> <mainClass>org.apache.openejb.client.Main</mainClass>
> </manifest>
> + <manifestEntries>
> + <Automatic-Module-Name>${tomee.build.name
> }</Automatic-Module-Name>
> + </manifestEntries>
> </archive>
> </configuration>
> </plugin>
>
>
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/apache-tomee/pom.xml
> ----------------------------------------------------------------------
> diff --git a/tomee/apache-tomee/pom.xml b/tomee/apache-tomee/pom.xml
> index 574c562..d02a785 100644
> --- a/tomee/apache-tomee/pom.xml
> +++ b/tomee/apache-tomee/pom.xml
> @@ -140,6 +140,11 @@
> <include>**/RemoteTomEEEJBContainer*.class</include>
>
> <include>**/META-INF/services/javax.ejb.spi.EJBContainerProvider</include>
> </includes>
> + <archive combine.children="append">
> + <manifestEntries>
> + <Automatic-Module-Name>${tomee.build.name
> }</Automatic-Module-Name>
> + </manifestEntries>
> + </archive>
> </configuration>
> </plugin>
> <plugin>
>
>
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/tomee-embedded/pom.xml
> ----------------------------------------------------------------------
> diff --git a/tomee/tomee-embedded/pom.xml b/tomee/tomee-embedded/pom.xml
> index 659fa1c..bcb6c72 100644
> --- a/tomee/tomee-embedded/pom.xml
> +++ b/tomee/tomee-embedded/pom.xml
> @@ -44,11 +44,14 @@
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-jar-plugin</artifactId>
> <configuration>
> - <archive>
> + <archive combine.children="append">
> <manifest>
> <Main-Class>org.apache.tomee.embedded.Main</Main-Class>
> <addClasspath>true</addClasspath>
> </manifest>
> + <manifestEntries>
> + <Automatic-Module-Name>${tomee.build.name
> }</Automatic-Module-Name>
> + </manifestEntries>
> </archive>
> </configuration>
> </plugin>
>
>
> http://git-wip-us.apache.org/repos/asf/tomee/blob/12720725/tomee/tomee-webapp/pom.xml
> ----------------------------------------------------------------------
> diff --git a/tomee/tomee-webapp/pom.xml b/tomee/tomee-webapp/pom.xml
> index c538123..9ba180e 100644
> --- a/tomee/tomee-webapp/pom.xml
> +++ b/tomee/tomee-webapp/pom.xml
> @@ -76,6 +76,11 @@
> <include>META-INF/**/*</include>
> <include>juli.properties</include>
> </includes>
> + <archive combine.children="append">
> + <manifestEntries>
> + <Automatic-Module-Name>${tomee.build.name
> }</Automatic-Module-Name>
> + </manifestEntries>
> + </archive>
> </configuration>
> </plugin>
> <plugin>
>