You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by bu...@apache.org on 2006/01/11 06:23:33 UTC

DO NOT REPLY [Bug 38220] New: - CVS task breaks on project names containing spaces

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

http://issues.apache.org/bugzilla/show_bug.cgi?id=38220

           Summary: CVS task breaks on project names containing spaces
           Product: Ant
           Version: 1.6.2
          Platform: Macintosh
        OS/Version: Mac OS X 10.4
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Core tasks
        AssignedTo: dev@ant.apache.org
        ReportedBy: matt.hilliard@shaw.ca


Someone cleverly decided that the project attribute should split the project into multiple projects with the 
space character, completely disregarding CVS's ability use spaces in its project names.

When trying to access projects with a name containing a space character, this task becomes useless as it 
splits the project name into multiple project names and provides no way of escaping the space character.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


DO NOT REPLY [Bug 38220] - CVS task breaks on project names containing spaces

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

http://issues.apache.org/bugzilla/show_bug.cgi?id=38220





------- Additional Comments From stevel@apache.org  2006-01-11 13:02 -------
Do I detect some sarcasm there?

It looks more like (in AbstractCVSTask.configureCommandline() that the package
is being split, perhaps into a series of packages.

        if (cvsPackage != null) {
            c.createArgument().setLine(cvsPackage);
        }

This may be by design, though if it is so, it should be documented with
examples. I note that the method does the same for CVSROOT:

        if (cvsRoot != null) {
            c.createArgument(true).setLine("-d" + cvsRoot);
        }

Again, some more people will need to look at this to decide if its a bug or if
we are being "clever".

Now, have you tried escaping your package with single or double quotes (whatever
you are not using for your package:

package="'spaces in directories'"

The parsing logic should pick that up and treat it as a single thing.
-Steve

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


DO NOT REPLY [Bug 38220] - CVS task breaks on project names containing spaces

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

http://issues.apache.org/bugzilla/show_bug.cgi?id=38220





------- Additional Comments From bodewig@apache.org  2006-04-12 20:44 -------
I don't think anybody tried to be clever, the code is simply wrong and setValues
was intended.

This leaves us with a backwards compatibility problem.  While I don't think we
need to think about the cvsRoot attribute for too long, but people may have
abused the package attribute "feature".

Add a new attribute or even a nested element?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org