[VOTE] Struts 1.3.5 Quality

[Ted Husted]

The Struts 1.3.5 Test Build is available to evaluate for release quality.

The release plan is available on the wiki:
  * http://wiki.apache.org/struts/StrutsActionRelease135

The test build, including checksums and signatures, has been deployed to:
  * http://people.apache.org/builds/struts/1.3.5/

Once you have had a chance to review this test build, please respond
with a vote on its quality:

[ ] Alpha
[ ] Beta
[ ] General Availability (GA)

Everyone is invited to vote. Votes by PMC members are considered
binding. A vote passes if there are at least three +1s and more +1s
than -1s.

Please remember that a binding +1 for GA implies that you intend to
support the release with patches and posts to the user list.

-Ted.

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

[Hubert Rabago]

I found these issues:

1.) ForwardConfig inheritance doesn't work when trying to inherit
global forwards.  There's code to handle it within the config objects,
but they're not being triggered at app startup.

2.) Arbitrary properties don't get copied when creating ActionRedirect
objects based on an existing ForwardConfig.  Anyone have any
objections if I reuse BaseConfig.inheritProperties() to fix this?

Both issues affect features that weren't in the 1.2.x line, so people
upgrading from a previous GA release won't be affected.

I'll put in fixes within the week.


Hubert


On 7/23/06, Ted Husted <husted@...> wrote:

> The Struts 1.3.5 Test Build is available to evaluate for release quality.
>
> The release plan is available on the wiki:
>   * http://wiki.apache.org/struts/StrutsActionRelease135
>
> The test build, including checksums and signatures, has been deployed to:
>   * http://people.apache.org/builds/struts/1.3.5/
>
> Once you have had a chance to review this test build, please respond
> with a vote on its quality:
>
> [ ] Alpha
> [ ] Beta
> [ ] General Availability (GA)
>
> Everyone is invited to vote. Votes by PMC members are considered
> binding. A vote passes if there are at least three +1s and more +1s
> than -1s.
>
> Please remember that a binding +1 for GA implies that you intend to
> support the release with patches and posts to the user list.
>
> -Ted.
>

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

[Wendy Smoak-3]

On 7/23/06, Ted Husted <husted@...> wrote:

> Once you have had a chance to review this test build, please respond
> with a vote on its quality:

[ ] Alpha
[X] Beta
[ ] General Availability (GA)

* The struts-faces examples will not deploy on Tomcat 4.1.31 / JDK
1.4.2.  (I think web.xml is invalid.)

* As discussed after the last attempt, we should get user feedback
from a public beta before upgrading a 1.3 release to GA.

* I've been using 1.3.x since last year with no problems, and have had
1.3.5 in production since Sunday.

--
Wendy Smoak

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

[James Holmes-2]

Based on the issues Hubert Rabago and Wendy pointed out...

[ ] Alpha
[X] Beta
[ ] General Availability (GA)

James

-----Original Message-----
From: Wendy Smoak [mailto:wsmoak@...]
Sent: Tuesday, July 25, 2006 11:02 PM
To: Struts Developers List
Subject: Re: [VOTE] Struts 1.3.5 Quality

On 7/23/06, Ted Husted <husted@...> wrote:

> Once you have had a chance to review this test build, please respond
> with a vote on its quality:

[ ] Alpha
[X] Beta
[ ] General Availability (GA)

* The struts-faces examples will not deploy on Tomcat 4.1.31 / JDK
1.4.2.  (I think web.xml is invalid.)

* As discussed after the last attempt, we should get user feedback
from a public beta before upgrading a 1.3 release to GA.

* I've been using 1.3.x since last year with no problems, and have had
1.3.5 in production since Sunday.

--
Wendy Smoak

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


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

[Michael Jouravlev]

On 7/23/06, Ted Husted <husted@...> wrote:

> The Struts 1.3.5 Test Build is available to evaluate for release quality.
>
> The release plan is available on the wiki:
>   * http://wiki.apache.org/struts/StrutsActionRelease135
>
> The test build, including checksums and signatures, has been deployed to:
>   * http://people.apache.org/builds/struts/1.3.5/
>
> Once you have had a chance to review this test build, please respond
> with a vote on its quality:
>
> [ ] Alpha
> [ ] Beta
> [ ] General Availability (GA)
>
> Everyone is invited to vote. Votes by PMC members are considered
> binding. A vote passes if there are at least three +1s and more +1s
> than -1s.
>
> Please remember that a binding +1 for GA implies that you intend to
> support the release with patches and posts to the user list.
>
> -Ted.

I just have glanced over distro files, so the nitpicks are minor. I
have no opinion on the grade yet.

Documentation:
--------------

* Key Technology Primer (former 0.x section of the User's Guide) is
not included in the distro. KTP has been extracted from Struts 1 docs
to cater to both Struts 1 and Struts 2. Included documentation
contains live link to KTP on Apache website. I don't know is it ok or
not.

* Struts Tiles page contains link to itself.

* Struts Tag Reference contains bare listings of tag properties from
TLD file without discription. Yes, Taglibdoc does exist in Javadoc
format, but taglib had tag reference documentation earlier, now it is
not available. This happened quite a while ago, same thing on the live
website.

Cookbook:
---------

* Cookbook contains extracted and modified chain-config.xml, the
reason for that is not explained. It has Tiles section added, but
Cookbook does not seem to be using Tiles.

* Current 1.3.5 default chain-config.xml file contains
RemoveCachedMessages command (distro has been patched recently without
increasing version number), Cookbook's extracted file does not have
this entry.

Blank:
------

* struts-config.xml contains mappings that use non-existent classes or
pages. Should they be removed?

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

[Ted Husted]

On 7/27/06, Michael Jouravlev <jmikus@...> wrote:
> I just have glanced over distro files, so the nitpicks are minor. I
> have no opinion on the grade yet.

As it stands, we have only two other votes as to grade. We need a
quorum of at least three positive votes to proceed. (I'm abstraining
myself.)

I should not that it is helpful for other developers to cast
non-binding votes, to give the PMC a basis to believe that the
distribution is ready for wider distribution. We arleady know the code
works for us. What we need to know is whether it works for other
developers as well.


> * Key Technology Primer (former 0.x section of the User's Guide) is
> not included in the distro. KTP has been extracted from Struts 1 docs
> to cater to both Struts 1 and Struts 2. Included documentation
> contains live link to KTP on Apache website. I don't know is it ok or
> not.

The material in the Key Technology is not specific to a verson or
Struts (or even Struts), so there's no reason to include it n the
distribution. Better to link to a live version with up to date links.

-Ted.

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

[Don Brown-2]

+1 for GA quality

Don

Ted Husted wrote:

> On 7/27/06, Michael Jouravlev <jmikus@...> wrote:
>> I just have glanced over distro files, so the nitpicks are minor. I
>> have no opinion on the grade yet.
>
> As it stands, we have only two other votes as to grade. We need a
> quorum of at least three positive votes to proceed. (I'm abstraining
> myself.)
>
> I should not that it is helpful for other developers to cast
> non-binding votes, to give the PMC a basis to believe that the
> distribution is ready for wider distribution. We arleady know the code
> works for us. What we need to know is whether it works for other
> developers as well.
>
>
>> * Key Technology Primer (former 0.x section of the User's Guide) is
>> not included in the distro. KTP has been extracted from Struts 1 docs
>> to cater to both Struts 1 and Struts 2. Included documentation
>> contains live link to KTP on Apache website. I don't know is it ok or
>> not.
>
> The material in the Key Technology is not specific to a verson or
> Struts (or even Struts), so there's no reason to include it n the
> distribution. Better to link to a live version with up to date links.
>
> -Ted.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>

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