[ http://issues.apache.org/jira/browse/COCOON-1898?page=comments#action_12432367 ] Lars Trieloff commented on COCOON-1898: ---------------------------------------
I like the idea of being able to patch any XML file in cocoon, but I would not place them in src/main/resources, but to src/main/xpatch/, just to reduce the cluttering of the src/main/resources directory. Having multiple patch files targeting one file sounds like no good idea when you can have multiple patch targets in one patch file. This should also be the place where profile conditions should be enabled (ideally through property replacement as done in the original ant tasks.) The most important argument for chosing XPatch over any other format (e.g. the more widely distributed standard XUpdate) is that it is already in use in Cocoon. > [PATCH] XPatch support for maven-cocoon-deployer-plugin > ------------------------------------------------------- > > Key: COCOON-1898 > URL: http://issues.apache.org/jira/browse/COCOON-1898 > Project: Cocoon > Issue Type: Improvement > Components: - Build System: Maven > Affects Versions: 2.2-dev (Current SVN) > Reporter: Lars Trieloff > Attachments: maven-cocoon-deployer-plugin-with-xpatch-support.patch > > > The cocoon-deployer-plugin has currently no support for XPatch, which makes > it difficult to modify the web.xml when developing cocoon blocks. For example > the cocoon-xmldb-impl block should add, when deployed, a servlet for xindice > and a servlet mapping for the xindice servlet. This was possible in 2.1 using > the XConfToolTask, but is no longer possible with the current state of the > deployer-plugin. > My patch adds support for patching the web.xml file using *.xweb files in the > /conf directory of a block by filtering the block's jar file during > deployment for conf/*.xweb files, caching the patch document temporarily and > applying them (using code from the orgiginal XConfToolTask in 2.1) to the > web.xml. The patch has currently no support for other files than conf/*.xweb > files and does not support any property expansion. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira