+1, verified signature, build works fine Minor observations 1. Sentry has Readme.md file, Would be easy if README or README.txt is provided
2. sentry-1.2.0-incubating.tar.gz.sha<http://people.apache.org/~shreepadma/sentry-1.2.0/sentry-1.2.0-incubating.tar.gz.sha> should be sentry-1.2.0-incubating.tar.gz.sha<http://people.apache.org/~shreepadma/sentry-1.2.0/sentry-1.2.0-incubating.tar.gz.sha>1? Think signature is sha1 3. Why do we need to use -Pdownload-hadoop option? A lot of project like Crunch use Hadoop but don't ask for options. This download increases the execution time a lot. The test take around an hour to run for me 4. Would be good to add source or src in release name like sentry-1.2.0-incubating-source.tar.gz<http://people.apache.org/~shreepadma/sentry-1.2.0/sentry-1.2.0-incubating.tar.gz.sha> so that it makes it easy to identify that it's a source release. On Thu, Sep 19, 2013 at 7:01 AM, Shreepadma Venugopalan < [email protected]> wrote: > This vote is being concurrently called on > [email protected] > [email protected]. > > This is the first incubator release of Apache Sentry, version > 1.2.0-incubating. > > It fixes the following issues: goo.gl/Bx5D1x > > Source files : http://people.apache.org/~shreepadma/sentry-1.2.0/ > > Tag to be voted on (rc0): > > https://git-wip-us.apache.org/repos/asf/incubator-sentry/repo?p=incubator-sentry.git;a=log;h=refs/tags/release-1.2.0-rc0 > > Sentry's KEYS containing the PGP key we used to sign the release: > https://people.apache.org/keys/group/sentry.asc > > Note that this is a source only release and we are voting on the source > (tag). > > Vote will be open for 72 hours. > > [ ] +1 approve > [ ] +0 no opinion > [ ] -1 disapprove (and reason why) > > Thanks. > > Shreepadma > -- thanks ashish Blog: http://www.ashishpaliwal.com/blog My Photo Galleries: http://www.pbase.com/ashishpaliwal
