[ 
https://jira.codehaus.org/browse/MRELEASE-645?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=296180#comment-296180
 ] 

Robert Scholte commented on MRELEASE-645:
-----------------------------------------

To let the invoker-plugin check for a build failure you should add a 
{{invoker.properties}} with {{invoker.buildResult = failure}} (see 
http://maven.apache.org/plugins/maven-invoker-plugin/run-mojo.html#invokerPropertiesFile)
After going through the SCM-code, it looks like a consumer is used to read the 
output and to filter out the files. I'd hoped that an {{ScmFile.getPath()}} 
always returns me a path relative to the working directory, but it looks like 
it cannot guarantee that. In that case it doesn't matter if there's an IT. 
I'll try to figure out what different SCM's can return and see if I should add 
some adjustments. 
                
> Allow File/Directory Patterns for the checkModificationExcludes Option
> ----------------------------------------------------------------------
>
>                 Key: MRELEASE-645
>                 URL: https://jira.codehaus.org/browse/MRELEASE-645
>             Project: Maven 2.x Release Plugin
>          Issue Type: Improvement
>          Components: branch, prepare, scm
>    Affects Versions: 2.1
>         Environment: all
>            Reporter: Stefan Ferstl
>            Assignee: Robert Scholte
>            Priority: Minor
>         Attachments: maven-release-manager-r1305146.patch, 
> maven-release-plugin-it.patch, 
> maven-release-plugin-it-with-scm-provider.patch, modification-excludes.patch, 
> MRELEASE-645.patch
>
>
> The {{checkModificationExcludes}} option does currently only allow the 
> definition single files to be excluded from the SCM modification check. If 
> this option is defined, all files anywhere in the maven project structure 
> with the specified name will be excluded from the check. It is currently not 
> possible to exclude files only within a specific directory or to exclude 
> classes of files, i.e. all files matching a specific file name pattern.
> If the {{checkModificationExcludes}} option allowed the definition of file 
> and directory patterns, these things would be possible.
> *Example 1*: I'd like to exclude a test resource 
> {{src/test/resources/foo.properties}} from the modification check but the 
> real foo.properties in {{src/main/resources}} should still be checked.
> {code:xml} 
> <checkModificationExcludes>
>   
> <checkModificationExclude>src/test/resources/foo.properties</checkModificationExclude>
> </checkModificationExcludes>
> {code}
> *Example 2*: I'd like to exclude all properties files with the prefix {{bar}} 
> from the modification check:
> {code:xml} 
> <checkModificationExcludes>
>   <checkModificationExclude>**/bar*.properties</checkModificationExclude>
> </checkModificationExcludes>
> {code}
> The attached patch modifies the {{ScmCheckModificationsPhase}} to use the 
> {{DirectoryScanner}} from plexus-utils instead of doing a strict file name 
> comparison. The patch does not provide more unit tests for this feature but 
> it adjusts the existing tests to run without any failures.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to