Am 15.06.2015 um 21:34 schrieb Oliver Heger: > Hi Bruno, > > Am 15.06.2015 um 10:34 schrieb Bruno P. Kinoshita: >> Hi >> >> -1 not binding >> >> The following test fails in Ubuntu 14.04 with Java versions: >> >> * 1.8.0_45, vendor: Oracle Corporation, Java home: >> /usr/lib/jvm/java-8-oracle/jre* 1.7.0_79, vendor: Oracle Corporation Java >> home: /opt/jdk1.7.0_79/jre* 1.6.0_45, vendor: Sun Microsystems Inc. Java >> home: /usr/lib/jvm/java-6-oracle/jre >> >> And Maven: >> Apache Maven 3.2.5 (12a6b3acb947671f09b81f49094c53f426d8cea1; >> 2014-12-15T06:29:23+13:00) >> >> Failed tests: >> TestDefaultFileSystem.testGetOutputStreamInvalidPath Expected exception: >> org.apache.commons.configuration2.ex.ConfigurationException >> >> Tried with >> * 1.7.0_79, vendor: Oracle Corporation Java home: /opt/jdk1.7.0_79/jre >> And Maven >> Apache Maven 3.3.3 (7994120775791599e205a5524ec3e0dfe41d4a06; >> 2015-04-22T23:57:37+12:00) >> >> But the test still failed. >> Running in Eclipse to investigate it, looks like it tries to create a `new >> File("this is/<certainly>\\an#invalid#*path!")` and is expecting for an >> exception. >> >> Looking in my workspace I can see that Java successfully created some >> directories: >> kinow@chuva:/tmp/CONFIGURATION_2_0_beta1_RC1$ ls -R thi* >> this is: >> <certainly>\an#invalid#*path! >> kinow@chuva:/tmp/CONFIGURATION_2_0_beta1_RC1$ >> i.e. a directory "this is" was created, with a child directory >> "<certainly>\an#invalid#*path!" >> Later it fails to `rm` the child directory due to its name, so you need to >> delete it by its inode: cd this\ is && find -inum $(ls -i | awk '{print >> $1}') -exec rm {} \; >> One simple solution might be replace >> new File("this is/<certainly>\\an#invalid#*path!") >> by >> >> new File("") >> Which is an invalid path too. > > thanks for the review. I will have a look at the problematic test case > and probably follow your suggestion. But as this issue is only in test > code, I would not consider this blocking.
Fixed in SVN in revision 1686103. Thanks again for the proposal. I am afraid that the vote will not pass as there is no more feedback. I will leave it still open for a while and try to prepare another RC if there is no further reaction. Oliver > > Oliver > >> >> CheersBruno >> >> From: Oliver Heger <oliver.he...@oliver-heger.de> >> To: Commons Developers List <dev@commons.apache.org> >> Sent: Monday, June 15, 2015 8:10 AM >> Subject: [VOTE] Release Configuration 2.0-beta1 based on RC1 >> >> Hi all, >> >> this is a vote for the first beta version of [configuration] 2.0 based >> on the first release candidate. After two alpha versions, it is now time >> to move to a beta version. Since the last alpha version, there has only >> be a small change: an obsolete interface has been removed. Details are >> available in the release notes. >> >> Switching to beta indicates that we expect the interfaces to become more >> stable. However, there is still not yet a guarantee that there will be >> no more changes on the API. >> >> Because this is a beta and not yet an official release I did not deploy >> the artifacts to Nexus; only the distributions were created. >> >> Configuration 2.0-beta1 RC1 is available for review here: >> https://dist.apache.org/repos/dist/dev/commons/configuration >> (revision 9374) >> >> Details of changes since 1.10 and the alpha versions are in the >> release notes: >> >> https://dist.apache.org/repos/dist/dev/commons/configuration/RELEASE-NOTES.txt >> >> http://people.apache.org/~oheger/configuration-2.0-beta1-rc1/changes-report.html >> >> Here is the tag: >> >> http://svn.apache.org/repos/asf/commons/proper/configuration/tags/CONFIGURATION_2_0_beta1_RC1 >> (revision 1685450) >> >> Site: >> http://people.apache.org/~oheger/configuration-2.0-beta1-rc1/ >> (note some links in the menu for version 1.10 are not yet working) >> >> KEYS: >> http://www.apache.org/dist/commons/KEYS >> >> Please review the release candidate and vote. >> This vote will close no sooner than 72 hours from now, i.e. after 2000 >> GMT 17-June 2015 >> >> [ ] +1 Release these artifacts >> [ ] +0 OK, but... >> [ ] -0 OK, but really should fix... >> [ ] -1 I oppose this release because... >> >> Thanks! >> Oliver >> >> --------------------------------------------------------------------- >> 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 > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org