Issue Details (XML | Word | Printable)

Key: FOR-916
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Mathieu Champlon
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Forrest

adding site-wide configuration files

Created: 10/Aug/06 03:37 AM   Updated: 19/Sep/06 09:56 AM
Return to search
Component/s: Core operations
Affects Version/s: 0.8
Fix Version/s: None

Time Tracking:
Not Specified

File Attachments:
  Size
File Licensed for inclusion in ASF works forrest_conf_module_cleanup.diff 2006-08-10 03:43 AM Mathieu Champlon 6 kB
File Licensed for inclusion in ASF works forrest_conf_module_refactoring.diff 2006-08-10 03:43 AM Mathieu Champlon 5 kB
File Licensed for inclusion in ASF works forrest_conf_module_user_global_home.diff 2006-08-10 03:43 AM Mathieu Champlon 4 kB
Issue Links:
Dependants
 

Other Info: Patch available


 Description  « Hide
see thread http://marc.theaimsgroup.com/?t=115493806000005&r=1&w=2

Patches follow as described in the thread, to be applied one after another.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Mathieu Champlon added a comment - 10/Aug/06 03:43 AM
The patch has been split in 3 to ease review, to be applied in this order :

forrest_conf_module_cleanup.diff performs 1/ and 2/ as described in http://marc.theaimsgroup.com/?l=forrest-dev&m=115493805008165&w=2

forrest_conf_module_refactoring.diff performs 3/

forrest_conf_module_user_global_home.diff performs 5/ and 6/

Mathieu Champlon made changes - 10/Aug/06 03:43 AM
Field Original Value New Value
Attachment forrest_conf_module_user_global_home.diff [ 12338551 ]
Attachment forrest_conf_module_refactoring.diff [ 12338550 ]
Attachment forrest_conf_module_cleanup.diff [ 12338549 ]
Thorsten Scherler made changes - 10/Aug/06 10:07 PM
Assignee Thorsten Scherler [ thorsten ]
Thorsten Scherler made changes - 10/Aug/06 10:08 PM
Status Open [ 1 ] In Progress [ 3 ]
Thorsten Scherler made changes - 10/Aug/06 11:26 PM
Assignee Thorsten Scherler [ thorsten ]
Resolution Fixed [ 1 ]
Status In Progress [ 3 ] Closed [ 6 ]
Repository Revision Date User Message
ASF #430588 Thu Aug 10 23:28:05 UTC 2006 thorsten FOR-916 - adding site-wide configuration files
Applying patches from Mathieu Champlon.
Thanks Mathieu for your contribution.
Files Changed
MODIFY /forrest/trunk/main/targets/context.xml
MODIFY /forrest/trunk/site-author/status.xml
MODIFY /forrest/trunk/main/targets/site.xml
MODIFY /forrest/trunk/main/java/org/apache/forrest/conf/ForrestConfModule.java
MODIFY /forrest/trunk/main/forrest.build.xml

Thorsten Scherler made changes - 19/Sep/06 09:56 AM
Link This issue blocks FOR-924 [ FOR-924 ]
Repository Revision Date User Message
ASF #447805 Tue Sep 19 10:46:56 UTC 2006 thorsten FOR-924
Readding the loading from (default-)forrest.properties which got removed in
r430588 when closing FOR-916. This caused tomcat to fail because
tomcat is not aware of the ant system properties.

Thanks Martin Stockhammer for pointing it out and providing us with a patch.
Files Changed
MODIFY /forrest/trunk/site-author/status.xml
MODIFY /forrest/trunk/main/java/org/apache/forrest/conf/ForrestConfModule.java