1
Vote

The base download directory gets a backslash appended, rather than the forward slash, in Linux

description

In Linux (and most unix-likes I've worked with), the path separation character is a forward-slash, "/", rather than a backslash, "\". A simple fix is to replace the instances of "\" with Path.DirectorySeparatorChar.ToString() in MainForm.cs.

I would have uploaded a patch, but I'm still new to CodePlex, and don't know the format of its patches. I'm used to unified diff's, and I've attached one to this issue. If you're not familiar with a "unified diff", this is a simple text file that describes changes between an original text file and a newer version. It's more commonplace in the Linux community, but all this one does is replace every instance of "\" with Path.DirectorySeparatorChar.ToString().

Also, this also compiles quite nicely in mono, provided the line '<Reference Include="System.Deployment" />' is removed from DownloadSVN.csproj.

file attachments

comments

aarona8511 wrote Nov 5, 2013 at 7:37 PM

It seems I should have doubled up on my backslashes. The first paragraph should have read:

In Linux (and most unix-likes I've worked with), the path separation character is a forward slash, "/", rather than a backslash, "\". A simple fix is to replace the instances of "\\" with Path.DirectorySeparatorChar.ToString() in MainForm.cs.

This would make DownloadSVN more multi-platform, since, regardless of what the path separator character should be, it will be appended, rather than simply appending a backslash.

And, thanks for this wonderful utility, it's come in VERY handy today!