Mailing-List: contact commons-dev-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Jakarta Commons Developers List" <commons-dev@jakarta.apache.org>
Received-SPF: pass (herse.apache.org: domain of niall.pemberton@gmail.com
 designates 66.249.82.237 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=OPRj6oUqBta1iexPB/p1rEKscHmhGL90HvLb94O+z0RXCvB4ItGaMncnjcJJrhQ7QDdUi6PMQ8F/k9Gr1X77f3f8HjO5I6pz7CP/LP8kUH4qvnQgUaFEePfm2ghoJoAnEtSWByXEHTpK1EkZAjQZfCwfhwGidOx4zP21DjxpmR0=
Message-ID: <55afdc850611251822m8a0a814g45247ebbcc17004e@mail.gmail.com>
Date: Sun, 26 Nov 2006 02:22:04 +0000
From: "Niall Pemberton" <niall.pemberton@gmail.com>
To: "Jakarta Commons Developers List" <commons-dev@jakarta.apache.org>
Subject: Re: VOTE: Release commons-fileupload 1.2
In-Reply-To: <e75283b10611150552s56e18c9vdbcd5dc68e99033d@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <e75283b10611150552s56e18c9vdbcd5dc68e99033d@mail.gmail.com>

On 11/15/06, Jochen Wiedmann <jochen.wiedmann@gmail.com> wrote:
> Hi,
>
> after the release of commons-parent, the hurdles for publishing the
> next version of commons-fileupload have now gone. So I'd like to
> request a release of
>
>     Commons Fileupload 1.2
>
> The proposed distributables can be found at
>
>     http://people.apache.org/~jochen/commons-fileupload/dist
>
> The proposed web site is at
>
>     http://people.apache.org/~jochen/commons-fileupload/site
>
> Notable changes in 1.2 are:
>
> - Made Streams.asString static. Thanks to Aaron Freeman.
> - Eliminated duplicate code. (FILEUPLOAD-109)
> - Added a streaming API. (FILEUPLOAD-112)
> - Eliminated the necessity of a content-length header. (FILEUPLOAD-93)
> - Eliminated the limitation of a maximum size for a single header
> line. (FILEUPLOAD-108)
>   Thanks to Amichai Rothman.
> - Added the ProgressListener, which allows to implement a progress bar.
>   (FILEUPLOAD-87)
> - Added support for header continuation lines. (FILEUPLOAD-111)
>   Thanks to Amichai Rothman.
> - It is now possible to limit the actual file size and not the request size.
>   (FILEUPLOAD-88) Thanks to Andrey Aristarkhov.
>
>
> Please cast your vote:
>
> [] +1
> [] =0
> [] -1

I would have preferred the release be cut using maven1 rather than m2.
The maven1 build is tried and tested and the gripes of those of us
that checked out previous releases have been fixed in the maven1
build. I guess that doesn't matter if the release is up to scratch but
would be interested to know if others think we're ready for releases
using m2 yet?

The first issue I have with checking out this RC is that you've only
posted the "tar.gz" source and binary distros. I would have liked to
see the full set - zip versions and md5 checksums (the maven1 build
for fileupload creates the md5 checksums for you).

I think there are three serious issues with this RC:
1) It doesn't comply with the new "ASF Source Header and Copyright
Notice Policy":
    http://www.apache.org/legal/src-headers.html

2) According to the jar's manifest file its been built using JDK
1.6.0-rc. Even if JDK 1.6 had been (just) released using it for a
release would make me nervous - but using a RC version of Java to cut
the fileupload release is bad news IMO.

3) The clirr report you produced: which shows the following
incompatibilities with the previous fileupload version:

1) FileUploadBase - public constant MAX_HEADER_SIZE removed.
2) FileUploadBase - protected method createItem removed
3) FileUploadBase - two public constructors for
SizeLimitExceededException removed
4) FileUploadBase - public static class UnknownSizeException removed
5) MulitpartStream - gone from public to package visibility

For these reasons my vote is -1.

One thing thats disappointing is that the last fileupload release had
only 5 checkstyle issues. This one has 376 of which 200 are for tab
characters which I personally dislike in source.

Looking at the source distro "rat" report for the nightly build - its
showing 7 missing license headers - the 2 that stand out are for the
test cases: FileUploadTestCase and ProgressListenerTest:

  http://vmbuild.apache.org/~commons/nightly/nightly_reports/commons-fileupload/src-rat.txt

Niall

> Jochen

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