Re: [VOTE][email] Release Commons Email 1.3 based on RC2
Hi Sebastian, thanks for your help - I will look at commons-email over the weekend. Cheers, Siegfried Goeschl On 08.12.11 00:28, sebb wrote: On 7 December 2011 16:10, sebbseb...@gmail.com wrote: On 6 December 2011 15:38, Siegfried Goeschlsgoes...@gmx.at wrote: Hi Gary, ad RAT - I added a license header for mime.types but doing this for the mail messages is not possible AFAIK since there is no such thing as a comment for mail messages Did you try using # as the comment marker? Works for me. ad src zip - need to check since I did a last minute upgrade to the latest commons-parent.pom ad Clirr errors - this reported errors are stemming from two changes 1) I introduced an EmailConstants.java to collect all the constants and Email implements EmailConstants Not sure about that one, I'll check. It may be a false positive as far as binary compatibility is concerned, but it certainly breaks source compatibility and should be clearly documented in the release notes. Constants are OK to remove, because the Java compiler inlines them. However, mutable fields are not OK to remove unless they are guaranteed not to be used by user code. The protected fields ssl and tls have been removed from the Email class. 2) the setter methods of Email return now consistently this instead of void This definitely breaks binary compatibilty; we had a similar issue in Commons IO. The JVM includes the return type in the method signature when resolving references; changing the return type will cause method not found or similar. IMHO both changes do not break binary compatibility for the minor release Sorry, not true. == Also just discovered a test failure when compiling and testing on Win/XP: testResolvingFileLenient(org.apache.commons.mail.resolver.DataSourceFileResolverTest) Time elapsed: 0 sec FAILURE! junit.framework.AssertionFailedError: at junit.framework.Assert.fail(Assert.java:47) at junit.framework.Assert.assertTrue(Assert.java:20) at junit.framework.Assert.assertNull(Assert.java:230) at org.apache.commons.mail.resolver.DataSourceFileResolverTest.testResolvingFileLenient(DataSourceFileResolverTest.java:50) Appears to be due to an OS-specific bug in DataSource resolver. It checks if the resource /images/asf_logo_wide.gif is absolute. Unfortunately File(/images/asf_logo_wide.gif) is not absolute on Windows: On Microsoft Windows systems, a pathname is absolute if its prefix is a drive specifier followed by \\, or if its prefix is \\. Cheers, Siegfried Goeschl On 05.12.11 23:11, Gary Gregory wrote: Hi Siegfried: Thank you for preparing the RC. This /sounds/ worrisome from RAT: Unapproved licenses: src/resources/META-INF/mime.types src/test/eml/attachment-only.eml src/test/eml/html-attachment.eml src/test/eml/multipart-report.eml src/test/eml/simple-reply.eml src/test/eml/simple.eml Where is the 'src' zip I can build from? Looks like there are three easy to fix checkstyle errors (not the missing Javadocs.) -1: Can you justify the 51 Clirr errors in a minor release? Our current guideline is that such a change means a major version and new package. I do not care which way we go but we should be mindful of binary compatibility for minor release. Thank you, Gary On Mon, Dec 5, 2011 at 4:31 PM, Siegfried Goeschlsgoes...@gmx.at mailto:sgoes...@gmx.at wrote: Hi folks, I would like to call a vote to release commons-email-1.3 which contains the following bug fixes and improvements found here http://people.apache.org/__builds/commons/email/1.3/RC2/__site/changes-report.html http://people.apache.org/builds/commons/email/1.3/RC2/site/changes-report.html Tag: https://svn.apache.org/repos/__asf/commons/proper/email/tags/__EMAIL_1_3_RC2 https://svn.apache.org/repos/asf/commons/proper/email/tags/EMAIL_1_3_RC2 Site: http://people.apache.org/__builds/commons/email/1.3/RC2/__site/index.html http://people.apache.org/builds/commons/email/1.3/RC2/site/index.html Binaries: http://people.apache.org/__builds/commons/email/1.3/RC2/__staged/org/apache/commons/__commons-email/1.3/ http://people.apache.org/builds/commons/email/1.3/RC2/staged/org/apache/commons/commons-email/1.3/ [ ] +1 release it [ ] +0 go ahead I don't care [ ] -1 no, do not release it because Thanks in advance Siegfried Goeschl --__--__- To unsubscribe, e-mail: dev-unsubscribe@commons.__apache.org mailto:dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org mailto:dev-h...@commons.apache.org -- E-Mail: garydgreg...@gmail.commailto:garydgreg...@gmail.com | ggreg...@apache.orgmailto:ggreg...@apache.org JUnit in Action, 2nd Ed:http://goog_1249600977http://bit.ly/ECvg0 Spring Batch in Action:http://s.apache.org/HOqhttp://bit.ly/bqpbCK Blog:
Re: [VOTE][email] Release Commons Email 1.3 based on RC2
On 6 December 2011 15:38, Siegfried Goeschl sgoes...@gmx.at wrote: Hi Gary, ad RAT - I added a license header for mime.types but doing this for the mail messages is not possible AFAIK since there is no such thing as a comment for mail messages ad src zip - need to check since I did a last minute upgrade to the latest commons-parent.pom ad Clirr errors - this reported errors are stemming from two changes 1) I introduced an EmailConstants.java to collect all the constants and Email implements EmailConstants Not sure about that one, I'll check. It may be a false positive as far as binary compatibility is concerned, but it certainly breaks source compatibility and should be clearly documented in the release notes. 2) the setter methods of Email return now consistently this instead of void This definitely breaks binary compatibilty; we had a similar issue in Commons IO. The JVM includes the return type in the method signature when resolving references; changing the return type will cause method not found or similar. IMHO both changes do not break binary compatibility for the minor release Cheers, Siegfried Goeschl On 05.12.11 23:11, Gary Gregory wrote: Hi Siegfried: Thank you for preparing the RC. This /sounds/ worrisome from RAT: Unapproved licenses: src/resources/META-INF/mime.types src/test/eml/attachment-only.eml src/test/eml/html-attachment.eml src/test/eml/multipart-report.eml src/test/eml/simple-reply.eml src/test/eml/simple.eml Where is the 'src' zip I can build from? Looks like there are three easy to fix checkstyle errors (not the missing Javadocs.) -1: Can you justify the 51 Clirr errors in a minor release? Our current guideline is that such a change means a major version and new package. I do not care which way we go but we should be mindful of binary compatibility for minor release. Thank you, Gary On Mon, Dec 5, 2011 at 4:31 PM, Siegfried Goeschl sgoes...@gmx.at mailto:sgoes...@gmx.at wrote: Hi folks, I would like to call a vote to release commons-email-1.3 which contains the following bug fixes and improvements found here http://people.apache.org/__builds/commons/email/1.3/RC2/__site/changes-report.html http://people.apache.org/builds/commons/email/1.3/RC2/site/changes-report.html Tag: https://svn.apache.org/repos/__asf/commons/proper/email/tags/__EMAIL_1_3_RC2 https://svn.apache.org/repos/asf/commons/proper/email/tags/EMAIL_1_3_RC2 Site: http://people.apache.org/__builds/commons/email/1.3/RC2/__site/index.html http://people.apache.org/builds/commons/email/1.3/RC2/site/index.html Binaries: http://people.apache.org/__builds/commons/email/1.3/RC2/__staged/org/apache/commons/__commons-email/1.3/ http://people.apache.org/builds/commons/email/1.3/RC2/staged/org/apache/commons/commons-email/1.3/ [ ] +1 release it [ ] +0 go ahead I don't care [ ] -1 no, do not release it because Thanks in advance Siegfried Goeschl --__--__- To unsubscribe, e-mail: dev-unsubscribe@commons.__apache.org mailto:dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org mailto:dev-h...@commons.apache.org -- E-Mail: garydgreg...@gmail.com mailto:garydgreg...@gmail.com | ggreg...@apache.org mailto:ggreg...@apache.org JUnit in Action, 2nd Ed: http://goog_1249600977http://bit.ly/ECvg0 Spring Batch in Action: http://s.apache.org/HOqhttp://bit.ly/bqpbCK Blog: http://garygregory.wordpress.com http://garygregory.wordpress.com/ Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: [VOTE][email] Release Commons Email 1.3 based on RC2
On 7 December 2011 16:10, sebb seb...@gmail.com wrote: On 6 December 2011 15:38, Siegfried Goeschl sgoes...@gmx.at wrote: Hi Gary, ad RAT - I added a license header for mime.types but doing this for the mail messages is not possible AFAIK since there is no such thing as a comment for mail messages Did you try using # as the comment marker? Works for me. ad src zip - need to check since I did a last minute upgrade to the latest commons-parent.pom ad Clirr errors - this reported errors are stemming from two changes 1) I introduced an EmailConstants.java to collect all the constants and Email implements EmailConstants Not sure about that one, I'll check. It may be a false positive as far as binary compatibility is concerned, but it certainly breaks source compatibility and should be clearly documented in the release notes. Constants are OK to remove, because the Java compiler inlines them. However, mutable fields are not OK to remove unless they are guaranteed not to be used by user code. The protected fields ssl and tls have been removed from the Email class. 2) the setter methods of Email return now consistently this instead of void This definitely breaks binary compatibilty; we had a similar issue in Commons IO. The JVM includes the return type in the method signature when resolving references; changing the return type will cause method not found or similar. IMHO both changes do not break binary compatibility for the minor release Sorry, not true. == Also just discovered a test failure when compiling and testing on Win/XP: testResolvingFileLenient(org.apache.commons.mail.resolver.DataSourceFileResolverTest) Time elapsed: 0 sec FAILURE! junit.framework.AssertionFailedError: at junit.framework.Assert.fail(Assert.java:47) at junit.framework.Assert.assertTrue(Assert.java:20) at junit.framework.Assert.assertNull(Assert.java:230) at org.apache.commons.mail.resolver.DataSourceFileResolverTest.testResolvingFileLenient(DataSourceFileResolverTest.java:50) Appears to be due to an OS-specific bug in DataSource resolver. It checks if the resource /images/asf_logo_wide.gif is absolute. Unfortunately File(/images/asf_logo_wide.gif) is not absolute on Windows: On Microsoft Windows systems, a pathname is absolute if its prefix is a drive specifier followed by \\, or if its prefix is \\. Cheers, Siegfried Goeschl On 05.12.11 23:11, Gary Gregory wrote: Hi Siegfried: Thank you for preparing the RC. This /sounds/ worrisome from RAT: Unapproved licenses: src/resources/META-INF/mime.types src/test/eml/attachment-only.eml src/test/eml/html-attachment.eml src/test/eml/multipart-report.eml src/test/eml/simple-reply.eml src/test/eml/simple.eml Where is the 'src' zip I can build from? Looks like there are three easy to fix checkstyle errors (not the missing Javadocs.) -1: Can you justify the 51 Clirr errors in a minor release? Our current guideline is that such a change means a major version and new package. I do not care which way we go but we should be mindful of binary compatibility for minor release. Thank you, Gary On Mon, Dec 5, 2011 at 4:31 PM, Siegfried Goeschl sgoes...@gmx.at mailto:sgoes...@gmx.at wrote: Hi folks, I would like to call a vote to release commons-email-1.3 which contains the following bug fixes and improvements found here http://people.apache.org/__builds/commons/email/1.3/RC2/__site/changes-report.html http://people.apache.org/builds/commons/email/1.3/RC2/site/changes-report.html Tag: https://svn.apache.org/repos/__asf/commons/proper/email/tags/__EMAIL_1_3_RC2 https://svn.apache.org/repos/asf/commons/proper/email/tags/EMAIL_1_3_RC2 Site: http://people.apache.org/__builds/commons/email/1.3/RC2/__site/index.html http://people.apache.org/builds/commons/email/1.3/RC2/site/index.html Binaries: http://people.apache.org/__builds/commons/email/1.3/RC2/__staged/org/apache/commons/__commons-email/1.3/ http://people.apache.org/builds/commons/email/1.3/RC2/staged/org/apache/commons/commons-email/1.3/ [ ] +1 release it [ ] +0 go ahead I don't care [ ] -1 no, do not release it because Thanks in advance Siegfried Goeschl --__--__- To unsubscribe, e-mail: dev-unsubscribe@commons.__apache.org mailto:dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org mailto:dev-h...@commons.apache.org -- E-Mail: garydgreg...@gmail.com mailto:garydgreg...@gmail.com | ggreg...@apache.org mailto:ggreg...@apache.org JUnit in Action, 2nd Ed: http://goog_1249600977http://bit.ly/ECvg0 Spring Batch in Action: http://s.apache.org/HOqhttp://bit.ly/bqpbCK Blog: http://garygregory.wordpress.com http://garygregory.wordpress.com/ Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory
Re: [VOTE][email] Release Commons Email 1.3 based on RC2
Hi Gary, ad RAT - I added a license header for mime.types but doing this for the mail messages is not possible AFAIK since there is no such thing as a comment for mail messages ad src zip - need to check since I did a last minute upgrade to the latest commons-parent.pom ad Clirr errors - this reported errors are stemming from two changes 1) I introduced an EmailConstants.java to collect all the constants and Email implements EmailConstants 2) the setter methods of Email return now consistently this instead of void IMHO both changes do not break binary compatibility for the minor release Cheers, Siegfried Goeschl On 05.12.11 23:11, Gary Gregory wrote: Hi Siegfried: Thank you for preparing the RC. This /sounds/ worrisome from RAT: Unapproved licenses: src/resources/META-INF/mime.types src/test/eml/attachment-only.eml src/test/eml/html-attachment.eml src/test/eml/multipart-report.eml src/test/eml/simple-reply.eml src/test/eml/simple.eml Where is the 'src' zip I can build from? Looks like there are three easy to fix checkstyle errors (not the missing Javadocs.) -1: Can you justify the 51 Clirr errors in a minor release? Our current guideline is that such a change means a major version and new package. I do not care which way we go but we should be mindful of binary compatibility for minor release. Thank you, Gary On Mon, Dec 5, 2011 at 4:31 PM, Siegfried Goeschl sgoes...@gmx.at mailto:sgoes...@gmx.at wrote: Hi folks, I would like to call a vote to release commons-email-1.3 which contains the following bug fixes and improvements found here http://people.apache.org/__builds/commons/email/1.3/RC2/__site/changes-report.html http://people.apache.org/builds/commons/email/1.3/RC2/site/changes-report.html Tag: https://svn.apache.org/repos/__asf/commons/proper/email/tags/__EMAIL_1_3_RC2 https://svn.apache.org/repos/asf/commons/proper/email/tags/EMAIL_1_3_RC2 Site: http://people.apache.org/__builds/commons/email/1.3/RC2/__site/index.html http://people.apache.org/builds/commons/email/1.3/RC2/site/index.html Binaries: http://people.apache.org/__builds/commons/email/1.3/RC2/__staged/org/apache/commons/__commons-email/1.3/ http://people.apache.org/builds/commons/email/1.3/RC2/staged/org/apache/commons/commons-email/1.3/ [ ] +1 release it [ ] +0 go ahead I don't care [ ] -1 no, do not release it because Thanks in advance Siegfried Goeschl --__--__- To unsubscribe, e-mail: dev-unsubscribe@commons.__apache.org mailto:dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org mailto:dev-h...@commons.apache.org -- E-Mail: garydgreg...@gmail.com mailto:garydgreg...@gmail.com | ggreg...@apache.org mailto:ggreg...@apache.org JUnit in Action, 2nd Ed: http://goog_1249600977http://bit.ly/ECvg0 Spring Batch in Action: http://s.apache.org/HOqhttp://bit.ly/bqpbCK Blog: http://garygregory.wordpress.com http://garygregory.wordpress.com/ Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: [VOTE][email] Release Commons Email 1.3 based on RC2
Hi Stefan, sounds good - I will check Siegfried Goeschl On 06.12.11 16:52, Stefan Bodewig wrote: On 2011-12-06, Siegfried Goeschl wrote: ad RAT - I added a license header for mime.types but doing this for the mail messages is not possible AFAIK since there is no such thing as a comment for mail messages Since you use the latest Commons parent you now have access to RAT 0.7 (I think) which allows you to exclude files that you know cannot contain a license. Stefan - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: [VOTE][email] Release Commons Email 1.3 based on RC2
Hi Siegfried: Thank you for preparing the RC. This /sounds/ worrisome from RAT: Unapproved licenses: src/resources/META-INF/mime.types src/test/eml/attachment-only.eml src/test/eml/html-attachment.eml src/test/eml/multipart-report.eml src/test/eml/simple-reply.eml src/test/eml/simple.eml Where is the 'src' zip I can build from? Looks like there are three easy to fix checkstyle errors (not the missing Javadocs.) -1: Can you justify the 51 Clirr errors in a minor release? Our current guideline is that such a change means a major version and new package. I do not care which way we go but we should be mindful of binary compatibility for minor release. Thank you, Gary On Mon, Dec 5, 2011 at 4:31 PM, Siegfried Goeschl sgoes...@gmx.at wrote: Hi folks, I would like to call a vote to release commons-email-1.3 which contains the following bug fixes and improvements found here http://people.apache.org/**builds/commons/email/1.3/RC2/** site/changes-report.htmlhttp://people.apache.org/builds/commons/email/1.3/RC2/site/changes-report.html Tag: https://svn.apache.org/repos/**asf/commons/proper/email/tags/** EMAIL_1_3_RC2https://svn.apache.org/repos/asf/commons/proper/email/tags/EMAIL_1_3_RC2 Site: http://people.apache.org/**builds/commons/email/1.3/RC2/**site/index.htmlhttp://people.apache.org/builds/commons/email/1.3/RC2/site/index.html Binaries: http://people.apache.org/**builds/commons/email/1.3/RC2/** staged/org/apache/commons/**commons-email/1.3/http://people.apache.org/builds/commons/email/1.3/RC2/staged/org/apache/commons/commons-email/1.3/ [ ] +1 release it [ ] +0 go ahead I don't care [ ] -1 no, do not release it because Thanks in advance Siegfried Goeschl --**--**- To unsubscribe, e-mail: dev-unsubscribe@commons.**apache.orgdev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org -- E-Mail: garydgreg...@gmail.com | ggreg...@apache.org JUnit in Action, 2nd Ed: http://goog_1249600977http://bit.ly/ECvg0 Spring Batch in Action: http://s.apache.org/HOqhttp://bit.ly/bqpbCK Blog: http://garygregory.wordpress.com Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory
Re: [VOTE][email] Release Commons Email 1.3 based on RC2
Hi Siegfried, Siegfried Goeschl wrote: Hi folks, I would like to call a vote to release commons-email-1.3 which contains the following bug fixes and improvements found here http://people.apache.org/builds/commons/email/1.3/RC2/site/changes- report.html Tag: https://svn.apache.org/repos/asf/commons/proper/email/tags/EMAIL_1_3_RC2 Site: http://people.apache.org/builds/commons/email/1.3/RC2/site/index.html Binaries: http://people.apache.org/builds/commons/email/1.3/RC2/staged/org/apache/commons/commons- email/1.3/ [ ] +1 release it [ ] +0 go ahead I don't care [ ] -1 no, do not release it because -1 like Gary I miss the distribution .zip and .tar.gz files for binaries and source. Cheers, Jörg - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org