[Rdiff-backup-bugs] [bug #24100] Can't specify the root directory ('c:\') as src in native win32 binary.

[Anders Hellstrand]

Follow-up Comment #15, bug #24100 (project rdiff-backup):

In the procedure 'parse_file_desc' there are a source code comment which I
don't fully understand.

>> "The complication is to allow for quoting of : by a .

Can somebody explain why/when this is needed? Otherwise I suppose the use of
file_desc.split("::") could be helpful to simplify the procedure?

Regarding the handling of long filenames in windows containing spaces i guess
that the easiest thing is that the user call rdiff-backup using pathnames in
the DOS convention naming style (8.3 syntax) where "c:long folder name" is
equal to "C:LONGFO~1". (Spaces removed and strings longer then 8 characters
are normally truncated to 6 characters and a tilde and digit is added to
differentiate potential naming conflicts)

If i make a 'unit test' of the procudure which testcases should I add more
then the following? (some are similiar I know but rather too many then too
few! I wanted to highligh some variants like windows using front OR backslash
and also that it should work using an optional traling backslash after a
folder name; unfortunantly not using a trailing backslash of a root folder
does not give it the same meaning and thats why this 
ticket is still open, the posibility to use "c:")

testcases:

singleword
/linux/path/
c:/program1
c:program2
one:colon   << no real case, i don't know why i added it...
two::colons
server.domain::/destdir1
address@hidden::/destdir2
c:dir1
c:dir2    << currently failing because of trailing backslash
c:         << currently failing because of trailing backslash

BR / ahd71

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?24100>

_______________________________________________
  Meddelandet skickades via/av Savannah
  http://savannah.nongnu.org/