(Adding mailing lists for wider audience.) I still think you want a *plugin-level* dep... but if you want a *feature-level *one too, then here's another PR:
https://git.eclipse.org/r/123951 Note that I'm using *equivalent* to allow 1.2.x, rather than *perfect*... which might in fact break if a different version of 1.2.0 appears in Orbit compared to the version used to build. (1.2.0.vyyyymmddsome-date != 1.2.0.vyyyymmddlater-date). Equivalent seems safer unless you know for sure that 1.2.1 is incompatible with 1.2.0. If you like it, and PMC+2 it, you can merge it and I'll respin RC3 [1] for you. If you don't like it, and want to apply your own fix, I'll make sure that it's rebuilt and re-aggregated too. Just make sure you do whatever PMC approval process you feel is necessary in the BZ [2] so that the rest of the PMC don't complain/revert/quote wiki-scripture at you. :D [1] http://download.eclipse.org/webtools/downloads/drops/R3.10.0/S-3.10.0.RC3-20180601164520/repository/ [2] https://bugs.eclipse.org/bugs/show_bug.cgi?id=265772 On Mon, Jun 4, 2018 at 3:18 PM, Keith Chong <kch...@ca.ibm.com> wrote: > Unfortunately, it's more of a dependency at runtime. In > org.eclipse.jst.ws.axis.consumption.ui, there is a command that copies > the saaj.jar from the lib folder of the 1.2 version of javax.xml.soap into > the Web project. The 1.3 version does not have it. > > The feature is not the place to fix this? The change should be (instead of > using greaterOrEqual): > > <import plugin="javax.xml.soap" version="1.2.0" match="perfect"/> > > Not sure why the CXF plugin needs to be changed since this is Axis1 > related. > > Regards, > Keith Chong > WTP Web Services > > > > [image: Inactive hide details for Nick Boldt ---2018/06/04 03:02:04 > PM---I've documented the JavaEE Perspective issue here: https://url]Nick > Boldt ---2018/06/04 03:02:04 PM---I've documented the JavaEE Perspective > issue here: INVALID URI REMOVED > > From: Nick Boldt <nbo...@redhat.com> > To: Keith Chong <kch...@ca.ibm.com> > Cc: Elson Yuen <ey...@ca.ibm.com>, Mat Booth <mat.bo...@redhat.com>, > Shane Clarke <shane.cla...@outlook.com> > Date: 2018/06/04 03:02 PM > > Subject: Re: gerrit review for JAXWS bugzilla > ------------------------------ > > > > I've documented the JavaEE Perspective issue here: > > *https://wiki.eclipse.org/New_Help_for_Old_Friends_X* > <https://wiki.eclipse.org/New_Help_for_Old_Friends_X> (the old doc was > empty so instead of creating an XI version, I just repurposed the X > version. Now they align with the WTP version, in that 3.10 ==> 10 ==> X. > > ---- > > Regarding the version of *javax.xml.soap*, I'd prefer if the dependency > was expressed in a MANIFEST file so we know exactly which plugin requires > it, and which version. > > Why is 1.2 required, and why 1.3 is not compatible? > > Note that while the WTP site contains javax.xml.soap *1.3.0*.v201105210645, > the Photon site contains javax.xml.soap *1.2.0*.v201005080501 so both are > available to be installed. > > When I installed 20180531-0841_eclipse-jee-photon-RC2-linux.gtk.x86_64.tar.gz > then installed into that all of WTP from the RC2 site [1], I got > javax.xml.soap 1.2.0 installed. > > [1] > *http://download.eclipse.org/webtools/downloads/drops/R3.10.0/S-3.10.0.RC2-20180524070951/repository/* > <http://download.eclipse.org/webtools/downloads/drops/R3.10.0/S-3.10.0.RC2-20180524070951/repository/> > > When I try to update to WTP 3.10.0.RC3 [2], or install from scratch using > [2], I get javax.xml.soap *1.3.0*.v201105210645 installed. > > [2] > *http://download.eclipse.org/webtools/downloads/drops/R3.10.0/S-3.10.0.RC3-20180601164520/repository/* > <http://download.eclipse.org/webtools/downloads/drops/R3.10.0/S-3.10.0.RC3-20180601164520/repository/> > > So... here's a PR to restrict the version of javax.xml.soap needed in > org.eclipse.jst.ws.jaxws.core: > > *https://git.eclipse.org/r/#/c/123947/* > <https://git.eclipse.org/r/#/c/123947/> > > > > > On Mon, Jun 4, 2018 at 2:25 PM, Keith Chong <*kch...@ca.ibm.com* > <kch...@ca.ibm.com>> wrote: > > Yes, it should be added. > > I hit a regression with the Axis web services wizard. It fails with an > NPE. > > I noticed we are now pulling in version 1.3.0 of javax.xml.soap. The > original had a hard dependency on 1.2: > > <import plugin="javax.xml.soap" version="1.2.0" > match="greaterOrEqual"/> > > Could we get this changed back to 1.2 in webservices/webtools.webservices/ > core/features/org.eclipse.wst.ws_core.feature/*feature.xml* > > Regards, > Keith Chong > WTP Web Services > > > > [image: Inactive hide details for Nick Boldt ---2018/06/04 01:07:54 > PM---No prob. Should this be added to a New Help For Old Friends pa]Nick > Boldt ---2018/06/04 01:07:54 PM---No prob. Should this be added to a New > Help For Old Friends page? Seems not immediately intuitive. > > From: Nick Boldt <*nbo...@redhat.com* <nbo...@redhat.com>> > To: Keith Chong <*kch...@ca.ibm.com* <kch...@ca.ibm.com>> > Cc: Elson Yuen <*ey...@ca.ibm.com* <ey...@ca.ibm.com>>, Mat Booth < > *mat.bo...@redhat.com* <mat.bo...@redhat.com>>, Shane Clarke < > *shane.cla...@outlook.com* <shane.cla...@outlook.com>> > Date: 2018/06/04 01:07 PM > > > Subject: Re: gerrit review for JAXWS bugzilla > ------------------------------ > > > > No prob. Should this be added to a New Help For Old Friends page? > Seems not immediately intuitive. > > *https://wiki.eclipse.org/New_Help_for_Old_Friends_X* > <https://wiki.eclipse.org/New_Help_for_Old_Friends_X> > *https://wiki.eclipse.org/New_Help_for_Old_Friends_XI* > <https://wiki.eclipse.org/New_Help_for_Old_Friends_XI> > > On Mon, Jun 4, 2018 at 12:56 PM, Keith Chong <*kch...@ca.ibm.com* > <kch...@ca.ibm.com>> wrote: > Hey Nick, thanks. Still trying to catch up on the e-mails - That > resolved it. > > Regards, > Keith Chong > WTP Web Services > > > [image: Inactive hide details for Nick Boldt ---2018/06/04 > 12:33:30 PM---This problem (or similar) was reported last week for > the RC2 > s]Nick Boldt ---2018/06/04 12:33:30 PM---This problem (or > similar) was reported last week for the RC2 smoke test too, so it's > hopefully not r > > From: Nick Boldt <*nbo...@redhat.com* <nbo...@redhat.com>> > To: Keith Chong <*kch...@ca.ibm.com* <kch...@ca.ibm.com>> > Cc: Mat Booth <*mat.bo...@redhat.com* <mat.bo...@redhat.com>>, > Shane Clarke <*shane.cla...@outlook.com* > <shane.cla...@outlook.com>>, Elson Yuen <*ey...@ca.ibm.com* > <ey...@ca.ibm.com>> > Date: 2018/06/04 12:33 PM > > > Subject: Re: gerrit review for JAXWS bugzilla > > ------------------------------ > > > > This problem (or similar) was reported last week for the RC2 > smoke test too, so it's hopefully not related. > > From Ian Trimble: > > I saw some related (I think) weirdness. > > When I tried to open the “Java EE” perspective, it was > initially not present. I had to select the “Show all” checkbox below > the > list (I’ve never even noticed this checkbox before, let alone > selected it.) > At this point, the “Java EE” perspective was shown in the list. I > selected > it and clicked “Open”, which caused a “Confirm Enablement” dialog to > display. I confirmed enablement of “Java EE Development”, and from > there > was able to complete my smoke test without further issue. > > -- > *https://dev.eclipse.org/mhonarc/lists/wtp-releng/msg10982.html* > <https://dev.eclipse.org/mhonarc/lists/wtp-releng/msg10982.html> > > Cheers, > > Nick > > On Mon, Jun 4, 2018 at 12:24 PM, Keith Chong <*kch...@ca.ibm.com* > <kch...@ca.ibm.com>> wrote: > In the RC3 Smoke Test driver, the WSDL Editor doesn't appear. > Looks like the most, if not all, core and JAX-WS web > services plugins are > only in the Starting state, and not loaded. For example: > > *org.eclipse.wst.ws* <http://org.eclipse.wst.ws> (1.1.400.v201805181041) > "Web Services" [Starting] > org.eclipse.wst.wsdl (1.2.500.v201805181041) "WSDL > Model" [Starting] > org.eclipse.wst.wsdl.ui (1.2.801.v201805181041) > "WSDL UI" [Starting] > > Not sure if it is because of this change but if it > is, is it too late to back it out? > *https://bugs.eclipse.org/bugs/show_bug.cgi?id=265772* > <https://bugs.eclipse.org/bugs/show_bug.cgi?id=265772> > > Regards, > Keith Chong > WTP Web Services > > > [image: Inactive hide details for Keith > Chong---2018/06/04 11:49:25 AM---I added a comment to > https://bugs.eclipse.org/bugs/show_bug.cg]Keith > Chong---2018/06/04 11:49:25 AM---I added a comment to > *https://bugs.eclipse.org/bugs/show_bug.cgi?id=265772Mat* > > <https://bugs.eclipse.org/bugs/show_bug.cgi?id=265772Mat>, > your change to org.ecl > > From: Keith Chong/Toronto/IBM > To: Mat Booth <*mat.bo...@redhat.com* > <mat.bo...@redhat.com>> > Cc: Nick Boldt <*nbo...@redhat.com* > <nbo...@redhat.com>>, Shane Clarke < > *shane.cla...@outlook.com* <shane.cla...@outlook.com>>, > Elson Yuen/Toronto/IBM@IBMCA > Date: 2018/06/04 11:49 AM > Subject: Re: gerrit review for JAXWS bugzilla > ------------------------------ > > > I added a comment to > > *https://bugs.eclipse.org/bugs/show_bug.cgi?id=265772* > <https://bugs.eclipse.org/bugs/show_bug.cgi?id=265772> > > Mat, your change to org.eclipse.wst.wsdl (the WTP > EMF-based WSDL model) only implements the newer versions > of the interfaces? > It looks like you are just implementing the new methods. > So I think > adopters of the WTP Photon WSDL model _should_ be ok. > > The other concern is that if WTP only 'ships' 1.6.2, > then are we forcing adopters to move their 1.5.1-based > plugins to 1.6.2? > > FWIW, I know that some of the new changes in 1.6.x > include a new package and classes that supports SOAP 1.2 > and there is a new > WSDLElement interface that makes your code cleaner (at a > more abstract > level). So, if you don't use these new features, > technically, your > 1.5.2-based plugin will still compile cleanly against > 1.6.2. > > > Regards, > Keith Chong > WTP Web Services > > > > > [image: Inactive hide details for Mat Booth > ---2018/06/04 11:23:54 AM---Aha, thanks. I suppose we'll > need to move the > Fedora RPM away f]Mat Booth ---2018/06/04 11:23:54 > AM---Aha, thanks. I suppose we'll need to move the > Fedora RPM away from the > monolithic build for next rel > > From: Mat Booth <*mat.bo...@redhat.com* > <mat.bo...@redhat.com>> > To: Nick Boldt <*nbo...@redhat.com* > <nbo...@redhat.com>> > Cc: Mat Booth <*mat.bo...@redhat.com* > <mat.bo...@redhat.com>>, Shane Clarke < > *shane.cla...@outlook.com* <shane.cla...@outlook.com>>, > Keith Chong <*kch...@ca.ibm.com* <kch...@ca.ibm.com>> > Date: 2018/06/04 11:23 AM > Subject: Re: gerrit review for JAXWS bugzilla > ------------------------------ > > > > > Aha, thanks. I suppose we'll need to move the Fedora > RPM away from the monolithic build for next release then > :-) > > > > On 4 June 2018 at 16:17, Nick Boldt < > *nbo...@redhat.com* <nbo...@redhat.com>> wrote: > I could but we don't need to any more -- that > doesn't drive the builds. > > I suppose in the interests > of good housekeeping I should do > so all the way through this release, then > stop for >3.10.0. > > 4 commits pulled from > submodules and committed to > webtools.releng.aggregator project: > > * d8c87e4 - (HEAD -> master, > origin/master) > webtools.webservices :: pull latest commits(s) from > origin/master up to > a2d0ac9a89106c161fe219d4bc0a00eed0e9b7e3 > *nbo...@redhat.com* > <nbo...@redhat.com> (2 > minutes ago) > * cd6f94f - > webtools.sourceediting :: pull > latest commits(s) from origin/master up to > > 0c532000ad09e53c9fb8ddfb1c3f1def983f57c0 > *nbo...@redhat.com* > <nbo...@redhat.com> (2 > minutes ago) > * 9c8f9a2 - > webtools.servertools :: pull > latest commits(s) from origin/master up to > > 9033b8ab53498bb7c701031994bcfd542bc02cc2 > *nbo...@redhat.com* > <nbo...@redhat.com> (2 > minutes ago) > * 39d14c0 - webtools.javaee > :: pull latest commits(s) from > origin/master up to > > 5462e7660c53030225f2152353c94967fd5ebfb8 > *nbo...@redhat.com* > <nbo...@redhat.com> (2 > minutes ago) > > > > On Mon, Jun 4, 2018 at 11:07 > AM, Mat Booth < > *mat.bo...@redhat.com* > <mat.bo...@redhat.com>> > wrote: > Thanks all. > > Nick, do you intend to > update the submodule references > in the aggregator repo? > > > On 31 May 2018 at 07:47, > Shane Clarke < > *shane.cla...@outlook.com* > <shane.cla...@outlook.com>> > wrote: > Hi Guys, > > Sorry > for > being late with getting to this. > > For > > *https://git.eclipse.org/r/#/c/122761/* > > <https://git.eclipse.org/r/#/c/122761/> if > we can > get the PMC votes for RC3 i'll get it in. > > And > for > > *https://git.eclipse.org/r/#/c/122932/* > > <https://git.eclipse.org/r/#/c/122932/> i > think > Keith should sign off on it if possible. > > > Thanks, > Shane > > > > > ------------------------------ > > * From:* Nick Boldt <*nbo...@redhat.com* <nbo...@redhat.com>> > * Sent:* Wednesday 30 May 2018 14:18 > * To:* Mat Booth > * Cc:* Shane Clarke; Keith Chong > * Subject:* Re: gerrit review for JAXWS bugzilla > > I've > merged > the fix for parent pom and rebuilt it. > > > > *https://hudson.eclipse.org/webtools/job/webtools-webservices-gerrit_master/* > > <https://hudson.eclipse.org/webtools/job/webtools-webservices-gerrit_master/> > will > trigger once the current wtp common build is done (it's rebuilding > because > of the parent pom change). > > On > Wed, May > 30, 2018 at 8:19 AM, Mat Booth < > > *mat.bo...@redhat.com* > > <mat.bo...@redhat.com>> > wrote: > Another > update > -- My Orbit fix is now available in an I-build, so I submitted the > > following change for the wtp-parent to update its orbit repo reference: > > *https://git.eclipse.org/r/#/c/123635/* > > <https://git.eclipse.org/r/#/c/123635/> > > If > that is > merged, then jenkins should be able to do a successful build of > this > change: > > *https://git.eclipse.org/r/#/c/122932/* > > <https://git.eclipse.org/r/#/c/122932/> > > > On > 23 May > 2018 at 10:45, Mat Booth < > > *mat.bo...@redhat.com* > > <mat.bo...@redhat.com>> > wrote: > > Regarding > > this second change, my fix for Orbit was merged, and the new > wsil4j bundle > > is available from the latest Orbit nightly repo: > > > *http://download.eclipse.org/tools/orbit/N-builds/N20180522205425/* > > > <http://download.eclipse.org/tools/orbit/N-builds/N20180522205425/> > > > > On > > 19 May 2018 at 01:18, Nick Boldt < > > *nbo...@redhat.com* > > <nbo...@redhat.com>> > > wrote: > > There's > > also a second gerrit for review here [1] but that requires a > fix in Orbit > > [2] first. > > > > [1] > > *https://git.eclipse.org/r/#/c/122932/* > > <https://git.eclipse.org/r/#/c/122932/> > > > [2] > > *https://bugs.eclipse.org/bugs/show_bug.cgi?id=534859* > > <https://bugs.eclipse.org/bugs/show_bug.cgi?id=534859> > > > So... > > I guess that will require the post-RC1 PMC approval process. > > > On > > Fri, May 18, 2018 at 9:06 AM, Nick Boldt < > > *nbo...@redhat.com* > > <nbo...@redhat.com>> > > wrote: > > Please > > have a look and if you approve, merge > this: > > *https://git.eclipse.org/r/#/c/122761/* > > <https://git.eclipse.org/r/#/c/122761/> > > > > Bug > > 534747 - Use import-package for deps > that can be supplied by the jre > > > This > > makes WTP more compatible with other > platforms (like linux) where Eclipse > > can be installed via rpm. > > > Hopefully > > the Jenkins on which the gerrit is > building won't crap out like the ones > > the other day. Seems > > *ci.eclipse.org* > > <http://ci.eclipse.org> > > is a bit overloaded these days. Must > be RC season. :) > > > > Nick > > > -- > > > Nick > > Boldt > > Principal > > Software Engineer, RHCSA > > Productization > > Lead :: JBoss Tools & Dev Studio > > IM: > > @nickboldt / @nboldt / > > *http://nick.divbyzero.com* > > <http://nick.divbyzero.com> > > > <https://red.ht/sig> *TRIED. > > TESTED. TRUSTED.* > > <https://redhat.com/trusted>*@ > > @redhatnews* > > <https://twitter.com/redhatnews> > > * > > Red Hat* > > <https://www.facebook.com/RedHatInc> > > <https://www.facebook.com/RedHatInc> > > > > “The > > Only Thing That Is Constant Is Change” > - Heraclitus > > > > > -- > > > Nick > > Boldt > > Principal > > Software Engineer, RHCSA > > Productization > > Lead :: JBoss Tools & Dev Studio > > IM: > > @nickboldt / @nboldt / > > *http://nick.divbyzero.com* > > <http://nick.divbyzero.com> > > > <https://red.ht/sig> *TRIED. > > TESTED. TRUSTED.* > > <https://redhat.com/trusted>*@ > > @redhatnews* > > <https://twitter.com/redhatnews> > > * > > Red Hat* > > <https://www.facebook.com/RedHatInc> > > <https://www.facebook.com/RedHatInc> > > > > “The > > Only Thing That Is Constant Is Change” - Heraclitus > > > > -- > Nick > Boldt > Principal > Software > Engineer, RHCSA > > Productization > Lead :: > JBoss Tools & Dev Studio > IM: > > @nickboldt / @nboldt / > > *http://nick.divbyzero.com* > > <http://nick.divbyzero.com> > > > <https://red.ht/sig> *TRIED. > TESTED. > TRUSTED.* > > <https://redhat.com/trusted>*@ > > @redhatnews* > > <https://twitter.com/redhatnews> > * > Red > Hat* > > <https://www.facebook.com/RedHatInc> > > <https://www.facebook.com/RedHatInc> > > > “The > Only > Thing That Is Constant Is Change” - Heraclitus > > > > -- > Nick Boldt > Principal Software Engineer, > RHCSA > Productization Lead :: JBoss > Tools & Dev Studio > IM: @nickboldt / @nboldt / > *http://nick.divbyzero.com* > <http://nick.divbyzero.com> > <https://red.ht/sig> *TRIED. > TESTED. TRUSTED.* > <https://redhat.com/trusted>*@ > @redhatnews* > <https://twitter.com/redhatnews> > * Red Hat* > > <https://www.facebook.com/RedHatInc> > > <https://www.facebook.com/RedHatInc> > > > “The Only Thing That Is > Constant Is Change” - Heraclitus > > > > > > -- > Nick Boldt > Principal Software Engineer, RHCSA > Productization Lead :: JBoss Tools & Dev Studio > IM: @nickboldt / @nboldt / *http://nick.divbyzero.com* > <http://nick.divbyzero.com> > > <https://red.ht/sig> *TRIED. TESTED. TRUSTED.* > <https://redhat.com/trusted> > *@ @redhatnews* <https://twitter.com/redhatnews> * Red Hat* > <https://www.facebook.com/RedHatInc> > <https://www.facebook.com/RedHatInc> > > > “The Only Thing That Is Constant Is Change” - Heraclitus > > > > > -- > Nick Boldt > Principal Software Engineer, RHCSA > Productization Lead :: JBoss Tools & Dev Studio > IM: @nickboldt / @nboldt / *http://nick.divbyzero.com* > <http://nick.divbyzero.com> > > > > <https://red.ht/sig> *TRIED. TESTED. TRUSTED.* > <https://redhat.com/trusted> > > *@ @redhatnews* <https://twitter.com/redhatnews> * Red Hat* > <https://www.facebook.com/RedHatInc> > <https://www.facebook.com/RedHatInc> > > > “The Only Thing That Is Constant Is Change” - Heraclitus > > > > > > > -- > Nick Boldt > Principal Software Engineer, RHCSA > Productization Lead :: JBoss Tools & Dev Studio > IM: @nickboldt / @nboldt / *http://nick.divbyzero.com* > <http://nick.divbyzero.com> > > <https://red.ht/sig> *TRIED. TESTED. TRUSTED.* > <https://redhat.com/trusted>@ @redhatnews <https://twitter.com/redhatnews> > Red Hat <https://www.facebook.com/RedHatInc> > <https://www.facebook.com/RedHatInc> > > > “The Only Thing That Is Constant Is Change” - Heraclitus > > > -- Nick Boldt Principal Software Engineer, RHCSA Productization Lead :: JBoss Tools & Dev Studio IM: @nickboldt / @nboldt / http://nick.divbyzero.com <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> @ @redhatnews <https://twitter.com/redhatnews> Red Hat <https://www.facebook.com/RedHatInc> <https://www.facebook.com/RedHatInc> “The Only Thing That Is Constant Is Change” - Heraclitus
_______________________________________________ wtp-dev mailing list wtp-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/wtp-dev