Hi,
build worked for me with Java 1.5 and 1.6 on Windows 7. However, I found
the following problems:
- commons-email-1.3.jar from the binary distribution does not contain
the license and NOTICE files in its META-INF folder.
- The clirr report contains 29 errors. I did not follow discussions
during development of [email], so I don't know whether this is a problem
for users or not. However, I would at least expect that incompatible
changes are clearly stated in the release notes.
- The binary distribution is pretty minimalistic. Other commons
components typically ship the Javadocs and other artifacts like the
sources jar. (Maybe not a major problem, but it would be nice if we were
more consistent within Commons.)
- Minor nit: Could the source distribution deflate in a different
directory than the binary one?
<offTopic>
Is it possible with our current Common Parent setup to test the release
with Maven 3 on a JDK 1.4? When I run mvn package -P java-1.4 I get the
following error:
java.lang.UnsupportedClassVersionError:
org/apache/maven/surefire/junit/JUnit3Pr
ovider (Unsupported major.minor version 49.0)
at java.lang.ClassLoader.defineClass0(Native Method)
at java.lang.ClassLoader.defineClass(ClassLoader.java:539)
at
java.security.SecureClassLoader.defineClass(SecureClassLoader.java:12
3)
at java.net.URLClassLoader.defineClass(URLClassLoader.java:251)
at java.net.URLClassLoader.access$100(URLClassLoader.java:55)
at java.net.URLClassLoader$1.run(URLClassLoader.java:194)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:187)
at java.lang.ClassLoader.loadClass(ClassLoader.java:289)
at java.lang.ClassLoader.loadClass(ClassLoader.java:235)
at
org.apache.maven.surefire.booter.IsolatedClassLoader.loadClass(Isolat
edClassLoader.java:97)
at
org.apache.maven.surefire.util.ReflectionUtils.loadClass(ReflectionUt
ils.java:228)
at
org.apache.maven.surefire.util.ReflectionUtils.instantiateOneArg(Refl
ectionUtils.java:128)
at
org.apache.maven.surefire.booter.SurefireReflector.instantiateProvide
r(SurefireReflector.java:239)
at
org.apache.maven.surefire.booter.ProviderFactory.createProvider(Provi
derFactory.java:122)
at
org.apache.maven.surefire.booter.ProviderFactory.invokeProvider(Provi
derFactory.java:81)
at
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(Fork
edBooter.java:103)
at
org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:
74)
</offTopic>
Oliver
Am 04.05.2012 17:31, schrieb Siegfried Goeschl:
Hi,
I would like to call a vote from commons-email-1.3 based on RC3
Changes in this version include:
New features:
o Update the current trunk to be binary compatible with the
commons-email-1.2 release. Issue: EMAIL-111. Thanks to Florian Pirchner.
o Added unit test to ensure that parsing the broken mime message does
not cause an OutOfMemoryException. Issue: EMAIL-110. Thanks to Thomas
Pummer.
o HtmlmageEmail should support class path resources Issue: EMAIL-108.
Thanks to Elisabeth Kasimir, Alexander Kasimir.
o Added a MultiPartEmail.attach(File) method since attaching a file is a
simple and common.
o Added MimeMessageParser and MimeMessageUtils.
Fixed Bugs:
o DataSourceFileResolverTest fails under IBM JDK 1.4 and 1.6 running on
Windows. Issue: EMAIL-112. Thanks to Peter Kofler.
o Maven Site fails with error in Checkstyle configuration. Issue:
EMAIL-113. Thanks to Peter Kofler.
o The patch actually broke sending emails over a secured connection -
disabled the "MAIL_SMTP_SSL_CHECKSERVERIDENTITY" and
"MAIL_SMTP_SSL_ENABLE" activation. Tested the functionality using GMail,
GMX and Office365 so the code is at least working for a couple of
existing SMTP servers. Also added 'sslCheckServerIdentity' including
setter and getter. Also added a chapter regarding "Security" to the user
manual. Issue: EMAIL-105. Thanks to Siegfried Goeschl.
o Added mime.types to META-INF - the definition is actually found in
activation.jar but did not work. Issue: EMAIL-107. Thanks to Claus
Polanka, Michael Jakl.
o STARTTLS can be used even without authenticator. Issue: EMAIL-106.
Thanks to Bruno Harbulot.
o Clarified the meaning of setTLS() which actually sends a "STARTTLS"
command from the client to the SMTP server. Please note that some
"protected" variables were renamed which could break existing code.
Issue: EMAIL-105. Thanks to Bruno Harbulot.
o Fixed HtmlEmail embed toLowerCase bug with Turkish locale. Issue:
EMAIL-102. Thanks to Okan Özeren.
o Specified Content-ID is now used when embedding a File object in an
HtmlEmail. Issue: EMAIL-101. Thanks to Andrew Starodub.
o Restore Java 1.4 compatibility.
o Throwing an IllegalStateException when setting mail session properties
for an already created mail session because the settings would be
ignored. Please note that this change could potentially break existing
(but invalid) code. Issue: EMAIL-96.
o Encoding and folding of headers is now done by commons-email. Issue:
EMAIL-98. Thanks to Mario Daepp.
o The default connection timeout is set to a reasonable default value of
60 seconds. Issue: EMAIL-100. Thanks to David Parks.
o Moving the various constants from 'EMail' to 'EmailConstants'
o All setters are returning "this" to simplify building an email. Issue:
EMAIL-76. Thanks to Yu Kobayashi.
o Adding ImageHtmlEmail to create HTML emails with embedded images
either downloaded from HTTP or from the local file system. Issue:
EMAIL-92. Thanks to Dominik Stadler.
o Calling buildMimeMessage() before invoking send() caused duplicated
mime parts for HtmlEmail. The implementation now enforces that an email
can be only used once and throw an exception when multiple invocations
of buildMimeMessage() are detected. Issue: EMAIL-95.
o Incorrect SMTP Port number shown in error message when an email fails
to send due to a blocked port and SSL is used. Issue: EMAIL-91. Thanks
to Kevin Lester.
The files:
The artifacts are deployed to Nexus [1] (and [2]).
The tag:
http://svn.apache.org/viewvc/commons/proper/email/tags/EMAIL_1_3_RC3/
The site:
http://people.apache.org/builds/commons/email/1.3/RC3/site/
Additional Notes:
o the RC is binary compatible to commons-email-1.2 whereas the remaining
Clirr warnings stem from moving constants to an interface
Please take a look at the commons-email-1.3 artifacts and vote!
Please note:
This vote is "majority approval" with a minimum of three +1 votes (see
[3]).
------------------------------------------------
[ ] +1 for community members who have reviewed the bits
[ ] +0
[ ] -1 for fatal flaws that should cause these bits not to be released,
and why..............
------------------------------------------------
Thanks in advance,
Siegfried Goeschl
[1]
https://repository.apache.org/content/repositories/orgapachecommons-095/
[2]
https://repository.apache.org/content/repositories/orgapachecommons-095/org/apache/commons/commons-email/1.3/commons-email-1.3-source-release.zip
[3] http://www.apache.org/foundation/voting.html#ReleaseVotes
---------------------------------------------------------------------
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
