Gary Peskin created SCM-772:
-------------------------------

             Summary: GitStatusConsumer does not properly handle quoted paths 
output from GitStatusCommand
                 Key: SCM-772
                 URL: https://jira.codehaus.org/browse/SCM-772
             Project: Maven SCM
          Issue Type: Bug
          Components: maven-scm-provider-git
    Affects Versions: 1.9.1, 1.9
         Environment: All
            Reporter: Gary Peskin


We have several files in our repository with spaces in the file name.  Even 
with --porcelain specified, the filename appears with quotes around it.  Here 
is an example:

 M src/main/webapp/admin-index.html
 M src/main/webapp/adminhelp/en_US/css/AboutThisBook.css
 M "src/main/webapp/adminhelp/en_US/css/Appendix A Role Manager Defs.css"

This messes up the regular expression parsing in GitStatusConsumer.

A better approach would be to modify GitStatusCommand to specify the "-z" 
option instead of the "--porcelain" option.  This option is specifically 
designed to generate machine parseable output from the git status command.  
This would necessitate changes to GitStatusConsumer but would allow the 
elimination of all of the regex parsing in GitStatusConsumer making for a more 
reliable solution.

If you'd like, please let me know and I can work up a patch for the changes to 
these two classes and send it on.



--
This message was sent by Atlassian JIRA
(v6.1.6#6162)

Reply via email to