I recently upgraded our instance of ReviewBoard from 1.6.11 to 2.0.15. 
Quite a few people have been reporting issues with uploading diffs that 
contain ./ as the beginning of the file path and also using ./ as the base 
directory. They just get an "Unknown Error" on the patch upload screen. For 
example, if the patch file has this

--- ./path/to/File.java
+++ ./path/to/File.java

that won't work. I've had to ask them to do a find and replace in the file 
and replace ./ with just / and use / as the basedir. So, the patch file 
ends up looking like this:

--- /path/to/File.java
+++ /path/to/File.java

Should ./ not work anymore? Also, I know that ideally, we should be using 
the RBTools to upload files, but it hasn't been approved by our security 
department as a tool we can have installed yet. So, in the meantime I'm 
trying to make people's lives as simple as possible.


Supercharge your Review Board with Power Pack: 
Want us to host Review Board for you? Check out RBCommons: 
Happy user? Let us know! https://www.reviewboard.org/users/
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 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to