-1 (binding)

Issues:
* You've included 444 class files in the source tarball. All of them should
be deleted.
* You also included 7 jars that need to be removed.
* The license file includes a second section for Apache license. Anything
that is Apache licensed is covered by the first section.
* The .project, .settings/*, and .classpath files don't have Apache
licenses at the top.
* The
file 
agents-audit/src/main/java/com/xasecure/audit/provider/MultiDestAuditProvider.java
should move the license up in the file.
* security-admin/3rd_party_party_licenses_text.txt should be folded into
the LICENSE.txt
* The file security-admin/src/bin/install.properties doesn't have a license
header.
* The
file security-admin/src/main/resources/META-INF/infinispan-cache-config.xml
has a JBoss license header
* The file security-admin/src/main/webapp/.htacces doesn't have a license
header.
* The file security-admin/src/main/webapp/copyrightPluginList.html doesn't
have a license header
* The files under security-admin/src/main/webapp/libs/bower/* has a
DocumentCloud header that needs to be removed and replaced with an Apache
header. (Update the LICENSES.txt as necessary.) Many files under this
subdirectory need to have their headers replaced.
* The files under security-admin/src/main/webapp/libs/other* have bad
headers.
* The files under security-admin/src/main/webapp/META-INF/contextXML/* need
Apache headers.
* You need to remove
the security-admin/src/main/webapp/images/hortonworks*.png files.
* Why are we releasing keystores and certs? That seems really problematic.
We need to have scripts to build those keystores and certs and not the
binary artifacts.
* Remove the linux executable unixauthnative/native/credValidator
* You need a header on unixauthnative/target/antrun/build-main.xml
* You need a header on unixauthservice/conf.dist/jaas.conf
* You need to remove the windows
executable winpkg/src/resources/servicehost.exe
* You need a header on winpkg/src/template/bin/xasecure-hadoop-env.cmd
* (as Alan said) Move top level of tarball into a ranger-0.4.0 directory

Issues for next release:
* Repackage out of com.xasecure to org.apache.ranger


On Thu, Nov 6, 2014 at 9:32 AM, Ashish Singh <[email protected]> wrote:

> +1
> Verified md5 and build.
>
> Regards
> Ashish
>
> > On Nov 6, 2014, at 9:03 AM, Rmani <[email protected]> wrote:
> >
> > +1
> >
> > Regards
> > Ramesh
> >
> >
> >> On Nov 4, 2014, at 2:47 PM, Selvamohan Neethiraj <[email protected]>
> wrote:
> >>
> >> Hi:
> >>
> >> Based on feedback from dev group here, I have cancelled vote from
> Apache IPMC for the Apache Ranger 0.4.0 with ranger-0.4.0-rc1.
> >>
> >> After updating the License text information in LICENSE.txt and
> NOTICE.txt files, I've created a ranger-0.4.0-rc2 release candidate with
> the following artifacts up for a vote within dev community:
> >>
> >> Git for release:
> https://git-wip-us.apache.org/repos/asf?p=incubator-argus.git;a=shortlog;h=refs/tags/ranger-0.4.0-rc2
> >>
> >> Source release:
> http://people.apache.org/~sneethir/ranger/ranger-0.4.0-rc2/ranger-0.4.0-rc2.tar.z
> >> For the source release file (ranger-0.4.0-rc2.tar.z):
> >>    PGP Signature:
> http://people.apache.org/~sneethir/ranger/ranger-0.4.0-rc2/ranger-0.4.0-rc2.tar.z.asc
> >>    MD5/SHA Hash:
> http://people.apache.org/~sneethir/ranger/ranger-0.4.0-rc2/ranger-0.4.0-rc2.tar.z.mds
> >>
> >> Keys to verify the signature of the release artifact are available at:
> https://people.apache.org/keys/group/argus.asc
> >>
> >> The vote will be open for at least 72 hours.
> >> [ ] +1  approve
> >> [ ] +0  no opinion
> >> [ ] -1  disapprove (and reason why)
> >>
> >>
> >> Thanks
> >> Selva-
> >
> > --
> > CONFIDENTIALITY NOTICE
> > NOTICE: This message is intended for the use of the individual or entity
> to
> > which it is addressed and may contain information that is confidential,
> > privileged and exempt from disclosure under applicable law. If the reader
> > of this message is not the intended recipient, you are hereby notified
> that
> > any printing, copying, dissemination, distribution, disclosure or
> > forwarding of this communication is strictly prohibited. If you have
> > received this communication in error, please contact the sender
> immediately
> > and delete it from your system. Thank You.
>
> --
> CONFIDENTIALITY NOTICE
> NOTICE: This message is intended for the use of the individual or entity to
> which it is addressed and may contain information that is confidential,
> privileged and exempt from disclosure under applicable law. If the reader
> of this message is not the intended recipient, you are hereby notified that
> any printing, copying, dissemination, distribution, disclosure or
> forwarding of this communication is strictly prohibited. If you have
> received this communication in error, please contact the sender immediately
> and delete it from your system. Thank You.
>

Reply via email to