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. Thanks! -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ 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.
