Details
-
Type:
Improvement
-
Status:
Closed
-
Priority:
Major
-
Resolution: Won't Fix
-
Affects Version/s: 1.0.4
-
Fix Version/s: None
-
Component/s: Core
-
Labels:None
-
Complexity:Intermediate
-
Testcase included:yes
-
Patch Submitted:Yes
-
Number of attachments :
Description
The following patch adds support for directory configuration. This allows to not only copy one file but a set of file based on a root directory and a set of includes/excludes patterns.
The patch is rather limited and needs a second round, namely:
- The configfile property is not supported as the underlying implementation uses the Ant copy task to do the job
- The overwrite property is not supported either as it would be better to handle overwriting on file-by-file basis instead of the whole directory
I haven't found a way to improve this with the current code base. However, a second patch will follow for the Maven plugin that will provide extra configuration option for directories, using the Plexus directory scanner. That way, we'll be able to build the set of FileConfig based on the files to copy and apply proper overwriting/filtering.
Hello Stephane
I have trouble reading your patch since it has changes in the code style mixed with the functional changes.
Can you please only send the functional changes, or even perhaps send two files (one with the style changes on the existing code, another with the functional changes) ?
Thank you