Re: Unable to start ReviewBoard

2011-11-08 Thread יונתן קינן
You also need to change the SITE_ROOT variable located in /var/www/reviewboard/conf/settings_local.py(in my installation at least) You need to change SITE_ROOT from '/' to '/review/'(depends on the chosen sub directory) 2011/11/8 Christian Hammond chip...@chipx86.com I just replied to someone

Re: Problem with WSGI on fresh install

2011-11-08 Thread יונתן קינן
You will also need to change the SITE_ROOT variable located in: /var/www/reviewboard/conf/settings_local.py(in my installation at least) You need to change SITE_ROOT from '/' to '/review/'(depends on the chosen sub directory) 2011/11/8 Christian Hammond chip...@chipx86.com 1.6.2 will ship with

Re: Submit plain diff without revision info

2011-11-08 Thread Christian Hammond
Hi, No, it's not possible, and far from trivial. Review Board's whole diff viewer process depends upon being able to fetch the proper files from the repository in order to build a custom side-by-side view. Adding support for generic diffs without revisions would be a very large amount of work.

Re: gitolite and ReviewBoard

2011-11-08 Thread adrian
I have no doubt that the .reviweboardrc mechanism will make post- review work... but it's still a work-around to the actual problem. It is kind of cumbersome to have to rely on NFS plus the .reviewboardrc. We would rather connect directly via ssh:// but we can't get this to work properly.

Re: Submit plain diff without revision info

2011-11-08 Thread Annulen
On 8 ноя, 14:43, Christian Hammond chip...@chipx86.com wrote: Really, you just need to use properly formatted diffs. Use case 1: Some users are coming to upload 1-2 patches., a they don't want to learn VCS we use. Use case 2: People use VCS, but they work on Windows and don't want to install

Re: Problem with WSGI on fresh install

2011-11-08 Thread Long VU
@Christian, @jonkeinan, Thanks, both your work-arounds worked and we need both of them together. On Nov 8, 5:20 am, יונתן קינן jonkei...@gmail.com wrote: You will also need to change the SITE_ROOT variable located in: /var/www/reviewboard/conf/settings_local.py(in my installation at least)

Re: Unable to start ReviewBoard

2011-11-08 Thread Long VU
@Christian, @jonkeinan Both of your work-arounds worked and we do need both work-arounds together. Thanks !!! On Nov 8, 5:22 am, יונתן קינן jonkei...@gmail.com wrote: You also need to change the SITE_ROOT variable located in /var/www/reviewboard/conf/settings_local.py(in my installation at

Re: Submit plain diff without revision info

2011-11-08 Thread Stephen Gallagher
On Tue, 2011-11-08 at 03:09 -0800, Annulen wrote: Use case 2: People use VCS, but they work on Windows and don't want to install Python to get post-review. It's almost impossible to format diff manually for git or hg. This is not true at all, at least for git: git format-patch -1 -M -C

Unable to add a CVS repository when username is in the form domain\user

2011-11-08 Thread George Eckert
I have installed ReviewBoard 1.6.1 on both an Ubuntu 10.04 server and a W2K3 R2 server, trying to get ReviewBoard happy with our CVS repository. The behavior I am seeing when trying to configure the repository is the same on both systems. When I attempt to add the CVS repository, I cannot

Re: gitolite and ReviewBoard

2011-11-08 Thread Christian Hammond
You still need .reviewboardrc so post-review knows which server to talk to in the first place. It's actually the recommended setup. You shouldn't have to rely on NFS. Looking over your previous e-mail, I noticed the ssh:// URL. Git actually doesn't use this format. Rather, if you just do

Re: Unable to add a CVS repository when username is in the form domain\user

2011-11-08 Thread Christian Hammond
Hi George, There very well may be a bug involving backslashes in the username. I don't know of one off-hand, but it wouldn't shock me. Mind filing a bug about this? And if you know any Python or are willing to get your hands dirty, you could probably trace where this is failing in

Re: Submit plain diff without revision info

2011-11-08 Thread Christian Hammond
On Tuesday, November 8, 2011, Stephen Gallagher step...@gallagherhome.com wrote: On Tue, 2011-11-08 at 03:09 -0800, Annulen wrote: Use case 2: People use VCS, but they work on Windows and don't want to install Python to get post-review. It's almost impossible to format diff manually for git or

Re: Issue 535 in reviewboard: Fix for CVS error Absolute Module Reference Invalid

2011-11-08 Thread reviewboard
Comment #5 on issue 535 by geckert@gmail.com: Fix for CVS error Absolute Module Reference Invalid http://code.google.com/p/reviewboard/issues/detail?id=535 This is definitely broken in 1.6.1 using a newer version of cvsnt and its diff to generate the file: Concurrent Versions System