Here's my -1. Please run RAT on the source. There are multiple source files missing source license headers. For example:
framework/modules/geronimo-security/src/main/java/org/apache/geronimo/security/realm/providers/GenericHttpHeaderLdapLoginModule.java I can help with this, if needed. Signature/checksum looked good. I have not, yet, run a build. I also haven't run any tests (but assume others have checked this, already). --kevan On Nov 20, 2010, at 7:29 AM, Rex Wang wrote: > Hi Devs, > > A release candidate for Geronimo 2.1.7 has been created and staged. > > The tag has been created here: > > https://svn.apache.org/repos/asf/geronimo/server/tags/geronimo-2.1.7/ > > The staging repo is here: > > https://repository.apache.org/content/repositories/orgapachegeronimo-015/ > > The main artifacts up for vote are the source release archives: > > https://repository.apache.org/content/repositories/orgapachegeronimo-015/org/apache/geronimo/geronimo/2.1.7/geronimo-2.1.7-source-release.tar.gz > https://repository.apache.org/content/repositories/orgapachegeronimo-015/org/apache/geronimo/geronimo/2.1.7/geronimo-2.1.7-source-release.zip > > > We have to verify the binaries pass the tck, so the vote may be open for > longer than the 72-hour minimum. > > [ ] +1 about time to push this out the door > [ ] 0 no opinion > [ ] -1 not this one (please explain why) > > -- > Lei Wang (Rex) > rwonly AT apache.org
