Error on adding repository to reviewboard

2012-05-31 Thread Arvind
Hello, I have setup a review board on my Ubuntu machine (10.04). When I try to add a repository which is on perforce server, following error message gets flashed everytime : *[P4#run] Errors during command execution(p4 describe s1) [Error]: Client is too old to use this server. Please

Re: Error on adding repository to reviewboard

2012-05-31 Thread Christian Hammond
Hi Arvind, The P4PythonInstaller package we use to install the Perforce bindings may need to be updated. Try the following: 1) Download http://reviewboard.googlecode.com/svn/trunk/P4PythonInstaller/setup.py 2) Modify p4api_version in that file and set it to 12.2 3) Run: sudo python setup.py

Error Log while installing Reviewboard.

2012-05-31 Thread Soumen Hajra
[root@ip-10-28-87-177 logs]# cat error_log [Thu May 31 04:52:40 2012] [notice] SELinux policy enabled; httpd running as context unconfined_u:system_r:httpd_t:s0 [Thu May 31 04:52:40 2012] [notice] suEXEC mechanism enabled (wrapper: /usr/sbin/suexec) [Thu May 31 04:52:40 2012] [warn] module

Error Log while installing Reviewboard.

2012-05-31 Thread Soumen Hajra
While debugging ReviewBoard installation and configuration, we found following error in the error log. Can some one review this and let me know what is the issue here? Many thanks in advance. /etc/httpd/logs [root@ip-10-28-87-177 logs]# cat error_log [Thu May 31 04:52:40 2012] [notice] SELinux

Reviewboard does not like big files?

2012-05-31 Thread Neel.
Hello, I am trying to use Reviewboard. On most cases it works fine but we have one file with around 5 line\2.5 MB - C#. Everytime I submit diff for this file, no matter how big or small the diff is, the swirlie never stops spinning and diff is NEVER displayed. Is this known problem and do

Re: Final procedure for perforce integration?

2012-05-31 Thread Neel.
Hmmm.. I completely misunderstood - I was under the impression that someone, namely you, were ready to work on change and needed help setup-wise. no matter. I will try to find time. In interest of saving time, do we have ANY kind of code documentation? basically what I am trying to do is to

Re: Error on adding repository to reviewboard

2012-05-31 Thread Arvind
Hello Christian, Thanks for your input, I changed p4api_version to 12.1 in setup.py and I got rid of the error. (12.2 didn't work because there was no directory on ftp.performce.com ) However now I ran into another error : *[P4#run] Errors during command execution(p4 describe s1) [Error]:

Re: Error Log while installing Reviewboard.

2012-05-31 Thread Soumen Hajra
All, I deinstalled Django_evaluation 1.4 and reinstall 1.3.1 Performing migrations ... [root@ip-10-28-87-177 ~]# python; Python 2.6.6 (r266:84292, Sep 12 2011, 14:03:14) [GCC 4.4.5 20110214 (Red Hat 4.4.5-6)] on linux2 Type help, copyright, credits or license for more information. import django

Re: Reviewboard does not like big files?

2012-05-31 Thread Christian Hammond
A file that large is just too large for anyone to review and for a browser to render. It really should be split up. We won't be able to optimize for a file that size, mostly because a browser just won't handle it and generating a diff from it would just take too long. Christian On May 31,

Re: Error Log while installing Reviewboard.

2012-05-31 Thread Soumen Hajra
Christian, I could able to solve all the issues with your help. I installed reviewboard successfully. Thank you for all the help. I really appreciate. Thanks, Soumen www.snapstech.com -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us

Re: Reviewboard does not like big files?

2012-05-31 Thread David Trowbridge
One thing that may help is turning off syntax highlighting. - David On May 31, 2012, at 11:14 AM, Christian Hammond chip...@gmail.com wrote: A file that large is just too large for anyone to review and for a browser to render. It really should be split up. We won't be able to optimize for a

How to find out the whitespace difference

2012-05-31 Thread linq936
Hi, It becomes quite often when I look at the diff view, some lines are highlighed, but as far as the text goes, there is no difference. It is either there is a tab or some extra space characters somewhere, but the diff view only shows the whole line. I am wondering if I can easily identify where

ssh login

2012-05-31 Thread Ryan Raasch
Hello, I am having some frustrations in the setup of the reviewboard tool. It has to do with the login to an git server over SSH. Within the web interface, I add the correct git path, however when outputting debug information, I can see that it tries to login with root, a user I did not setup.

Re: ssh login

2012-05-31 Thread Christian Hammond
If no SSH key is available, it will try to use the user invoking the tool (in this case, root), much like standard ssh does. Do you have an SSH key configured on Review Board, and does that key have permission to access the Git repository? Christian -- Christian Hammond - chip...@chipx86.com

Issue 2616 in reviewboard: post-review cannot post utf-8 in summary or description

2012-05-31 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2616 by karsten@gmail.com: post-review cannot post utf-8 in summary or description http://code.google.com/p/reviewboard/issues/detail?id=2616 * NOTE: Do not post confidential information in this bug report. * *

Re: Issue 2616 in reviewboard: post-review cannot post utf-8 in summary or description

2012-05-31 Thread reviewboard
Comment #1 on issue 2616 by karsten@gmail.com: post-review cannot post utf-8 in summary or description http://code.google.com/p/reviewboard/issues/detail?id=2616 Actually body does not need to be converted. The problem is that body is a type 'str' url is of type 'unicode Somewhere in the

Re: Issue 2616 in reviewboard: post-review cannot post utf-8 in summary or description

2012-05-31 Thread reviewboard
Comment #2 on issue 2616 by karsten@gmail.com: post-review cannot post utf-8 in summary or description http://code.google.com/p/reviewboard/issues/detail?id=2616 Patch submitted as review 3133 (http://reviews.reviewboard.org/r/3133/) Attachments: ttt 542 bytes -- You received

Re: Issue 2616 in reviewboard: post-review cannot post utf-8 in summary or description

2012-05-31 Thread reviewboard
Comment #3 on issue 2616 by karsten@gmail.com: post-review cannot post utf-8 in summary or description http://code.google.com/p/reviewboard/issues/detail?id=2616 Patch submitted as review 3133 (http://reviews.reviewboard.org/r/3133/) Attachments: patch 542 bytes -- You

Issue 2617 in reviewboard: Broken link to user page in review page auto update feature

2012-05-31 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2617 by andrewsa...@gmail.com: Broken link to user page in review page auto update feature http://code.google.com/p/reviewboard/issues/detail?id=2617 * NOTE: Do not post confidential information in this bug report. * *

Re: Issue 2359 in reviewboard: Review creation fails with patches created with Subversion 1.7.x if they include a property change.

2012-05-31 Thread reviewboard
Comment #20 on issue 2359 by ben.mar...@gmail.com: Review creation fails with patches created with Subversion 1.7.x if they include a property change. http://code.google.com/p/reviewboard/issues/detail?id=2359 My development team uses 'feature branches' so almost every time we generate a