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

Robert Scholte commented on MRELEASE-601:
-----------------------------------------

The last line seems to be the cause:
{code}
// we need to eliminate any extra whitespace inside elements, as JDOM will nuke 
it
content = content.replaceAll( "<([^!][^>]*?)\\s{2,}([^>]*?)>", "<$1 $2>" );
content = content.replaceAll( "(\\s{2,}|[^\\s])/>", "$1 />" );
{code}

I think it should be
{code}
content = content.replaceAll( "(\\s{2,})/>", "$1 />" );
{code}

I've tried it with Arnaud's testcase and with the adjustment all other tests 
still succeed.
Unless somebody can explain the reason of this part of the regular expression, 
I'll fix the issue like this.
                
> The Maven 2 release plugin modifies CDATA elements in pom.xml files.
> --------------------------------------------------------------------
>
>                 Key: MRELEASE-601
>                 URL: https://jira.codehaus.org/browse/MRELEASE-601
>             Project: Maven 2.x Release Plugin
>          Issue Type: Bug
>    Affects Versions: 2.0-beta-7
>            Reporter: Brandon Enochs
>            Priority: Blocker
>         Attachments: pom.xml
>
>


--
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