Michael Wechner wrote:
Thorsten Scherler wrote:
Hi all,
I wonder where I can configure the
max.upload.size
for lenya?
I found out that I need to patch the web.xml directly but that should be
really configurable in lenya.
Now I see at least two possibilities to implement it:
1) config/web-xml/*.xweb - the usual way in 1.4.x
you mean src/confpatch ?
something like
src/confpatch/local.upload.xweb?
IMO we should move that to webapp/config/web-xml like the patches
for cocoon.xconf and sitemap.xmap.
2) in local.build.properties - enable.upload and max.upload.size seem to
belong together
I have made a NOTE to build.properties about this.
I think enable.upload and max.upload.size belong together.
Also I think that the most important stuff should be within one config file
and upload seems to me one of them.
I'd rather see them configured in a unique way. Settings in web.xml
should be configured in web.xml patch files. I don't see why people
should look for web.xml settings in local.build.properties, which
IMO refers to build options and not to runtime options, at least that's
what the file name suggests ...
Patch files can be used in the core, in modules, and in publications
in the same way. E.g., you could ship the upload size config file in your
publication, without requiring to change a global properties file.
Also I think we need to have a unified way of configuring the build.
Doing some stuff within (local.)build.properties some stuff somewhere is
else
is very confusing :-(
I agree. I wouldn't configure any of these options in local.build.properties.
Also I think it's very important to have local.* files such that default
setting can be overwritten and it's very clear what is local and what not
and also the code repository (SVN) won't have problems with it ...
+1
-- Andreas
--
Andreas Hartmann
Wyona Inc. - Open Source Content Management - Apache Lenya
http://www.wyona.com http://lenya.apache.org
[EMAIL PROTECTED] [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
