Issue 1525 in reviewboard: Function to expand hidden lines in a diff does not work for 1.5 Beta 1

2010-03-03 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 1525 by jamesmcdougall: Function to expand hidden lines in a diff  
does not work for 1.5 Beta 1

http://code.google.com/p/reviewboard/issues/detail?id=1525

*NOTE: Do not post confidential information in this bug report.*

What version are you running?
1.5 Beta 1

What's the URL of the page containing the problem?
http://mrbump/reviews/r/13/diff/#index_header

What steps will reproduce the problem?
1. Post a small diff in a large file.
2. Click on the link above or below the actual diff saying  lines
hidden [expand]

What is the expected output? What do you see instead?
Expect to see the hidden lines displayed.
Actually see no change apart from Loading... being displayed at the top
of the screen for a few seconds.

What operating system are you using? What browser?
Windows XP SP3 with Firefox 3.5.8.
Windows XP SP3 with Chrome 4.0.249.89
Windows XP SP3 with IE8

Please provide any additional information below.


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 769 in reviewboard: post-review should warn if there's no diff in the user's path

2010-03-03 Thread reviewboard


Comment #6 on issue 769 by rponton: post-review should warn if there's  
no diff in the user's path

http://code.google.com/p/reviewboard/issues/detail?id=769

This needs to be made a configuration option.  The default diff util in my  
default
path is a crappy old version that is not fully-featured enough to work with  
svn and

reviewboard.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1500 in reviewboard: xx lines hidden [Expand] link stopped working

2010-03-03 Thread reviewboard


Comment #6 on issue 1500 by trowbrds: xx lines hidden [Expand] link  
stopped working

http://code.google.com/p/reviewboard/issues/detail?id=1500

Issue 1525 has been merged into this issue.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1525 in reviewboard: Function to expand hidden lines in a diff does not work for 1.5 Beta 1

2010-03-03 Thread reviewboard

Updates:
Status: Duplicate
Mergedinto: 1500

Comment #1 on issue 1525 by trowbrds: Function to expand hidden lines in a  
diff does not work for 1.5 Beta 1

http://code.google.com/p/reviewboard/issues/detail?id=1525

This is fixed in the nightlies.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1060 in reviewboard: easy_install P4PythonInstaller *Error*

2010-03-03 Thread reviewboard


Comment #20 on issue 1060 by trowbrds: easy_install P4PythonInstaller  
*Error*

http://code.google.com/p/reviewboard/issues/detail?id=1060

The error you just posted looks like gcc isn't fully installed. Maybe  
you're missing

C++ compilers?

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1060 in reviewboard: easy_install P4PythonInstaller *Error*

2010-03-03 Thread reviewboard


Comment #21 on issue 1060 by hbetts3: easy_install P4PythonInstaller *Error*
http://code.google.com/p/reviewboard/issues/detail?id=1060

Trowbrds,

the gcc was installed but not the g++.  I installed that and now everythign  
works.


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 769 in reviewboard: post-review should warn if there's no diff in the user's path

2010-03-03 Thread reviewboard


Comment #7 on issue 769 by chipx86: post-review should warn if there's  
no diff in the user's path

http://code.google.com/p/reviewboard/issues/detail?id=769

The plan after the RBTools 0.2 release is to bundle a custom diff tool that  
we can

use instead of GNU diff for this.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.