A JDK8 build, JDK11 test config would require more work, but I'll try and give it a shot. At least we have the JDK8 build, which should help check if anything breaks.
Jon On Fri, Jul 26, 2019 at 11:26 AM Jean-Louis Monteiro < [email protected]> wrote: > Hey Jon, > > Thanks so much for the help and the update. Definitely helps. > On the Java 11 builder, we would need to keep building with JDK 8 and run > with Java 11. > > Currently, it's configured to also build with Java 11 which does not work > or would require more work. > > -- > Jean-Louis Monteiro > http://twitter.com/jlouismonteiro > http://www.tomitribe.com > > > On Fri, Jul 26, 2019 at 12:18 PM Jonathan Gallimore < > [email protected]> wrote: > > > Thanks for the update JL. I'm just wiring up builds for your branch now: > > > > c['builders'].append(tomee_trunk_win()) > > > c['builders'].append(slicker_than_butter_builder("tomee-trunk-ubuntu-jvm8", > > "master", "latest1.8", "-Xmx1536m")) > > > > > > > > > *c['builders'].append(slicker_than_butter_builder("tomee-java11-take2-ubuntu-jvm8", > > "java11-take2", "latest1.8", > > > > > "-Xmx1536m"))c['builders'].append(slicker_than_butter_builder("tomee-java11-take2-ubuntu-jvm11", > > "java11-take2", "latest11", > > > > > "-Xmx1536m"))*c['builders'].append(slicker_than_butter_builder("tomee-7.1.x-ubuntu-jvm8", > > "tomee-7.1.x", "latest1.8", "-Xmx1536m")) > > > c['builders'].append(slicker_than_butter_builder("tomee-7.0.x-ubuntu-jvm8", > > "tomee-7.0.x", "latest1.8", "-Xmx1536m")) > > c['builders'].append(slicker_than_butter_builder("tomee-7.0.x-ubuntu", > > "tomee-7.0.x", "latest1.7", "-Xmx1536m -XX:PermSize=128m > > -XX:MaxPermSize=768m -XX:ReservedCodeCacheSize=96m -Xss1024k")) > > c['builders'].append(slicker_than_butter_builder("tomee-1.7.x-ubuntu", > > "tomee-1.7.x", "latest1.7", "-Xmx1024m -XX:PermSize=96m > > -XX:MaxPermSize=384m -XX:ReservedCodeCacheSize=96m -Xss1024k")) > > > > Here's the JDK8 build of your branch: > > https://ci.apache.org/builders/tomee-java11-take2-ubuntu-jvm8 > > And JDK11: > https://ci.apache.org/builders/tomee-java11-take2-ubuntu-jvm11 > > > > I've kicked off builds for both. I have terribly connectivity at the > > moment, but I'll keep an eye out on how these get on. > > > > Jon > > > > On Fri, Jul 26, 2019 at 9:50 AM Jean-Louis Monteiro < > > [email protected]> wrote: > > > > > Hi guys, > > > > > > I think I have done good progress. Most of the build is passing with > Java > > > 11. > > > With Java 8, there should not be any backward compatibility issues as I > > > tested locally. > > > > > > I don't want to stack the PR so it becomes a huge PR and we are afraid > to > > > merge it. > > > Any issue if I merge the current one and get a new snapshot deployed? > > > > > > If something is broken with Java 8, I can either revert or fix very > quick > > > afterwards. > > > > > > Thoughts? > > > PR is https://github.com/apache/tomee/pull/523 > > > > > > I also would like to run Jakarta EE 8 TCK with both Java 8 and Java 11 > so > > > we get a view of were we are. > > > > > > I have a couple of tickets and tasks that I gonna file into JIRA today > > > > > > -- > > > Jean-Louis Monteiro > > > http://twitter.com/jlouismonteiro > > > http://www.tomitribe.com > > > > > > > > > On Tue, Jul 23, 2019 at 4:30 PM Jean-Louis Monteiro < > > > [email protected]> wrote: > > > > > > > I ended up throwing all the PR away and starting from scratch again. > > > > There were too many changes and I did not know anymore what they were > > > for. > > > > > > > > Went with a different approach to see if it works better. > > > > On master, we introduced some garbage by the way, not sure why but > > we'll > > > > have to fix some pom stuff because I suspect we have too much stuff > in > > > the > > > > binaries. > > > > -- > > > > Jean-Louis Monteiro > > > > http://twitter.com/jlouismonteiro > > > > http://www.tomitribe.com > > > > > > > > > > > > On Tue, Jul 23, 2019 at 3:28 PM Jean-Louis Monteiro < > > > > [email protected]> wrote: > > > > > > > >> Hey David, > > > >> > > > >> Thanks for the feedback. > > > >> I already removed some of them. It has some side effects that I > still > > > >> need to investigate. For instance, removing soap-api, makes the > > > >> UniverseTest to fail. The Geronimo service locator seems to mess up > > the > > > >> test. > > > >> > > > >> > > > >> > > > >> -- > > > >> Jean-Louis Monteiro > > > >> http://twitter.com/jlouismonteiro > > > >> http://www.tomitribe.com > > > >> > > > >> > > > >> On Tue, Jul 23, 2019 at 3:11 PM David Blevins < > > [email protected]> > > > >> wrote: > > > >> > > > >>> > On Jul 23, 2019, at 5:46 AM, Jonathan Gallimore < > > > >>> [email protected]> wrote: > > > >>> > > > > >>> > I did a build on master and on the branch. The "plus" artifact > for > > > the > > > >>> > branch has the following library diffs compared with master: > > > >>> > > > >>> Thanks! > > > >>> > > > >>> > +jacorb-3.9 > > > >>> > +jacorb-omgapi-3.9 > > > >>> > +jboss-rmi-api_1.0_spec-1.0.6.Final > > > >>> > > > >>> These should be cut out as we do not actually implement the > > CORBA/IIOP > > > >>> and CSIv2 code required by the full profile. > > > >>> > > > >>> > +jakartax.activation-api-1.2.1 > > > >>> > +javax.activation-1.2.0 > > > >>> > +javax.activation-api-1.2.0 > > > >>> > +javax.xml.soap-api-1.4.0 > > > >>> > +management-api-3.0.0-b012 > > > >>> > > > >>> The Geronimo versions of these should already be in our javaee-api > > jar. > > > >>> > > > >>> > > > >>> -David > > > >>> > > > >>> > > > > > >
