Re: Cannot get reviewboard to display my whitespace-only diff
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 my 1.6.9 server, but not my two 1.7.6 servers. Am I the only person experiencing this problem? --Steve On Friday, April 26, 2013 3:14:44 PM UTC-7, Steve wrote: 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 reproducible. Do you see anything in the browser's console log when you click the Show Whitespace Changes? Using firefox with the console on, I see this with the 1.7.6 server: [14:37:19.943] Unknown property 'box-sizing'. Declaration dropped. @ https://reviewboard/r/64801/diff/#index_header [14:37:19.951] Use of Mutation Events is deprecated. Use MutationObserver instead. @ https://reviewboard/static/lib/js/jquery-1.8.2.min.js:2 [14:37:20.012] Error in parsing value for 'white-space'. Declaration dropped. @ https://reviewboard/r/64801/diff/#index_header But I see the same messages with 1.6.9, and the diff shows correctly, so that may not mean much. Sorry, I was mistaken. I get those messages when I click on the 'Expand changes' box, which is working fine. When I click on the 'Show Whitespace changes' I don't see any messages with either 1.7.6 or 1.6.9. I know, I'm making a confusing problem even more confusing. --Steve What version of 1.6 are you testing in the sandbox? 1.6.9. Thanks --Steve Christian On Apr 26, 2013, at 8:29, Steve seide...@gmail.com wrote: We have 1.7.6 in production on a linux server. I posted a review of a php file with whitespace-only diffs on one line. When I view it on reviewboard it won't show me the diff. With these settings: + Expand Changes+ Show Whitespace changes I see the message: This file contains only whitespace changes. That makes sense. When I click on the 'Show Whitespace changes' box, I still don't see any diff. I posted the same exact review to my 1.6 sandbox, and there it behaves the way I would expect, namely when I click on the Show Whitespace changes, I see my 1 line diff. This is a serious problem because those whitespace differences are not leading/trailing spaces and they're significant to the change under review. I compared my site settings for the diff viewer, I don't see anything that jumps out at me. Maybe something will jump out at you: # 1.6 server that behaves the way I want u'diffviewer_context_num_lines': 5, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 20, u'diffviewer_paginate_orphans': 10, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': 0, # 1.7.6 server that does not behave the way I want u'diffviewer_context_num_lines': 7, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 10, u'diffviewer_paginate_orphans': 3, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': None, I don't know why the last setting is 0 for 1.6 and None for 1.7. They seem logically the same and not related to whitespace, but I obviously don't know that for sure. The 1.7.6 database was migrated from a 1.5 server so maybe that's why. The 1.6 sandbox was created from scratch. Is this a bug, or do I have something misconfigured? --Steve -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out. -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- You received this
Re: Cannot get reviewboard to display my whitespace-only diff
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 testing in the sandbox? Christian On Apr 26, 2013, at 8:29, Steve seide.al...@gmail.com wrote: We have 1.7.6 in production on a linux server. I posted a review of a php file with whitespace-only diffs on one line. When I view it on reviewboard it won't show me the diff. With these settings: + Expand Changes+ Show Whitespace changes I see the message: This file contains only whitespace changes. That makes sense. When I click on the 'Show Whitespace changes' box, I still don't see any diff. I posted the same exact review to my 1.6 sandbox, and there it behaves the way I would expect, namely when I click on the Show Whitespace changes, I see my 1 line diff. This is a serious problem because those whitespace differences are not leading/trailing spaces and they're significant to the change under review. I compared my site settings for the diff viewer, I don't see anything that jumps out at me. Maybe something will jump out at you: # 1.6 server that behaves the way I want u'diffviewer_context_num_lines': 5, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 20, u'diffviewer_paginate_orphans': 10, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': 0, # 1.7.6 server that does not behave the way I want u'diffviewer_context_num_lines': 7, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 10, u'diffviewer_paginate_orphans': 3, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': None, I don't know why the last setting is 0 for 1.6 and None for 1.7. They seem logically the same and not related to whitespace, but I obviously don't know that for sure. The 1.7.6 database was migrated from a 1.5 server so maybe that's why. The 1.6 sandbox was created from scratch. Is this a bug, or do I have something misconfigured? --Steve -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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/groups/opt_out. -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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/groups/opt_out.
Re: Cannot get reviewboard to display my whitespace-only diff
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 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 testing in the sandbox? Christian On Apr 26, 2013, at 8:29, Steve seide.al...@gmail.com wrote: We have 1.7.6 in production on a linux server. I posted a review of a php file with whitespace-only diffs on one line. When I view it on reviewboard it won't show me the diff. With these settings: + Expand Changes+ Show Whitespace changes I see the message: This file contains only whitespace changes. That makes sense. When I click on the 'Show Whitespace changes' box, I still don't see any diff. I posted the same exact review to my 1.6 sandbox, and there it behaves the way I would expect, namely when I click on the Show Whitespace changes, I see my 1 line diff. This is a serious problem because those whitespace differences are not leading/trailing spaces and they're significant to the change under review. I compared my site settings for the diff viewer, I don't see anything that jumps out at me. Maybe something will jump out at you: # 1.6 server that behaves the way I want u'diffviewer_context_num_lines': 5, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 20, u'diffviewer_paginate_orphans': 10, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': 0, # 1.7.6 server that does not behave the way I want u'diffviewer_context_num_lines': 7, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 10, u'diffviewer_paginate_orphans': 3, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': None, I don't know why the last setting is 0 for 1.6 and None for 1.7. They seem logically the same and not related to whitespace, but I obviously don't know that for sure. The 1.7.6 database was migrated from a 1.5 server so maybe that's why. The 1.6 sandbox was created from scratch. Is this a bug, or do I have something misconfigured? --Steve -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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/groups/opt_out. -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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/groups/opt_out. -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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/groups/opt_out.
Re: Cannot get reviewboard to display my whitespace-only diff
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 log when you click the Show Whitespace Changes? Using firefox with the console on, I see this with the 1.7.6 server: [14:37:19.943] Unknown property 'box-sizing'. Declaration dropped. @ https://reviewboard/r/64801/diff/#index_header [14:37:19.951] Use of Mutation Events is deprecated. Use MutationObserver instead. @ https://reviewboard/static/lib/js/jquery-1.8.2.min.js:2 [14:37:20.012] Error in parsing value for 'white-space'. Declaration dropped. @ https://reviewboard/r/64801/diff/#index_header But I see the same messages with 1.6.9, and the diff shows correctly, so that may not mean much. What version of 1.6 are you testing in the sandbox? 1.6.9. Thanks --Steve Christian On Apr 26, 2013, at 8:29, Steve seide...@gmail.com javascript: wrote: We have 1.7.6 in production on a linux server. I posted a review of a php file with whitespace-only diffs on one line. When I view it on reviewboard it won't show me the diff. With these settings: + Expand Changes+ Show Whitespace changes I see the message: This file contains only whitespace changes. That makes sense. When I click on the 'Show Whitespace changes' box, I still don't see any diff. I posted the same exact review to my 1.6 sandbox, and there it behaves the way I would expect, namely when I click on the Show Whitespace changes, I see my 1 line diff. This is a serious problem because those whitespace differences are not leading/trailing spaces and they're significant to the change under review. I compared my site settings for the diff viewer, I don't see anything that jumps out at me. Maybe something will jump out at you: # 1.6 server that behaves the way I want u'diffviewer_context_num_lines': 5, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 20, u'diffviewer_paginate_orphans': 10, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': 0, # 1.7.6 server that does not behave the way I want u'diffviewer_context_num_lines': 7, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 10, u'diffviewer_paginate_orphans': 3, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': None, I don't know why the last setting is 0 for 1.6 and None for 1.7. They seem logically the same and not related to whitespace, but I obviously don't know that for sure. The 1.7.6 database was migrated from a 1.5 server so maybe that's why. The 1.6 sandbox was created from scratch. Is this a bug, or do I have something misconfigured? --Steve -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard...@googlegroups.com javascript: For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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...@googlegroups.com javascript:. For more options, visit https://groups.google.com/groups/opt_out. -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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/groups/opt_out.
Re: Cannot get reviewboard to display my whitespace-only diff
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 chi...@gmail.comjavascript: 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 log when you click the Show Whitespace Changes? What version of 1.6 are you testing in the sandbox? Christian On Apr 26, 2013, at 8:29, Steve seide...@gmail.com javascript: wrote: We have 1.7.6 in production on a linux server. I posted a review of a php file with whitespace-only diffs on one line. When I view it on reviewboard it won't show me the diff. With these settings: + Expand Changes+ Show Whitespace changes I see the message: This file contains only whitespace changes. That makes sense. When I click on the 'Show Whitespace changes' box, I still don't see any diff. I posted the same exact review to my 1.6 sandbox, and there it behaves the way I would expect, namely when I click on the Show Whitespace changes, I see my 1 line diff. This is a serious problem because those whitespace differences are not leading/trailing spaces and they're significant to the change under review. I compared my site settings for the diff viewer, I don't see anything that jumps out at me. Maybe something will jump out at you: # 1.6 server that behaves the way I want u'diffviewer_context_num_lines': 5, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 20, u'diffviewer_paginate_orphans': 10, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': 0, # 1.7.6 server that does not behave the way I want u'diffviewer_context_num_lines': 7, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 10, u'diffviewer_paginate_orphans': 3, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': None, I don't know why the last setting is 0 for 1.6 and None for 1.7. They seem logically the same and not related to whitespace, but I obviously don't know that for sure. The 1.7.6 database was migrated from a 1.5 server so maybe that's why. The 1.6 sandbox was created from scratch. Is this a bug, or do I have something misconfigured? --Steve -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard...@googlegroups.com javascript: For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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...@googlegroups.com javascript:. For more options, visit https://groups.google.com/groups/opt_out. -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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/groups/opt_out.
Re: Cannot get reviewboard to display my whitespace-only diff
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 reproducible. Do you see anything in the browser's console log when you click the Show Whitespace Changes? Using firefox with the console on, I see this with the 1.7.6 server: [14:37:19.943] Unknown property 'box-sizing'. Declaration dropped. @ https://reviewboard/r/64801/diff/#index_header [14:37:19.951] Use of Mutation Events is deprecated. Use MutationObserver instead. @ https://reviewboard/static/lib/js/jquery-1.8.2.min.js:2 [14:37:20.012] Error in parsing value for 'white-space'. Declaration dropped. @ https://reviewboard/r/64801/diff/#index_header But I see the same messages with 1.6.9, and the diff shows correctly, so that may not mean much. Sorry, I was mistaken. I get those messages when I click on the 'Expand changes' box, which is working fine. When I click on the 'Show Whitespace changes' I don't see any messages with either 1.7.6 or 1.6.9. I know, I'm making a confusing problem even more confusing. --Steve What version of 1.6 are you testing in the sandbox? 1.6.9. Thanks --Steve Christian On Apr 26, 2013, at 8:29, Steve seide...@gmail.com wrote: We have 1.7.6 in production on a linux server. I posted a review of a php file with whitespace-only diffs on one line. When I view it on reviewboard it won't show me the diff. With these settings: + Expand Changes+ Show Whitespace changes I see the message: This file contains only whitespace changes. That makes sense. When I click on the 'Show Whitespace changes' box, I still don't see any diff. I posted the same exact review to my 1.6 sandbox, and there it behaves the way I would expect, namely when I click on the Show Whitespace changes, I see my 1 line diff. This is a serious problem because those whitespace differences are not leading/trailing spaces and they're significant to the change under review. I compared my site settings for the diff viewer, I don't see anything that jumps out at me. Maybe something will jump out at you: # 1.6 server that behaves the way I want u'diffviewer_context_num_lines': 5, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 20, u'diffviewer_paginate_orphans': 10, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': 0, # 1.7.6 server that does not behave the way I want u'diffviewer_context_num_lines': 7, u'diffviewer_include_space_patterns': [u''], u'diffviewer_max_diff_size': 0, u'diffviewer_paginate_by': 10, u'diffviewer_paginate_orphans': 3, u'diffviewer_show_trailing_whitespace': False, u'diffviewer_syntax_highlighting': True, u'diffviewer_syntax_highlighting_threshold': None, I don't know why the last setting is 0 for 1.6 and None for 1.7. They seem logically the same and not related to whitespace, but I obviously don't know that for sure. The 1.7.6 database was migrated from a 1.5 server so maybe that's why. The 1.6 sandbox was created from scratch. Is this a bug, or do I have something misconfigured? --Steve -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out. -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- 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/groups/opt_out.