Rick, Source, signature, checksum, build -- all look good. Here's my +1. One note, your source release is:
https://repository.apache.org/content/repositories/orgapachegeronimo-008/org/apache/geronimo/javamail/geronimo-javamail_1.4/1.8.2/geronimo-javamail_1.4-1.8.2-source-release.tar.gz or https://repository.apache.org/content/repositories/orgapachegeronimo-008/org/apache/geronimo/javamail/geronimo-javamail_1.4/1.8.2/geronimo-javamail_1.4-1.8.2-source-release.zip Your note refers to the source jars that are, IMO, not appropriate as release artifacts. The above artifact (tar.gz) is what I've reviewed and am voting on. --kevan On Sep 10, 2010, at 8:04 AM, Rick McGuire wrote: > This is a bug fix release to fix a problem with the javamail smtp provider > including the BCC recipients as a header in sent messages. The JIRA in > question is > > https://issues.apache.org/jira/browse/GERONIMO-5587 > > This is a vote for the javamail providers and combined mail jar. This fix > did not require an update to the javamail specs. . > > The 2 components up for vote are: > > org/apache/geronimo/javamail/geronimo-javamail_1.4_mail/1.8.2/geronimo-javamail_1.4_mail-1.8.2-sources.jar > org/apache/geronimo/javamail/geronimo-javamail_1.4_provider/1.8.2/geronimo-javamail_1.4_provider-1.8.2-sources.jar > > > from the staging repository at: > > https://repository.apache.org/content/repositories/orgapachegeronimo-008/ > > The source repo is: > > https://svn.apache.org/repos/asf/geronimo/javamail/tags/geronimo-javamail_1.4-1.8.2/ > > Vote will be open for 72 hours. > > [ ] +1 approve > [ ] +0 no opinion > [ ] -1 disapprove (and reason why) > > > Rick >
