Hi Adam,

Thanks fo the patch!  It looks good and has been committed as revision
12230.

Cheers,
--Van

> On Fri, 2007-12-14 at 18:44 -0800, Adam Monsen wrote:
> [...]
>> A unit test is failing.
> [...]
> 
> To be precise, I broke the unit tests. I must not have run the full
> unit test suite before submitting. My bad! Van, thanks for your help
> isolating the problem. I try to always run all unit tests, and do a
> quick deployment before posting a patch.   
> 
> Here's an updated summary and patch superseding others on this thread:
> 
> * Moved session timeout from the CONFIG_KEY_VALUE_INTEGER table into
>   application-wide configuration,
>   applicationConfiguration.default.properties
> * SystemConfiguration.java: cleaned up formatting and license, removed
>   commented dead code


-------------------------------------------------------------------------
SF.Net email is sponsored by:
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services
for just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace

Reply via email to