Hi John, Thanks for the review of the release candidate.
- NOTICE file for Guice: The NOTICE_BINARY file under tephra-distribution, contains the notice for Guice. Please let us how we can fix it and we will create a JIRA to fix this issue. We had followed the example of including libraries that has Apache License 2.0 in NOTICE, similar to the Fineract project. - LGPL License: The Release Candidate - 2 doesn't include the LGPL license at all. Do you see LGPL license included anywhere (this might have been present in the Release Candidate - 1 which was fixed in Release Candidate - 2)? Thanks, Gokul On Mon, Sep 18, 2017 at 5:08 PM, John D. Ament <johndam...@apache.org> wrote: > Hi, > > I'm inclined to vote -1 unless someone convinces me otherwise. > > TEPHRA-231 was created to address issues identified, however it doesn't > seem to have been solved properly. > > - The changes to the NOTICE file don't match the NOTICE file for Guice. > There is no need to declare the license usage for single licensed code. > - You added the LGPL license. You cannot include LGPL binaries. However, > I can't find the binaries so it should probably just be omitted. > > Other things look fine. > > John > > On Mon, Sep 18, 2017 at 1:13 PM Andreas Neumann <a...@apache.org> wrote: > > > Hi all, > > > > This is a call for a vote on releasing Apache Tephra 0.13.0-incubating, > > release candidate 2. This is the sixth release of Tephra. > > > > The Apache Tephra community has voted and approved the release. > > > > Vote thread: > > http://mail-archives.apache.org/mod_mbox/incubator-tephra-de > v/201709.mbox/% > > 3CCANpmDOfN7MPHYjKt-PqNd3yY1KtgyNn6L1ANT63xxPEdSsXQuA%40mail.gmail.com > %3E > > <http://mail-archives.apache.org/mod_mbox/incubator-tephra-d > ev/201709.mbox/%3CCANpmDOfN7MPHYjKt-PqNd3yY1KtgyNn6L1ANT63xx > PEdSsXQuA%40mail.gmail.com%3E> > > > > Result thread: > > http://mail-archives.apache.org/mod_mbox/incubator-tephra-de > v/201709.mbox/% > > 3CCANpmDOe8T70zU4h%2BgknSJ%2B%3DTOAkUw9praf46GTtuz-5Cr2-7jA% > > 40mail.gmail.com%3E > > > > The source tarball, including signatures, digests, etc. can be found at: > > https://dist.apache.org/repos/dist/dev/incubator/tephra/0.13 > > .0-incubating-rc2/src > > > > The tag to be voted upon is v0.13.0-incubating: > > https://git-wip-us.apache.org/repos/asf?p=incubator-tephra.g > > it;a=shortlog;h=refs/tags/v0.13.0-incubating > > > > The release hash is 37467ce63934aca86f84eb61493733bb470450d7: > > https://git-wip-us.apache.org/repos/asf?p=incubator-tephra.g > > it;a=commit;h=37467ce63934aca86f84eb61493733bb470450d7 > > > > The Nexus Staging URL: > > https://repository.apache.org/content/repositories/orgapachetephra-1010 > > > > Release artifacts are signed with the following key: > > http://people.apache.org/keys/committer/anew > > > > KEYS file available: > > https://dist.apache.org/repos/dist/dev/incubator/tephra/KEYS > > > > For information about the contents of this release, see: > > https://dist.apache.org/repos/dist/dev/incubator/tephra/0.13 > > .0-incubating-rc2/CHANGES.txt > > > > Please vote on releasing this package as Apache Tephra 0.13.0-incubating > > > > The vote will be open for 72 hours. > > > > [ ] +1 Release this package as Apache Tephra 0.13.0-incubating > > [ ] +0 no opinion > > [ ] -1 Do not release this package because ... > > > > Thanks, > > Andreas > > >
