[resteasy-dev] Remove top jaxrs dir in upstream

Tomaž Cerar tomaz.cerar at gmail.com
Tue May 24 17:56:26 EDT 2016 

Alessio,

I've added extra commit on top of commits in your PR
that fixes all maven warnings about inconsistent/problematic pom.xml
structure.
you can cherry pick it from my branch
https://github.com/resteasy/Resteasy/compare/master...ctomc:small-build-fix
and include it in your PR.

Or I can send PR once yours is merged.

--
tomaz




On Tue, May 24, 2016 at 11:16 PM, Tomaž Cerar <tomaz.cerar at gmail.com> wrote:

> well, i cannot reproduce you failure as i get failure earlier in the build
> (on windows 10 / JDK8u74
>
> but it does look like they follow the same pattern, with the address
> already in use.
> they all come from TJWS web server and its testsuite.
>
> I think this should be merged, than we can all chip in to fix the failures
> we find on our platforms.
>
> As my experience from the past is that testsuite never passed for me
> anyway.
> it cannot be worse.
>
> --
> tomaz
>
>
> On Tue, May 24, 2016 at 10:32 PM, Rebecca Searls <rsearls at redhat.com>
> wrote:
>
>>
>> Fedora 23
>> JDK 1.7.0_55
>>
>>
>>
>>
>> ----- Original Message -----
>> > From: "Tomaž Cerar" <tomaz.cerar at gmail.com>
>> > To: "Rebecca Searls" <rsearls at redhat.com>
>> > Cc: "Alessio Soldano" <asoldano at redhat.com>,
>> resteasy-dev at lists.jboss.org
>> > Sent: Tuesday, May 24, 2016 4:27:23 PM
>> > Subject: Re: [resteasy-dev] Remove top jaxrs dir in upstream
>> >
>> > what platform / jdk are you on?
>> >
>> > On Tue, May 24, 2016 at 10:14 PM, Rebecca Searls <rsearls at redhat.com>
>> wrote:
>> >
>> > >
>> > > I checked there is no old running instance.
>> > > Usually when that happens I get an error about a port already under
>> use.
>> > >
>> > > I found the missing class very odd as well.  I've done nothing but
>> > > checkout the code and run it.
>> > > I haven't touched anything.
>> > >
>> > >
>> > > ----- Original Message -----
>> > > > From: "Alessio Soldano" <asoldano at redhat.com>
>> > > > To: "Rebecca Searls" <rsearls at redhat.com>
>> > > > Cc: resteasy-dev at lists.jboss.org
>> > > > Sent: Tuesday, May 24, 2016 4:09:21 PM
>> > > > Subject: Re: [resteasy-dev] Remove top jaxrs dir in upstream
>> > > >
>> > > > mmh... strange... perhaps you have (forgotten) a wfly instance
>> running
>> > > > on the same machine? Just wondering..
>> > > >
>> > > > Il 24/05/2016 22:05, Rebecca Searls ha scritto:
>> > > > > Yes, I'm using mvn clean install
>> > > > > mvn test -rf :RESTEASY-903-WF8      ; This did not get me past the
>> > > failure.
>> > > > >
>> > > > > Here is the cause of the deployment failure.
>> > > > > (I've attached the server.log for more detailed information)
>> > > > >
>> > > > > 2016-05-24 15:59:42,741 ERROR
>> > > > > [org.jboss.as.controller.management-operation]
>> > > (management-handler-thread
>> > > > > - 4) JBAS014613: Operation ("deploy") failed - address:
>> > > ([("deployment" =>
>> > > > > "RESTEASY-903.war")]) - failure description: {"JBAS014771:
>> Services
>> > > with
>> > > > > missing/unavailable dependencies" =>
>> > > > >
>> > >
>> ["jboss.naming.context.java.module.RESTEASY-903.RESTEASY-903.DefaultJMSConnectionFactory
>> > > > > is missing
>> > > > > [jboss.naming.context.java.jboss.DefaultJMSConnectionFactory]"]}
>> > > > >
>> > > > >
>> > > > >
>> > > > > ----- Original Message -----
>> > > > >> From: "Alessio Soldano" <asoldano at redhat.com>
>> > > > >> To: "Rebecca Searls" <rsearls at redhat.com>
>> > > > >> Cc: resteasy-dev at lists.jboss.org
>> > > > >> Sent: Tuesday, May 24, 2016 3:26:22 PM
>> > > > >> Subject: Re: [resteasy-dev] Remove top jaxrs dir in upstream
>> > > > >>
>> > > > >> It's definitely not expected, but I can't reproduce it here.
>> > > > >> Btw, Travis also successfully passed the tests below, even if it
>> later
>> > > > >> failed on a different one. I've restarted the build there.
>> > > > >> Which command are you using? 'mvn clean install' ? Are you able
>> to get
>> > > > >> through the failure by re-running the build (perhaps even using
>> -rf
>> > > > >> :RESTEASY-903-WF8 option) ?
>> > > > >>
>> > > > >> Cheers
>> > > > >> Alessio
>> > > > >>
>> > > > >> Il 24/05/2016 18:56, Rebecca Searls ha scritto:
>> > > > >>> The structure looks good.
>> > > > >>>
>> > > > >>> I get a build failure when the tests are run.  Is this expected.
>> > > > >>>
>> > > > >>> mvn clean install
>> > > > >>>
>> > > > >>>      [INFO] RESTEASY-903-WF8 ...................................
>> > > FAILURE
>> > > > >>>      [
>> > > > >>>      6.727 s]
>> > > > >>>        Tests in error:
>> > > > >>>           org.jboss.resteasy.test.resteasy903.UndertowTest:
>> Cannot
>> > > > >>>           deploy:
>> > > > >>>           RESTEASY-903.war
>> > > > >>>           org.jboss.resteasy.test.resteasy903.FilterTest: Cannot
>> > > deploy:
>> > > > >>>           RESTEASY-903.war
>> > > > >>>
>> > > > >>>
>> > > > >>>
>> > > > >>> ----- Original Message -----
>> > > > >>>> From: "Alessio Soldano" <asoldano at redhat.com>
>> > > > >>>> To: resteasy-dev at lists.jboss.org
>> > > > >>>> Sent: Tuesday, May 24, 2016 12:06:40 PM
>> > > > >>>> Subject: Re: [resteasy-dev] Remove top jaxrs dir in upstream
>> > > > >>>>
>> > > > >>>> The first PR for RESTEASY-1361 is being tested now (passed
>> locally):
>> > > > >>>> https://github.com/resteasy/Resteasy/pull/808
>> > > > >>>> It includes removal of the jaxrs dir and pruning of the
>> examples
>> > > (the
>> > > > >>>> jaxrs
>> > > > >>>> 2.0 samples are still there, tested by default, while other
>> examples
>> > > > >>>> have
>> > > > >>>> been moved to https://github.com/resteasy/resteasy-examples )
>> > > > >>>>
>> > > > >>>> Please have a look, I'd like to have this merged by tomorrow,
>> so
>> > > that we
>> > > > >>>> can
>> > > > >>>> go on with the build improvements.
>> > > > >>>>
>> > > > >>>> Cheers
>> > > > >>>> Alessio
>> > > > >>>>
>> > > > >>>> Il 21/05/2016 01:25, Ron Sigal ha scritto:
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> Got it. Thanks, guys.
>> > > > >>>>
>> > > > >>>> On 05/20/2016 03:36 PM, Alessio Soldano wrote:
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> Il 20/05/2016 19:35, Weinan Li ha scritto:
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> - Weinan Li
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> On May 21, 2016, at 1:24 AM, Ron Sigal <rsigal at redhat.com>
>> wrote:
>> > > > >>>>
>> > > > >>>> Two questions.
>> > > > >>>>
>> > > > >>>> 1. jaxrs will continue to exist on the 3.0.x branch, right?
>> > > > >>>> Right(I guess so).
>> > > > >>>> Sure
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> 2. More generally, what exactly goes into branch 3.0.x? I
>> assume
>> > > just
>> > > > >>>> bug
>> > > > >>>> fixes. Are they just bug fixes that come out of EAP 7? Or do
>> all bug
>> > > > >>>> fixes
>> > > > >>>> have to be applied to both 3.0.x and master?
>> > > > >>>> IMHO we should fully focus on 3.1.x from now on, and next
>> release
>> > > goes
>> > > > >>>> into
>> > > > >>>> Wildfly will be 3.1.0.
>> > > > >>>> Exactly. We might decide to do another 3.0.x release if there's
>> > > really
>> > > > >>>> need to (for instance should we figure out 3.0.17.Final has a
>> > > critical
>> > > > >>>> issue deserving another community fix release). However the
>> focus
>> > > should
>> > > > >>>> now be on 3.1.x.
>> > > > >>>>
>> > > > >>>> Cheers
>> > > > >>>> Alessio
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> For 3.0.x, I guess we should treat it similar like 2.3.x, and
>> only
>> > > > >>>> maintain
>> > > > >>>> it for EAP one-offs and CVE fixes in redhat-resteasy repo.
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> -Ron
>> > > > >>>>
>> > > > >>>> On 05/20/2016 05:39 AM, Weinan Li wrote:
>> > > > >>>>
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> Hi Team,
>> > > > >>>>
>> > > > >>>> I'm planning to remove 'jaxrs' top dir next week, which I
>> guess will
>> > > > >>>> affect
>> > > > >>>> all the open PRs, wdyt?
>> > > > >>>>
>> > > > >>>> --
>> > > > >>>> Weinan Li / JBoss
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> _______________________________________________
>> > > > >>>> resteasy-dev mailing list resteasy-dev at lists.jboss.org
>> > > > >>>> https://lists.jboss.org/mailman/listinfo/resteasy-dev
>> > > > >>>> --
>> > > > >>>> My company's smarter than your company (unless you work for
>> Red Hat)
>> > > > >>>>
>> > > > >>>> _______________________________________________
>> > > > >>>> resteasy-dev mailing list resteasy-dev at lists.jboss.org
>> > > > >>>> https://lists.jboss.org/mailman/listinfo/resteasy-dev
>> > > > >>>> _______________________________________________
>> > > > >>>> resteasy-dev mailing list resteasy-dev at lists.jboss.org
>> > > > >>>> https://lists.jboss.org/mailman/listinfo/resteasy-dev
>> > > > >>>>
>> > > > >>>> --
>> > > > >>>> My company's smarter than your company (unless you work for
>> Red Hat)
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> _______________________________________________
>> > > > >>>> resteasy-dev mailing list resteasy-dev at lists.jboss.org
>> > > > >>>> https://lists.jboss.org/mailman/listinfo/resteasy-dev
>> > > > >>>>
>> > > > >>>>
>> > > > >>>> --
>> > > > >>>> Alessio Soldano
>> > > > >>>> Web Service Lead, JBoss
>> > > > >>>>
>> > > > >>>> _______________________________________________
>> > > > >>>> resteasy-dev mailing list
>> > > > >>>> resteasy-dev at lists.jboss.org
>> > > > >>>> https://lists.jboss.org/mailman/listinfo/resteasy-dev
>> > > > >>>>
>> > > > >>
>> > > > >> --
>> > > > >> Alessio Soldano
>> > > > >> Web Service Lead, JBoss
>> > > > >>
>> > > > >>
>> > > >
>> > > >
>> > > > --
>> > > > Alessio Soldano
>> > > > Web Service Lead, JBoss
>> > > >
>> > > >
>> > > _______________________________________________
>> > > resteasy-dev mailing list
>> > > resteasy-dev at lists.jboss.org
>> > > https://lists.jboss.org/mailman/listinfo/resteasy-dev
>> > >
>> >
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/resteasy-dev/attachments/20160524/030244f9/attachment-0001.html

More information about the resteasy-dev mailing list