commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Oliver Heger <oliver.he...@oliver-heger.de>
Subject Re: [VOTE] Release Commons IO 2.2-RC2
Date Sun, 18 Mar 2012 17:08:01 GMT
Hi Gary,

when checking the RC I found some minor things. None of them is blocking 
IMO. Here is the list:

The release notes could be improved:
- The statement "The Commons IO library contains utility classes, stream 
implementations, file filters, file comparators and endian classes." is 
repeated twice (with minor differences).
- There is an unresolved macro "$release.description.replaceAll("  ", "")"
- I am missing a comment about binary compatibility.

When testing the ant build (Windows 7, Java 1.5) I get a problem with a 
failing test case:
ant test
gives:
BUILD FAILED
D:\data\projects\OpenSource\io\commons-io-2.2-src\build.xml:185: Test 
org.apache.commons.io.FileUtilsTestCase failed

The relevant content of the surefire report looks as follows:
Testcase: testContentEqualsIgnoreEOL took 0,031 sec
	FAILED
null
junit.framework.AssertionFailedError
	at 
org.apache.commons.io.FileUtilsTestCase.testContentEqualsIgnoreEOL(FileUtilsTestCase.java:578)

However, the maven build is successful.

Regarding the site:
- Any idea why the success rate in the surefire report is only 99.891%? 
(My local build resulted in 100%)
- The Checkstyle link appears twice.
- There are 29 some Findbugs errors

Oliver

PS: I will be offline next week, so won't be able to respond.

Am 18.03.2012 04:53, schrieb Gary Gregory:
> Hello All:
>
> This is a VOTE to release Commons IO 2.2-RC2
>
> The changes from RC1 are:
>
> - Built from tag
> - Update release notes with version 2.2
> - Update notice file with 2012 copyright
> - Update @version with $Id$
> - Update site to link to only Javadoc 2.2 and 1.x.
>
> This VOTE is open for at least 72 hours until March 20 2012 at 23:55 EST.
>
> The files:
>
> https://repository.apache.org/content/repositories/orgapachecommons-086/
>
> The tag:
>
> https://svn.apache.org/repos/asf/commons/proper/io/tags/2.2-RC2
>
> The site:
>
> https://people.apache.org/builds/commons/commons-io/2.2/RC2/
>
> New features:
> o Use terabyte (TB) , petabyte (PB) and exabyte (EB) in
> FileUtils.byteCountToDisplaySize(long size)  Issue: IO-287. Thanks to Ron
> Kuris, Gary Gregory.
> o FileUtils.listFiles() doesn't return directories  Issue: IO-173. Thanks
> to Marcos Vinícius da Silva.
> o CharSequenceInputStream to efficiently stream content of a CharSequence
> Issue: IO-297. Thanks to Oleg Kalnichevski.
> o The second constructor of Tailer class does not pass 'delay' to the third
> one  Issue: IO-304. Thanks to liangly.
> o TeeOutputStream does not call branch.close() when main.close() throws an
> exception  Issue: IO-303. Thanks to fabian.barney.
> o ArrayIndexOutOfBoundsException in BOMInputStream when reading a file
> without BOM multiple times  Issue: IO-302. Thanks to jsteuerwald, detinho.
> o Add IOUtils.closeQuietly(Selector) necessary  Issue: IO-301. Thanks to
> kaykay.unique.
> o IOUtils.closeQuietly() should take a ServerSocket as a parameter  Issue:
> IO-292. Thanks to sebb.
> o Add read/readFully methods to IOUtils  Issue: IO-290. Thanks to sebb.
> o Supply a ReversedLinesFileReader  Issue: IO-288. Thanks to Georg Henzler.
> o Add new function FileUtils.directoryContains.  Issue: IO-291. Thanks to
> ggregory.
> o FileUtils.contentEquals and IOUtils.contentEquals - Add option to ignore
> "line endings"
>          Added contentEqualsIgnoreEOL methods to both classes  Issue:
> IO-275. Thanks to CJ Aspromgos.
>
> Fixed Bugs:
> o FileUtils.moveDirectoryToDirectory removes source directory if
> destination is a subdirectory  Issue: IO-300.
> o ReaderInputStream#read(byte[] b, int off, int len) should check for valid
> parameters  Issue: IO-307.
> o ReaderInputStream#read(byte[] b, int off, int len) should always return 0
> for length == 0  Issue: IO-306.
> o "FileUtils#deleteDirectoryOnExit(File)" does not work  Issue: IO-276.
> Thanks to nkami.
> o BoundedInputStream.read() treats max differently from
> BoundedInputStream.read(byte[]...)  Issue: IO-273. Thanks to sebb.
> o Various methods of class 'org.apache.commons.io.FileUtils' incorrectly
> suppress 'java.io.IOException'  Issue: IO-298. Thanks to Christian Schulte.
>
> Changes:
> o ReaderInputStream optimization: more efficient reading of small chunks of
> data  Issue: IO-296. Thanks to Oleg Kalnichevski.
>
> Thank you,
> Gary
>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message