I definitely agree, this is also a -1 for me. Glad that we caught it early on. I will cut a new RC later today.
Thanks everyone for your participation so far. Gregor On Tue, Jul 25, 2017 at 2:06 PM, Zoran Regvart <zo...@regvart.com> wrote: > Hi Cameleers, > I agree this would be quite unexpected for the users, changing my vote to -1 > > zoran > > On Tue, Jul 25, 2017 at 1:57 PM, Claus Ibsen <claus.ib...@gmail.com> wrote: > >> Hi >> >> I have now pushed a fix to 2.19.x and master branch for that servlet >> dependency problem. >> >> I also tested that with the camel in action 2 source code which also >> triggered this issue and failed an unit test/example. >> Now it works again. >> >> In light of this problem with this RC and that we have a fix for it >> committed I am voting -1 >> >> On Tue, Jul 25, 2017 at 12:48 PM, Andrea Cosentino >> <ancosen1...@yahoo.com> wrote: >> > +1 for a new cut. >> > >> > -- >> > Andrea Cosentino >> > ---------------------------------- >> > Apache Camel PMC Member >> > Apache Karaf Committer >> > Apache Servicemix PMC Member >> > Email: ancosen1...@yahoo.com >> > Twitter: @oscerd2 >> > Github: oscerd >> > >> > >> > >> > On Tuesday, July 25, 2017, 12:38:56 PM GMT+2, Claus Ibsen >> > <claus.ib...@gmail.com> wrote: >> > >> > >> > On Tue, Jul 25, 2017 at 12:34 PM, Andrea Cosentino >> > <ancosen1...@yahoo.com.invalid> wrote: >> >> Do we need to cut a new release? >> >> >> > >> > Frankly I would prefer that as otherwise an upgrade is not as simple >> > as changing 2.19.1 to 2.19.2 for end users. >> > >> > The HTTP components are commonly used components in Camel, its not a >> > little component that a few users uses. >> > >> > >> >> -- >> >> Andrea Cosentino >> >> ---------------------------------- >> >> Apache Camel PMC Member >> >> Apache Karaf Committer >> >> Apache Servicemix PMC Member >> >> Email: ancosen1...@yahoo.com >> >> Twitter: @oscerd2 >> >> Github: oscerd >> >> >> >> >> >> >> >> On Tuesday, July 25, 2017, 12:06:51 PM GMT+2, Claus Ibsen >> >> <claus.ib...@gmail.com> wrote: >> >> >> >> >> >> On Tue, Jul 25, 2017 at 11:37 AM, Gary Brown <gbr...@redhat.com> wrote: >> >>> Found the following issue: >> >>> https://issues.apache.org/jira/browse/CAMEL-11591 >> >>> >> >> >> >> Yeah same problem for camel-restlet which I just pushed a fix for. >> >> >> >> We need to add that servlet api for the various http component that >> >> uses it, jetty, http, http4, restlet, etc. Basically all the ones >> >> except for camel-servlet which is intended to run in existing servlet >> >> container. >> >> >> >> >> >> >> >> >> >>> This is a change from 2.19.1 to 2.19.2. >> >>> >> >>> Regards >> >>> Gary >> >>> >> >>> >> >>> On Tue, Jul 25, 2017 at 9:40 AM, Zoran Regvart <zo...@regvart.com> >> wrote: >> >>> >> >>>> +1 (binding) >> >>>> >> >>>> Tested with Salesforce/Slack example and run the Salesforce >> integration >> >>>> tests, >> >>>> >> >>>> zoran >> >>>> >> >>>> On Mon, Jul 24, 2017 at 10:15 PM, Gregor Zurowski >> >>>> <gre...@list.zurowski.org> wrote: >> >>>> > Hi Everyone: >> >>>> > >> >>>> > This is a vote to release Apache Camel 2.19.2, a new patch release >> >>>> > that includes 41 fixes and improvements. >> >>>> > >> >>>> > Release notes: https://issues.apache.org/ >> jira/secure/ReleaseNote.jspa? >> >>>> version=12340945&projectId=12311211 >> >>>> > >> >>>> > Staging repository: >> >>>> > >> >>>> > https://repository.apache.org/content/repositories/ >> orgapachecamel-1082/ >> >>>> > >> >>>> > Tarballs: https://repository.apache.org/content/repositories/ >> >>>> orgapachecamel-1082/org/apache/camel/apache-camel/2.19.2/ >> >>>> > >> >>>> > Tag: https://git-wip-us.apache.org/repos/asf?p=camel.git;a= >> commit;h= >> >>>> 076f409b82651a39edd7aac7b8736f70f05428a6 >> >>>> > >> >>>> > Please test this release candidate and cast your vote. >> >>>> > [ ] +1 Release the binary as Apache Camel 2.19.2 >> >>>> > [ ] -1 Veto the release (provide specific comments) >> >>>> > >> >>>> > The vote is open for at least 72 hours. >> >>>> > >> >>>> > Thanks, >> >>>> > Gregor >> >>>> >> >>>> >> >>>> >> >>>> -- >> >>>> Zoran Regvart >> >> >> >>>> >> >> >> >> >> >> >> >> -- >> >> Claus Ibsen >> >> ----------------- >> >> http://davsclaus.com @davsclaus >> >> Camel in Action 2: https://www.manning.com/ibsen2 >> > >> > >> > >> > >> > -- >> > Claus Ibsen >> > ----------------- >> > http://davsclaus.com @davsclaus >> > Camel in Action 2: https://www.manning.com/ibsen2 >> >> >> >> -- >> Claus Ibsen >> ----------------- >> http://davsclaus.com @davsclaus >> Camel in Action 2: https://www.manning.com/ibsen2 >> > > > > -- > Zoran Regvart
