Re: Cannot get reviewboard to display my whitespace-only diff

2013-05-09 Thread Steve
Well, I'm stumped. Just to be sure I hadn't messed up the installation of 1.7.6, I installed from scratch on a sandbox server and looked at the review over there - same issue. I can't see anything odd in the configuration. About the only thing I can say with certainty is it works properly on

Re: Cannot get reviewboard to display my whitespace-only diff

2013-04-26 Thread Christian Hammond
Hi Steve, I'm not aware off-hand of any changes that would have broken whitespace changes within a line. Definitely something that we need to fix if this is reproducible. Do you see anything in the browser's console log when you click the Show Whitespace Changes? What version of 1.6 are you

Re: Cannot get reviewboard to display my whitespace-only diff

2013-04-26 Thread David Trowbridge
One other question: how are you creating your diff? -David On Fri, Apr 26, 2013 at 12:01 PM, Christian Hammond chip...@gmail.com wrote: Hi Steve, I'm not aware off-hand of any changes that would have broken whitespace changes within a line. Definitely something that we need to fix if this is

Re: Cannot get reviewboard to display my whitespace-only diff

2013-04-26 Thread Steve
On Friday, April 26, 2013 12:01:45 PM UTC-7, Christian Hammond wrote: Hi Steve, I'm not aware off-hand of any changes that would have broken whitespace changes within a line. Definitely something that we need to fix if this is reproducible. Do you see anything in the browser's console

Re: Cannot get reviewboard to display my whitespace-only diff

2013-04-26 Thread Steve
Our SCM is perforce and I used post-review 0.4.2 to post the review. Thanks! --Steve On Friday, April 26, 2013 12:25:13 PM UTC-7, David Trowbridge wrote: One other question: how are you creating your diff? -David On Fri, Apr 26, 2013 at 12:01 PM, Christian Hammond

Re: Cannot get reviewboard to display my whitespace-only diff

2013-04-26 Thread Steve
On Friday, April 26, 2013 2:44:17 PM UTC-7, Steve wrote: On Friday, April 26, 2013 12:01:45 PM UTC-7, Christian Hammond wrote: Hi Steve, I'm not aware off-hand of any changes that would have broken whitespace changes within a line. Definitely something that we need to fix if this is