struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Musachy Barroso" <>
Subject Re: Convention plugin recap - proposal
Date Tue, 16 Dec 2008 17:50:12 GMT
The Multiple Unknown Handlers patch was merged into xwork and struts
trunks. Now we can move convention plugin out of the sanbox. Last call
for objections :) (2.1 build coming soon)


On Wed, Nov 26, 2008 at 5:06 PM, Musachy Barroso <> wrote:
> I updated the patch to use a 2.1 dtd, instead of 2.0, and removed the
> annoying empty lines from the diff. Crucible seems to have a bug, and even
> when I uncheck files from the "Patch" section, it keeps them in the review
> (take note Don :) )
> musachy
> On Sat, Nov 15, 2008 at 2:42 PM, <> wrote:
>> On our super long convention thread, we got to the agreement that in order
>> to move convention out of the sandbox, there should be a way to run both
>> plugins side by side, so backward compatibility could be possible. The
>> remaining issue to make that happen was to implement "multiple unknown
>> handler", for which I uploaded a new patch here (against trunk):
>> (
>> that creates an extension point for unknown handlers manager, instead of
>> some ugly static classes I wrote before. We also agreed to include
>> convention after 2.1 was released, but based on how long releases
>> take(hoping to change that soon), I propose that we move convention out of
>> trunk for the next 2.1 release so it will be available when 2.1 goes GA, and
>> not 1 or 2 years after that.
>> @Rainer, @Rene: could you guys take a look at the new patch? That thing
>> was a pain to merge being so old, the new one should merge fine.
>> regards
>> musachy
> --
> "Hey you! Would you help me to carry the stone?" Pink Floyd

"Hey you! Would you help me to carry the stone?" Pink Floyd

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message