2014-06-30 21:44 GMT+02:00 Thomas Neidhart <thomas.neidh...@gmail.com>:
> On 06/30/2014 04:27 PM, Thomas Neidhart wrote: > > On 06/30/2014 02:47 PM, Gary Gregory wrote: > >> If I build trunk with Java 8, I get: > >> > >> Tests in error: > >> HtmlEmailTest.initializationError » IllegalState Failed to transform > >> class wit... > >> ImageHtmlEmailTest.initializationError » IllegalState Failed to > transform > >> clas... > >> MultiPartEmailTest.initializationError » IllegalState Failed to > transform > >> clas... > >> > >> Tests run: 125, Failures: 0, Errors: 3, Skipped: 1 > >> > >> Apache Maven 3.2.2 (45f7c06d68e745d05611f7fd14efb6594181933e; > >> 2014-06-17T09:51:42-04:00) > >> Maven home: C:\Java\apache-maven-3.2.2 > >> Java version: 1.8.0_05, vendor: Oracle Corporation > >> Java home: C:\Program Files\Java\jdk1.8.0_05\jre > >> Default locale: en_US, platform encoding: Cp1252 > >> OS name: "windows 7", version: "6.1", arch: "amd64", family: "dos" > >> > >> With Java 7, all is well: > >> > >> Apache Maven 3.2.2 (45f7c06d68e745d05611f7fd14efb6594181933e; > >> 2014-06-17T09:51:42-04:00) > >> Maven home: C:\Java\apache-maven-3.2.2 > >> Java version: 1.7.0_60, vendor: Oracle Corporation > >> Java home: C:\Program Files\Java\jdk1.7.0_60\jre > >> Default locale: en_US, platform encoding: Cp1252 > >> OS name: "windows 7", version: "6.1", arch: "amd64", family: "windows" > >> Mon 06/30/2014- 8:46:17.47 > C:\vcs\svn\apache\commons\trunks-proper\email> > >> > >> Thoughts? > > > > has to do with powermock. Need to check if a more recent version solves > > the issue. > > it looks like there is no way to support both, Java 5 and Java 8 with > the same set of jars. > How about introducing profiles that pull in different versions of powermock, based on the jdk version we're building with? > > The reason powermock was added was to avoid a potential test failure in > some environments when an invalid url was resolved to something valid > (and to prevent -1 at release votes because of this). > > The test was adapted to mock an jdk class to always throw an exception > for these tests, but afaik the providers where this happened have > changed their setup and it should work fine now. Maybe we should remove > the dependency again and note this potential problem somewhere. > > Thomas > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > -- http://people.apache.org/~britter/ http://www.systemoutprintln.de/ http://twitter.com/BenediktRitter http://github.com/britter
