+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 >
