On Wed, 1 Sep 2004 05:16:44 +0100, Niall Pemberton
<[EMAIL PROTECTED]> wrote:
> I've just got round to testing Struts 1.2.2 and found the following major
> problems:
>
> * The binary distribution contains the wrong version of commons validator
> jar (the lib distribution seems to have the right one).
This leads me to believe that the binary and lib distros come from
different builds, which seems pretty odd. The 'release' target builds
everything at once.
> * Struts 1.2.2 is incompatible with JDK 1.3 because it uses
> Boolean.valueOf(boolean) introduced in JDK 1.4
This was my fault for doing what FindBugs told me. ;-{ Pity nobody
caught it before 1.2.2 went out.
> Also this distribution has been built/packaged differently to previous
> releases and IMO we should be doing things consistently:
>
> * The binary distribution zip file explodes to jakarta-struts-1.2.2/dist/
> directory rather than just jakarta-struts-1.2.2/
> * The source distribution zip file explodes to jakarta-struts directory
> rather than jakarta-struts-1.2.2-src
> * The source distribution contains all the CVS directories and files
This is very strange. I just ran 'ant release' on my local system, and
it worked just fine, creating all of the uploads correctly. I see none
of the above issues. Perhaps James built the distros some other way?
Not sure.
> James has just fixed the JDK 1.3 incompatibilities in CVS but I would say we
> need to downgrade the 1.2.2 version from a GA quality release and cut a new
> one.
Yes indeed. I've updated the build version, rolled a new release
distribution, and am in the process of uploading it to cvs.apache.org.
I have not tested it yet, and have not signed it, but if people have
the chance to try it out, that would be very helpful.
--
Martin Cooper
>
> Niall
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
