On 5/11/06, Joe Germuska <[EMAIL PROTECTED]> wrote:
There you go again, replying just to me ;-)

Sorry Joe, but its the default address that comes up when I hit reply
- and I don't usually check the address when I reply to something sent
to the list.

I wondered why other people comment when I thought I'd already agreed with you.

At 6:06 PM +0100 5/11/06, Niall Pemberton wrote:
>On 5/11/06, Joe Germuska <[EMAIL PROTECTED]> wrote:
>>  >I have taken some time to check out the 1.3.4 version today -
>>>upgrading my webapp to this version (was on 1.2.9) and the only other
>>>issue(s) I came up with is that we used to distribute things like the
>>>validator-rules.xml config and taglib tlds in the lib directory. I
>>>releaize that these are now all packaged in the jar and the distro
>>>does include the source - but I expect quite a few people will still
>>>have these manually configured and will have difficulty finding them
>>>in the distro.
>>
>>I'm mixed about this; I find them cluttery.  I guess if we've never
>>warned people before, then you're probably right, but I'm inclined to
>>try to deprecate them and steer people towards not expecting them in
>>the WEB-INF dir.  Not something I feel very strongly about, but it's
>>my taste...
>
>Yes, maybe you're right - I guess if people change to configure to use
>the validatior-rules.xml thats packaged in the jar then we'll get less
>questions with people forgetting to upgrade that file.

i don't really know how you do a deprecate-and-remove cycle for
packaging things, though!


>>  >Also the new chain-config.xml - which any tiles user (like myself)
>>>needs to be readily available so that the tiles commands can be added.
>>>Again its difficult to find the chain-config.xml and it seems that the
>>>*commented out* tiles commands have been removed. We should make it
>>>easy to find the chain-config.xml and easy to configure for tiles (by
>>>adding back the commented commands).
>>
>>I had been thinking that the preferred strategy was to steer people
>>to change their config to read
>>"org/apache/struts/tiles/chain-config.xml" from the classpath
>>(included in struts-tiles-1.3.4.jar).  I admit to ambivalence about
>>this vs. the "uncomment these lines" strategy, but since by default
>>most people wouldn't have removed the chain-config.xml from the ZIP,
>>I think it's easier to say "change this in web.xml" than "unpack this
>>thing from web.xml and then edit it."
>
>OK, I didn't realize there was a tiles version of the chain-config.xml
>in the tiles jar - guess I should have read the upgrade notes on the
>wiki :-)

well, I'm still not thrilled about either solution -- or in general
about options for changing the chain config without taking on a lot
of responsibility for tracking future changes; but I'm not sure what
would be better...

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to