API LDAP Authentiaction

2011-09-23 Thread Jeff Ward
I'm having more trouble getting LDAP authentication. I'll admit up front, I'm working with a web server where dedicating a portion of apache just to review board was not possible. I had to append the rules to the default site, at the directory root/reviews. This is a hard requirement. I am also

Re: Review Board 1.5.1 viewdiff says could not resolve hostname.

2011-09-23 Thread vijay bhaskar
Hi Christian, I had created 'servers' file in in site path ( /var/www/project.com/data/.subversion/servers ) $ls -al -rw-r--r-- 1 root root 7666 2011-09-23 12:40 servers Server file has below entries under [global] group [global] # http-proxy-exceptions = *.exception.com, www.internal-site.org

Re: Clean install of 1.6.1 serves up 404 error

2011-09-23 Thread Sandeep Tailor
Anton, you're correct, I'm using the SLES 11 SP1 Apache pretty much out of the box (but with the PHP, Python, Perl and Env modules enabled) and those settings came from apache2ctl (an alias for httpd?) -S -M . Sorry, Christian, should've been clearer about where they came from. Anton, I've moved

Upgrading database/site from 1.0 to 1.6.1

2011-09-23 Thread Sandeep Tailor
Okay, so I have a shiny, new, *working*,* *instance of Reviewboard 1.6.1. and my plan to migrate my data from my old site was to run rb-site upgrade on the old site and then copy the newly upgraded database to the new site. Unfortunately, the database still seem to have a certain amount of 1.0ness

Re: rb-site install failed - ImportError: Entry point ('console_scripts', 'rb-site') not found

2011-09-23 Thread Stephen Gallagher
On Fri, 2011-09-16 at 15:02 -0400, Stephen Gallagher wrote: On Fri, 2011-09-16 at 11:36 -0700, Nelson Jones wrote: Switched with --console rb-site --console install /var/www/mysite1 * Welcome to the Review Board site installation wizard This will prepare a Review Board

Re: Upgrading database/site from 1.0 to 1.6.1

2011-09-23 Thread Sandeep Tailor
By the way, the old site used an SQLite backend. I was going to get access to the legacy code reviews by having an SQLite site alongside the production MySQL site that we'll use going forwards. I notice that We don’t * officially* support converting a database from SQLite to other databases. How

Re: rb-site install failed - ImportError: Entry point ('console_scripts', 'rb-site') not found

2011-09-23 Thread Stephen Gallagher
On Sat, 2011-09-17 at 12:02 -0700, Christian Hammond wrote: Pretty sure, yes. For most people, it's just a few commands. The problem you're hitting is that you're trying to perform a custom Python install, instead of using native distro packages. So you're doing far too much by hand, instead

Re: PySVN not installed.

2011-09-23 Thread Sanjay Lal
Hi, Does yum install pysvn work for you? if this works, it should take care of all the dependencies. I tried the same and it installed pysvn withough any issues. -sanjay On Sep 13, 12:40 pm, Monica Sanchez castaneda.mon...@gmail.com wrote: I have the configuration below, Fedora 2.6

Re: Upgrading database/site from 1.0 to 1.6.1

2011-09-23 Thread Christian Hammond
Hi Sandeep, What manual updates did it show? There are several possible pages beyond database-related updates. Unofficially, you can do the following: On the old site: rb-site manage /path/to/site dumpdata dbdump.json On the new: . loaddata dbdump.json There are no guarantees it'll work,

Re: Review Board 1.5.1 viewdiff says could not resolve hostname.

2011-09-23 Thread Christian Hammond
Hi Vijay, Can you verify that, on that same server, you're able to perform a svn checkout of your repository from another user? (Also, you'll want to chown that servers file so Apache can write to it.) Christian -- Christian Hammond - chip...@chipx86.com Review Board -

Re: Clean install of 1.6.1 serves up 404 error

2011-09-23 Thread Christian Hammond
Glad you figured that out. I'm making sure these settings will be added for new sites in 1.6.2+. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Sep 23, 2011 at 5:49 AM, Sandeep Tailor

Trouble uploading git diff file

2011-09-23 Thread Paul Lynch
I recently switched from cvs to git. Our procedure with ReviewBoard with git is now similar to what we were doing before: commit changes, do a git pull, then create a diff of the unpushed changes with git diff FETCH_HEAD..HEAD, then push the changes. Very frequently, when I try to upload such a

Re: Trouble uploading git diff file

2011-09-23 Thread Christian Hammond
How is Review Board set up with that repository? Is it using a local clone? If so, you have to have something set up so that the clone is always up-to-date, or you'll hit errors like this. I generally recommend setting up something like cgit on the repository and then using that as the Raw File

Re: Trouble uploading git diff file

2011-09-23 Thread Paul Lynch
The Review Board repository path setting is the (bare) git repository that is our upstream repository to which we push our changes. I can see the sha1s that Review Board is complaining about in that repository, so they do exist there. On Sep 23, 5:19 pm, Christian Hammond chip...@chipx86.com

Re: Trouble uploading git diff file

2011-09-23 Thread Christian Hammond
Ah okay. How did you generate/upload the diff? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Sep 23, 2011 at 2:22 PM, Paul Lynch plynch...@gmail.com wrote: The Review Board repository path setting

Re: Trouble uploading git diff file

2011-09-23 Thread Paul Lynch
git diff FETCH_HEAD..HEAD (before the push). After that a push was done, and then I tried to upload the file. I have also done a pull in another clone of the repository, and can see the files there too. Uploading was done through the Review Board web site (now at 1.6.1) using the New Review

Re: Trouble uploading git diff file

2011-09-23 Thread Christian Hammond
So off-hand, I don't know why you're hitting this, but let's just run through a couple more things. During upload time, when we go to verify a file's existence, we run: git --git-dir=repo path cat-file -t revision If you run that on your bare repo, does that still work? What do you see? If

Re: Trouble uploading git diff file

2011-09-23 Thread Paul Lynch
It works, but if my current working directory is not in a git repository, I also get a fatal error, after which it outputs the type. For example: git --git-dir=/proj/git/def.git cat-file -t febdf30 fatal: Not a git repository (or any of the parent directories): .git blob (But not that it does

Re: Trouble uploading git diff file

2011-09-23 Thread Christian Hammond
I wouldn't expect it to need to be in that directory. We do look for a fatal: but only for Not a valid object name. However, it's very likely that the error code is when you execute that isn't 0, so we're failing. So for that repo, if you're in /proj/git/def.git, it works? What version of git is

Re: Trouble uploading git diff file

2011-09-23 Thread Paul Lynch
Yes. I tried cd'ing to that /proj/git/def.git, and the fatal error message went away. $ git --version git version 1.7.0.4 On Sep 23, 6:24 pm, Christian Hammond chip...@chipx86.com wrote: I wouldn't expect it to need to be in that directory. We do look for a fatal: but only for Not a valid

Re: Trouble uploading git diff file

2011-09-23 Thread Paul Lynch
It does seem that the error status is 0: git --git-dir=/proj/git/def.git cat-file -t febdf30 fatal: Not a git repository (or any of the parent directories): .git blob echo $? 0 I just notice an odd thing, though. I logged into two other systems that can see that repository (we have it mounted

Re: post-review: HTTP 500 error

2011-09-23 Thread Zamir Khan
Just a follow up. It does seem that what I described below was the problem. The solution was to move the repository to our Office B, where development is done. Now that the Perforce server is on the same LAN, with the same DNS resolution as the reviewboard server, I was able to install without

Issue 2304 in reviewboard: [Page not found] after upgrade to 1.6.1

2011-09-23 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2304 by eddiehor...@gmail.com: [Page not found] after upgrade to 1.6.1 http://code.google.com/p/reviewboard/issues/detail?id=2304 * NOTE: Do not post confidential information in this bug report. * * If you need

Re: Issue 2304 in reviewboard: [Page not found] after upgrade to 1.6.1

2011-09-23 Thread reviewboard
Updates: Owner: chip...@gmail.com Labels: -Priority-Medium Priority-High Component-Deployment Comment #1 on issue 2304 by chip...@gmail.com: [Page not found] after upgrade to 1.6.1 http://code.google.com/p/reviewboard/issues/detail?id=2304 Nothing significant changed between

Re: Issue 2304 in reviewboard: [Page not found] after upgrade to 1.6.1

2011-09-23 Thread reviewboard
Updates: Status: NeedInfo Comment #2 on issue 2304 by chip...@gmail.com: [Page not found] after upgrade to 1.6.1 http://code.google.com/p/reviewboard/issues/detail?id=2304 Try changing your Apache config so that ScriptAlias goes from: WSGIScriptAlias /p4reviewboard/

Re: Issue 2155 in reviewboard: post-review fails to upload diff if non-ascii characters are used in diff

2011-09-23 Thread reviewboard
Comment #5 on issue 2155 by chip...@gmail.com: post-review fails to upload diff if non-ascii characters are used in diff http://code.google.com/p/reviewboard/issues/detail?id=2155 Issue 2300 has been merged into this issue. -- You received this message because you are subscribed to the

Re: Issue 2300 in reviewboard: RBTools 0.3.3 not work with python2.7

2011-09-23 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 2155 Comment #2 on issue 2300 by chip...@gmail.com: RBTools 0.3.3 not work with python2.7 http://code.google.com/p/reviewboard/issues/detail?id=2300 (No comment was entered for this change.) -- You received this message because you are

Re: Issue 2155 in reviewboard: post-review fails to upload diff if non-ascii characters are used in diff

2011-09-23 Thread reviewboard
Updates: Status: Fixed Owner: chip...@gmail.com Labels: Milestone-RBTools-Release1.0 Component-RBTools Comment #6 on issue 2155 by chip...@gmail.com: post-review fails to upload diff if non-ascii characters are used in diff

Re: Issue 2304 in reviewboard: [Page not found] after upgrade to 1.6.1

2011-09-23 Thread reviewboard
Comment #3 on issue 2304 by eddiehor...@gmail.com: [Page not found] after upgrade to 1.6.1 http://code.google.com/p/reviewboard/issues/detail?id=2304 Thanks your prompt response. I'm using mod_python, and later now I found a workaround by set the site at host root direction (in conf,

Re: Issue 2304 in reviewboard: [Page not found] after upgrade to 1.6.1

2011-09-23 Thread reviewboard
Comment #4 on issue 2304 by chip...@gmail.com: [Page not found] after upgrade to 1.6.1 http://code.google.com/p/reviewboard/issues/detail?id=2304 It's possible they changed something that broke mod_python in Django 1.3.1. For the heck of it, try downgrading Django to 1.3.0 and see if it

Re: Issue 2304 in reviewboard: [Page not found] after upgrade to 1.6.1

2011-09-23 Thread reviewboard
Comment #5 on issue 2304 by eddiehor...@gmail.com: [Page not found] after upgrade to 1.6.1 http://code.google.com/p/reviewboard/issues/detail?id=2304 I removed Django 1.3.1 and installed 1.3, but still don't fix. -- You received this message because you are subscribed to the Google Groups

Re: Issue 2177 in reviewboard: git branch is not used

2011-09-23 Thread reviewboard
Updates: Status: Fixed Comment #9 on issue 2177 by trowb...@gmail.com: git branch is not used http://code.google.com/p/reviewboard/issues/detail?id=2177 Fixed in master (feeddca). Thanks! -- You received this message because you are subscribed to the Google Groups reviewboard-issues

Re: Issue 1227 in reviewboard: Data lost when resizing comment box in IE8

2011-09-23 Thread reviewboard
Updates: Status: Fixed Comment #2 on issue 1227 by trowb...@gmail.com: Data lost when resizing comment box in IE8 http://code.google.com/p/reviewboard/issues/detail?id=1227 I think this is fixed now. -- You received this message because you are subscribed to the Google Groups

Re: Issue 2290 in reviewboard: Adding Group for review

2011-09-23 Thread reviewboard
Updates: Status: NotABug Comment #3 on issue 2290 by trowb...@gmail.com: Adding Group for review http://code.google.com/p/reviewboard/issues/detail?id=2290 It is optional. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post

Re: Issue 489 in reviewboard: [ ] DON'T SHIP IT button

2011-09-23 Thread reviewboard
Comment #29 on issue 489 by mike.d.c...@gmail.com: [ ] DON'T SHIP IT button http://code.google.com/p/reviewboard/issues/detail?id=489 How about instead of Ship It, Don't Ship It, we use +1 / 0 / -1? Examples: I've just finished reviewing some code, and I feel it's not exactly ready for the

Re: Issue 1909 in reviewboard: Information passed from post-review to RB does not show up in draft

2011-09-23 Thread reviewboard
Updates: Status: UserError Comment #1 on issue 1909 by trowb...@gmail.com: Information passed from post-review to RB does not show up in draft http://code.google.com/p/reviewboard/issues/detail?id=1909 The only way this can happen is if you're not logged in on the web UI... -- You

Re: Issue 1722 in reviewboard: UTF-16 file from Perforce not displayed

2011-09-23 Thread reviewboard
Updates: Status: New Comment #5 on issue 1722 by trowb...@gmail.com: UTF-16 file from Perforce not displayed http://code.google.com/p/reviewboard/issues/detail?id=1722 (No comment was entered for this change.) -- You received this message because you are subscribed to the Google