DO NOT REPLY [Bug 11967] New: - cvstagdiff incorrectly parses rdiff output

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=11967>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=11967

cvstagdiff incorrectly parses rdiff output

           Summary: cvstagdiff incorrectly parses rdiff output
           Product: Ant
           Version: 1.5
          Platform: Other
        OS/Version: Windows NT/2K
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Core tasks
        AssignedTo: ant-dev@jakarta.apache.org
        ReportedBy: rvanoo@xs4all.nl


cvstagdiff assumes that each line in the output from rdiff starts with 'File 
module/', where module is the name of the cvs module specified in the package 
attribute. This is not always the case:

- the specified module might be an ampersand module
- the specified module might have a differently named (root) directory

I think the solution should be 2-fold:

- the task should check that the line from the rdiff output indeed begins 
with 'File module/', and if it doesn't, only strip of the 'File ' part, 
unless ...
- an attribute (to be added to the task implementation), which specifies the 
part of the path to strip off is specified in the task

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>