This vote thread is cancelled Howard. If you are voting on RC2 youll need to reply to the new thread.
Robbie On Wed, 6 Mar 2019 at 10:19, Howard Gao <[email protected]> wrote: > > +1 > That native stuff deserves a separate place from Artemis. > > Howard > > On Sat, Mar 2, 2019 at 8:08 AM Clebert Suconic <[email protected]> > wrote: > > > gitbox is down (confrmed it on twitter @ > > https://status.apache.org/incidents/4mqjzt9ysd8w?u=w1f97xydgrxq (down > > at the time I wrote the email). > > > > > > So, I will redo it on monday. > > > > On Fri, Mar 1, 2019 at 5:30 PM Clebert Suconic > > <[email protected]> wrote: > > > > > > Yes. I got that. > > > > > > > > > I am giving the -1 on this case. The native stays untouched for a long > > time. I wanted those fixes in. > > > > > > On Fri, Mar 1, 2019 at 5:24 PM Robbie Gemmell <[email protected]> > > wrote: > > >> > > >> Well, hopefully I was clear that I did not vote and didnt think it > > >> needed a respin, but OK :) > > >> > > >> Robbie > > >> > > >> On Fri, 1 Mar 2019 at 22:08, Clebert Suconic <[email protected]> > > wrote: > > >> > > > >> > @Robbie: I will take your findings as -1: Let me explain! > > >> > > > >> > > > >> > I'm thinking a next release will only happen rarely.. long time from > > now... > > >> > > > >> > So, I will rather fix it now.. it's easy to re-spin now. > > >> > > > >> > On Fri, Mar 1, 2019 at 4:34 PM Robbie Gemmell < > > [email protected]> wrote: > > >> > > > > >> > > Looks good. > > >> > > > > >> > > They indeed dont affect the binary, and they arent blockers, but do > > >> > > note the actual release is the source archive. > > >> > > > > >> > > Robbie > > >> > > > > >> > > On Fri, 1 Mar 2019 at 20:15, Clebert Suconic < > > [email protected]> wrote: > > >> > > > > > >> > > > Thanks Robbie: They do not affect the binary at all, and are minor > > >> > > > issues on the source. So, I am fixing these on this PR: > > >> > > > > > >> > > > https://github.com/apache/activemq-artemis-native/pull/2 > > >> > > > > > >> > > > (please if you could review it.. if I missed anything you will > > >> > > > probably be able to spot it on the PR.) > > >> > > > > > >> > > > On Fri, Mar 1, 2019 at 1:29 PM Robbie Gemmell < > > [email protected]> wrote: > > >> > > > > > > >> > > > > On Fri, 1 Mar 2019 at 17:25, Clebert Suconic < > > [email protected]> wrote: > > >> > > > > > > > >> > > > > > I would like to propose an Apache ActiveMQ Artemis Native > > 1.0.0 release. > > >> > > > > > > > >> > > > > > This is a sub component of ActiveMQ Artemis Native, > > >> > > > > > > > >> > > > > > Source distribution can be found here: > > >> > > > > > > > https://dist.apache.org/repos/dist/dev/activemq/activemq-artemis-native/1.0.0/ > > >> > > > > > > > >> > > > > > Maven repository is here: > > >> > > > > > > > https://repository.apache.org/content/repositories/orgapacheactivemq-1178 > > >> > > > > > > > >> > > > > > > > >> > > > > > The source tag: > > >> > > > > > > > https://gitbox.apache.org/repos/asf?p=activemq-artemis-native.git;a=tag;h=refs/tags/1.0.0 > > >> > > > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > Notice this is a sub component of ActiveMQ Artemis, and the > > release > > >> > > > > > notes will be part of the main component. And also, the > > binary here is > > >> > > > > > distributed through maven. > > >> > > > > > > > >> > > > > > > > >> > > > > > [ ] +1 approve the release as Apache Artemis 2.4.0 > > >> > > > > > [ ] +0 no opinion > > >> > > > > > [ ] -1 disapprove (and reason why) > > >> > > > > > > > >> > > > > > > > >> > > > > > Here's my +1 > > >> > > > > > > > >> > > > > > > > >> > > > > > -- > > >> > > > > > Clebert Suconic > > >> > > > > > > >> > > > > Not a vote as I cant try it out at the moment, though its not > > binding > > >> > > > > anyway. Just some comments from giving it a quick skim around > > trivial > > >> > > > > issues that should be fixed for the release if someone finds a > > >> > > > > blocking issue: > > >> > > > > > > >> > > > > - LICENSE contains some incorrect detail, though for a file that > > >> > > > > doesnt actually exist in the release. > > >> > > > > - NOTICE should have the contained name updated to more fully > > match the release. > > >> > > > > - The README ordering is a bit off. It initially lists a bunch > > of > > >> > > > > dependencies for you to install, then tells you how to build > > with > > >> > > > > Docker that presumably means you dont need them installed (but > > do need > > >> > > > > Docker, which isnt listed), and then after that goes back to > > giving > > >> > > > > manual build instructions where you would need them installed. > > >> > > > > - The example in the README sets JAVA_HOME to a Java7 install > > but 8 is required. > > >> > > > > > > >> > > > > Robbie > > >> > > > > > >> > > > > > >> > > > > > >> > > > -- > > >> > > > Clebert Suconic > > >> > > > >> > > > >> > > > >> > -- > > >> > Clebert Suconic > > > > > > -- > > > Clebert Suconic > > > > > > > > -- > > Clebert Suconic > >
