I added a: deleteWithRetry(file: paths.findTomEELibJar('jakarta.activation-api'))
to the SetupCommand.groovy. Turns out that works better than: deleteWithRetry(file: paths.findOpenEJBWebJar('jakarta.activation-api')) I'm quite tired :) Jon On Fri, Sep 13, 2019 at 12:47 AM David Blevins <dblev...@tomitribe.com> wrote: > Do tell the mystical secret that lead to the exclusions aligning for our > binaries most divine. > > On Thu, Sep 12, 2019 at 4:31 PM Jonathan Gallimore < > jonathan.gallim...@gmail.com> wrote: > > > Ok, got further. Hopefully now just update the license file, and then > roll > > the release. > > > > Jon > > > > On Fri, Sep 13, 2019 at 12:23 AM Jonathan Gallimore < > > jonathan.gallim...@gmail.com> wrote: > > > > > Tried that, right next to where swizzle-stream is removed - get this - > it > > > didn't work. Also tried the installer - same. I fear my sanity is > leaving > > > me... > > > > > > Jon > > > > > > On Fri, Sep 13, 2019 at 12:19 AM David Blevins <dblev...@tomitribe.com > > > > > wrote: > > > > > >> Somewhere in here: > > >> > > >> - > > >> > > >> > > > https://github.com/apache/tomee/blob/master/tomee/apache-tomee/src/main/groovy/commands/SetupCommand.groovy#L58 > > >> > > >> Hello from SFO baggage claim in route to CodeOne! :) > > >> > > >> On Thu, Sep 12, 2019 at 4:00 PM David Blevins <dblev...@tomitribe.com > > > > >> wrote: > > >> > > >> > If you find it too difficult a plan b could be to hack the Setup (or > > >> > whatever it’s called) java code that turns the war files into the > > >> assembly. > > >> > > > >> > I.e. add a line to delete it. > > >> > > > >> > On Thu, Sep 12, 2019 at 3:46 PM Jonathan Gallimore < > > >> > jonathan.gallim...@gmail.com> wrote: > > >> > > > >> >> If anyone knows how to exclude jakarta.activation-api in the new > bom > > >> >> module, I'd sure appreciate the help. I'd expect > > >> >> > > >> >> <exclusion> <!-- Is already included in java-ee-api --> > > >> >> <artifactId>jakarta.activation</artifactId> > > >> >> <groupId>jakarta.activation-api</groupId> > > >> >> </exclusion> > > >> >> > > >> >> to work, but it seems like no matter where I exclude it, it keeps > > >> popping > > >> >> up. > > >> >> > > >> >> Jon > > >> >> > > >> >> On Thu, Sep 12, 2019 at 10:03 PM Jonathan Gallimore < > > >> >> jonathan.gallim...@gmail.com> wrote: > > >> >> > > >> >> > Awesome, thanks for the detail. I'm working on the PR now. I post > > >> >> updates > > >> >> > here as I have them. Intention is to have take 2 rolled by the > end > > of > > >> >> the > > >> >> > evening. > > >> >> > > > >> >> > Have a safe flight! > > >> >> > > > >> >> > Jon > > >> >> > > > >> >> > On Thu, Sep 12, 2019 at 9:55 PM David Blevins < > > >> dblev...@tomitribe.com> > > >> >> > wrote: > > >> >> > > > >> >> >> Plane, but planet works too — although a slight exaggeration :) > > >> >> >> > > >> >> >> On Thu, Sep 12, 2019 at 1:53 PM David Blevins < > > >> dblev...@tomitribe.com> > > >> >> >> wrote: > > >> >> >> > > >> >> >> > Had a longer response but the planet started boarding and now > I > > >> have > > >> >> to > > >> >> >> > start again on cell :) > > >> >> >> > > > >> >> >> > Agree with all your removals. Note on Jakarta activation 1.2, > > >> we’ll > > >> >> >> want > > >> >> >> > to eventually switch to Geronimo Activation 1.2 when we get > that > > >> >> ready. > > >> >> >> > > > >> >> >> > We need to update the LICENSE and NOTICE files of each dust to > > add > > >> >> the > > >> >> >> EPL > > >> >> >> > v2. When I looked at this a few days ago, I also noticed only > > >> some > > >> >> of > > >> >> >> them > > >> >> >> > had the EPL v1, so that should get fixed as well; all our > dists > > >> have > > >> >> the > > >> >> >> > ecj jar so need the EPL v1. > > >> >> >> > > > >> >> >> > > > >> >> >> > > > >> >> >> > On Thu, Sep 12, 2019 at 1:38 PM Jonathan Gallimore < > > >> >> >> > jonathan.gallim...@gmail.com> wrote: > > >> >> >> > > > >> >> >> >> Here's the same analysis on TomEE 7.0.6: > > >> >> >> >> > > >> >> >> >> javax.activation: > > >> >> >> >> - lib/activation-1.1.jar > > >> >> >> >> - lib/javaee-api-7.0-1.jar > > >> >> >> >> > > >> >> >> >> javax.xml.stream: > > >> >> >> >> - lib/stax-api-1.0-2.jar > > >> >> >> >> - lib/javaee-api-7.0-1.jar > > >> >> >> >> > > >> >> >> >> javax/xml/ws/EndpointReference.class, > > >> >> >> javax/xml/ws/WebServiceFeature.class > > >> >> >> >> and javax/xml/ws/wsaddressing/W3CEndpointReference.class: > > >> >> >> >> - lib/openejb-client-7.0.6.jar > > >> >> >> >> - lib/javaee-api-7.0-1.jar > > >> >> >> >> > > >> >> >> >> So I think removing the following is the way to go: > > >> >> >> >> > > >> >> >> >> geronimo-activation_1.1_spec-1.1.jar (covered by > > >> >> >> >> jakarta.activation-1.2.1.jar) > > >> >> >> >> jakarta.activation-api-1.2.1.jar (covered by > > >> >> >> >> jakarta.activation-1.2.1.jar) > > >> >> >> >> geronimo-interceptor_1.2_spec-1.0.jar (covered by > > >> >> javaee-api-8.0-2.jar) > > >> >> >> >> geronimo-javamail_1.4_spec-1.7.1.jar (covered by > > >> >> >> >> geronimo-javamail_1.4_mail-1.9.0-alpha-2.jar) > > >> >> >> >> geronimo-jpa_2.2_spec-1.0.jar (covered by > javaee-api-8.0-2.jar) > > >> >> >> >> jakarta.xml.soap-api-1.4.1.jar (covered by > > javaee-api-8.0-2.jar) > > >> >> >> >> > > >> >> >> >> I'll get started, but please do shout if that looks wrong. > > >> >> >> >> > > >> >> >> >> Jon > > >> >> >> >> > > >> >> >> >> On Thu, Sep 12, 2019 at 9:19 PM David Blevins < > > >> >> david.blev...@gmail.com > > >> >> >> > > > >> >> >> >> wrote: > > >> >> >> >> > > >> >> >> >> > > On Sep 12, 2019, at 1:03 PM, Jonathan Gallimore < > > >> >> >> >> > jonathan.gallim...@gmail.com> wrote: > > >> >> >> >> > > > > >> >> >> >> > > Also, I'll take a look at openejb-client, but I'm not too > > >> sure > > >> >> >> what do > > >> >> >> >> > with > > >> >> >> >> > > this conflict: > > >> >> >> >> > > > > >> >> >> >> > > javax/xml/ws/EndpointReference.class, > > >> >> >> >> > javax/xml/ws/WebServiceFeature.class > > >> >> >> >> > > and javax/xml/ws/wsaddressing/W3CEndpointReference.class: > > >> >> >> >> > > > > >> >> >> >> > > - javaee-api-8.0-2.jar > > >> >> >> >> > > - openejb-client-8.0.0.jar > > >> >> >> >> > > > > >> >> >> >> > > Any thoughts? > > >> >> >> >> > > > >> >> >> >> > Only suspecting it might have already been there in a past > > >> >> release. > > >> >> >> >> > > > >> >> >> >> > If that's the case, IMO, any conflicts that were there in > > 7.x, > > >> we > > >> >> >> ignore > > >> >> >> >> > for now. So if this one is status quo, it's fine. > > >> >> >> >> > > > >> >> >> >> > > > >> >> >> >> > -David > > >> >> >> >> > > > >> >> >> >> > > > >> >> >> >> > > >> >> >> > -- > > >> >> >> > Sent from Gmail Mobile > > >> >> >> > > > >> >> >> -- > > >> >> >> Sent from Gmail Mobile > > >> >> >> > > >> >> > > > >> >> > > >> > -- > > >> > Sent from Gmail Mobile > > >> > > > >> -- > > >> Sent from Gmail Mobile > > >> > > > > > > -- > Sent from Gmail Mobile >