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

[Anders Hellstrand]

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

Andrew,

Thanks for the explanation of "Windows-long-path-names-with-spaces-in". Great
- both that it works and that it is already supported!

Regarding my testcase with "two::colon" it was just the worst possible names
choosen by me, it should have been written as "server::path" of course and
that passes the test (both with and without the patch).

I didn't know that :: was allowed in linux file names, but of course then the
.split method is a bad way to do it.

Take your time and give the proposed patch a second thought because I still
think that it may solve the problem. The only thing that the patch does is
that if a trailing slash exist in the string, rdiff-backup will not go thru
the following lines where the out of range error occur.
The only drawback i can think of, and you may know if there are a use case
there, is if there is a good reason to raise an error if a trailing slash
exists when used from Linux. (I suspect that the already existing error
handling are there for a reason?).
It may be neccessary to have a linux case and a windows case with an OS check
as in some other places of the code to allow/disallow a trailing slash?

BR / Anders

    _______________________________________________________

Reply to this item at:

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

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