Details
-
Improvement
-
Status: Closed
-
Major
-
Resolution: Fixed
-
1.2
-
None
-
None
-
All
Description
The attached patch allows system properties in the filename attributes of a config.xml file used in the factory.
The patch is very simple, since commons-digester parses the file and has a built in capability to do the property substitution.
The dependency of commons-configuration to commons-digester goes up from version 1.5 to 1.6. The most recent version 1.7 of commons-digester is not necessary.
The patch can easily be ported to 1.3 and trunk.
Use case: often the config.xml files are contained in a distribution unit like a war file, that should not be manipulated after packaging. The path names in config.xml need some way of making them externally variable so that the packaging unit can be adoped to a special environment without repackaging. By allowing system properties in the file names, this can be done easily.
The implementation is simple, short and low risk.
It would be nice, if you considered the patch for inclusion in the next release.