Hi Vincent,
In article <[EMAIL PROTECTED]>,
Sun, 27 Jun 2004 13:11:15 +0200,
"Vincent Massol" <[EMAIL PROTECTED]> wrote:
vmassol> Ok, I've committed the fix for the ServletTestRunner. I haven't written any
vmassol> automated test case for it though... :-( (it's quite hard to do...)
vmassol>
vmassol> Could you let me know if it works? (I've uploaded a nightly build in
vmassol> http://cvs.apache.org/builds/jakarta-cactus/nightly/2004-06-27/).
The committed change may not work for Volker.
ConfigurationInitializer#addSystemProperties() will not change
system property value if the one already exists.
> private static void addSystemProperties(ResourceBundle theBundle)
> {
> Enumeration keys = theBundle.getKeys();
>
> while (keys.hasMoreElements())
> {
> String key = (String) keys.nextElement();
> // Only set the system property if it does not already exist.
> // This allows to have a cactus properties file and override
> // some values on the command line.
> if (System.getProperty(key) == null)
> {
> System.setProperty(key, theBundle.getString(key));
> }
> }
> }
Can we change the logic without breaking other use cases?
Regards,
----
Kazuhito SUGURI
mailto:[EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
