So, I've been working on a git-specific helper script for posting reviews which, among other things, includes the git log in the diff.

For a while, I was seeing a bug where the log shows up incorrectly when posted from a Windows machine. I finally determined that this is caused by the diff quoting the file paths, i.e.:

diff --git "a/.gitlog" "b/.gitlog"
new file mode 100644
index 0000000..<sha>
--- /dev/null
+++ "b/.gitlog"

It seems that 'git diff' quotes the file names if they contain characters that might otherwise confuse the diff parser (what's not seen here is that the original file names were e.g. 'C:\Users\My User\Temporary Files\..." and have had string substitution performed to get to ".gitlog").

What I'm wondering is, if a file name actually needs to be quoted, does RB correctly parse the quoted name? And in particular, should it also be handling the above case "better" than it seems to be doing?


Get the Review Board Power Pack at
Sign up for Review Board hosting at RBCommons:
Happy user? Let us know at
--- You received this message because you are subscribed to the Google Groups "reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
For more options, visit

Reply via email to