Re: [VOTE] Bookkeeper 4.2.3 release candidate 0 (CANCELLING)

[Ivan Kelly <iv...@apache.org>]

I'm cancelling the rc since there is only one vote, and the legal
people don't seem to be responding. I'm just going to remove the
hedwig c++ client.

-Ivan


On Fri, Jun 13, 2014 at 11:51:29PM +0100, Flavio Junqueira wrote:
> Sure, it does say that autoconf is ok, but those m4 files are not exactly autoconf, right? I'm not exactly sure what they are, but I think that when they say autoconf, they probably mean scripts autoconf generated.
> 
> I have started a thread with legal to see if we can get a clarification. Please feel free to add to the discussion if I haven't characterized it properly.
> 
> -Flavio
> 
> On 12 Jun 2014, at 14:20, Ivan Kelly <iv...@apache.org> wrote:
> 
> > In the link you posts, it basically says autotools stuff is ok.
> > Also, all the m4 files except gtest.m4 have there other bsd style license
> > license. gtest.m4 comes from gtest, which is new bsd, so perhaps this
> > should also include in the file, though in source file the license
> > wasn't explicitly there.
> > 
> > In any case, I think we should remove the hedwig cpp client from the
> > source, and possibly all of hedwig, because it's not being used by
> > anyone and it's not being maintained. This was something I hoped to
> > discuss after 4.2.3 was out though.
> > 
> > -Ivan
> > 
> > On Thu, Jun 12, 2014 at 01:40:23PM +0100, Flavio Junqueira wrote:
> >> At this point, I'd rather proceed with the release candidate if there is no problem. I'm basically trying to determine if this is ok or not according to ASF. What you say makes sense, but I'd like to have some reference to point to.
> >> 
> >> The command I used was this one: 
> >> 
> >> java -jar ../apache-rat-0.10/apache-rat-0.10.jar .
> >> 
> >> -Flavio
> >> 
> >> On 12 Jun 2014, at 13:30, Ivan Kelly <iv...@apache.org> wrote:
> >> 
> >>> I mean what command did you run? It should have been
> >>> mvn apache-rat:check
> >>> 
> >>> These files have always been excluded from rat. 2 are auto generated,
> >>> one is the changes.txt which noone adds a license to. The others are
> >>> for the hedwig cpp configure script. These can't have a new license
> >>> added, as we didn't write them. Shipping the m4 files with the script
> >>> is the standard way to make a autotools project configurable.
> >>> 
> >>> If you want to remove them, we need to remove the whole cpp client,
> >>> which I have no problem with, as it's pretty much unmaintained.
> >>> 
> >>> -Ivan
> >>> 
> >>> On Thu, Jun 12, 2014 at 01:15:17PM +0100, Flavio Junqueira wrote:
> >>>> Directly from the command line, I haven't done it through maven, but what
> >>>> matters to me is that I'd like to know if we have a story for those files.
> >>>> In this page:
> >>>> 
> >>>> http://www.apache.org/dev/release.html#which-files-contain-license
> >>>> 
> >>>> It says that every file needs a license. This other page:
> >>>> 
> >>>> http://www.apache.org/legal/resolved.html
> >>>> 
> >>>> Talks about non-apache licenses, but I didn't really see anything that
> >>>> applies to us. I'm basically trying to determine if it is ok to exclude
> >>>> those and based on what arguments. I just don't want us to get in trouble as
> >>>> other projects that haven't been releasing artifacts properly.
> >>>> 
> >>>> -Flavio
> >>>> 
> >>>> -----Original Message-----
> >>>> From: Ivan Kelly [mailto:ivank@apache.org] 
> >>>> Sent: Thursday, June 12, 2014 1:00 PM
> >>>> To: bookkeeper-dev@zookeeper.apache.org
> >>>> Subject: Re: [VOTE] Bookkeeper 4.2.3 release candidate 0
> >>>> 
> >>>> This are all explicitly excluded in the pom. How did you run rat?
> >>>> 
> >>>> -Ivan
> >>>> 
> >>>> On Thu, Jun 12, 2014 at 11:20:05AM +0100, Flavio Junqueira wrote:
> >>>>> I ran the rat tool and the one thing that called my attention was this:
> >>>>> 
> >>>>> Unapproved licenses:
> >>>>> 
> >>>>> ./CHANGES.txt
> >>>>> 
> >>>> ./bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/DataFormats.ja
> >>>> va
> >>>>> ./hedwig-client/src/main/cpp/aminclude.am
> >>>>> ./hedwig-client/src/main/cpp/m4/ax_boost_asio.m4
> >>>>> ./hedwig-client/src/main/cpp/m4/ax_boost_base.m4
> >>>>> ./hedwig-client/src/main/cpp/m4/ax_boost_thread.m4
> >>>>> ./hedwig-client/src/main/cpp/m4/ax_doxygen.m4
> >>>>> ./hedwig-client/src/main/cpp/m4/gtest.m4
> >>>>> 
> >>>> ./hedwig-protocol/src/main/java/org/apache/hedwig/protocol/PubSubProtocol.ja
> >>>> va
> >>>>> ./hedwig-server/src/main/resources/p12.pass
> >>>>> 
> >>>>> I'm just wondering if we have a justification for these.
> >>>>> 
> >>>>> -Flavio
> >>>>> 
> >>>>> On 10 Jun 2014, at 16:36, Ivan Kelly <iv...@apache.org> wrote:
> >>>>> 
> >>>>>> This is the first release candidate for Apache Bookkeeper, version 
> >>>>>> 4.2.3.
> >>>>>> 
> >>>>>> This is a bugfix release for 4.2.2.
> >>>>>> 
> >>>>>> Notable fixes and improvements include new utilities to give 
> >>>>>> administrators better visibility of cluster state (BOOKKEEPER-746), 
> >>>>>> improvements to allow for smoother rolling upgrades 
> >>>>>> (BOOKKEEPER-745), fixes to ledger polling to ensure metadata updates 
> >>>>>> aren't missed
> >>>>>> (BOOKKEEPER-710 & BOOKKEEPER-747) and shading of protobuf libraries 
> >>>>>> to avoid conflicts when included with other version (BOOKKEEPER-708).
> >>>>>> 
> >>>>>> The full release notes is available at:
> >>>>>> 
> >>>>>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12325
> >>>>>> 569&styleName=Html&projectId=12311293
> >>>>>> 
> >>>>>> *** Please download, test and vote by June 16th 2014, 10:00 UTC+0. 
> >>>>>> ***
> >>>>>> 
> >>>>>> Note that we are voting upon the source (tag), binaries are provided 
> >>>>>> for convenience.
> >>>>>> 
> >>>>>> Source and binary files:
> >>>>>> http://people.apache.org/~ivank/bookkeeper-4.2.3-candidate-0/
> >>>>>> 
> >>>>>> Maven staging repo:
> >>>>>> https://repository.apache.org/content/repositories/orgapachebookkeep
> >>>>>> er-1000/
> >>>>>> 
> >>>>>> The tag to be voted upon:
> >>>>>> https://svn.apache.org/repos/asf/zookeeper/bookkeeper/tags/release-4
> >>>>>> .2.3
> >>>>>> 
> >>>>>> Bookkeeper's KEYS file containing PGP keys we use to sign the release:
> >>>>>> http://svn.apache.org/repos/asf/zookeeper/bookkeeper/dist/KEYS
> >>>>>> 
> >>>>>> Please download the the source package, and follow the README to 
> >>>>>> build and run a bookkeeper and hedwig service.
> >>>>> 
> >>>> 
> >> 
>