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]