[Cvs-syncmail] [ cvs-syncmail-Bugs-608641 ] no diffs on files w/ spaces and parens

[SourceForge.net <no...@so...>]

Bugs item #608641, was opened at 2002-09-12 18:25
Message generated for change (Comment added) made by fdrake
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450019&aid=608641&group_id=47611

Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: James Snyder (jsnyder)
>Assigned to: Fred L. Drake, Jr. (fdrake)
Summary: no diffs on files w/ spaces and parens

Initial Comment:
using  %{sVv} for passing file version and location info 
with a file contained in a dir w/ a space and parens in 
the name there is no diff listed in the email dispatched.

i cant get %s alone to work either as is suggested in 
the source

----------------------------------------------------------------------

>Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2003-07-09 00:56

Message:
Logged In: YES 
user_id=3066

I just made massive changes to the way filenames to diff are
recovered from CVS; the command line issues no longer apply
as of revision 1.31.  Closing this as fixed; I've tested the
new code with parentheses, spaces, and backslashes.

----------------------------------------------------------------------

Comment By: Craig Hughes (hughescr)
Date: 2002-11-26 17:42

Message:
Logged In: YES 
user_id=9266

I have a similar problem -- my CVS module has a space in 
the name, so whenever I do a commit, the %{sVv} comes 
through as:

"some directory filename,1.1,1.2 file2,1.1,1.2"

The line:

specs = string.split(args[0])

is the problem -- needs to be smarter at figuring out how to 
break the string up.  I don't know python at all, but is there 
some way to break the args[0] up using a regex like:

(.*)(?: (.*,[.0-9]*,[.0-9]*))+

That's perl regex syntax, but basically it'll make it work 
unless the first file which is modified contains a space in the 
filename.  I don't think it's going to be possible to deal with 
the case where both the directory name and the first filename 
on the line contain spaces in them, because of the fact that 
CVS does not quote the individual items that it passes when 
it substitutes %{sVv}.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450019&aid=608641&group_id=47611