Re: JSON api get-review-request-by-changenum

2011-09-27 Thread Christian Hammond
You want this format now: http:// /api/review-requests/?changenum=&repository= That will return a list matching that criteria. The first item will be what you want. You can use that data as-is or follow the links in the payload. Christian -- Christian Hammond - chip...@chipx86.com Review Board

Re: Error of post-review: "The current directory does not contain a checkout from a supported source code repository."

2011-09-27 Thread Christian Hammond
Home has nothing to do with the checks. That's just for our own usage. You need to run post-review from within an SVN, CVS, Git, etc. checkout directory. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Mon,

Re: JSON api get-review-request-by-changenum

2011-09-27 Thread jack jack
Thanks Chris I followed the link as per your suggestion but i could not see field review request id in XML output. What i am expecting the get_review_request_by_ changenum subroutine should return review id or i am missing something here. Regards, Jack. On Tue, Sep 27, 2011 at 1:33 PM, Christia

Re: JSON api get-review-request-by-changenum

2011-09-27 Thread Christian Hammond
It would help to know what you are seeing. If that review request is public, on the provided repository, with the provided changenum, you should see it. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Tue, S

Re: JSON api get-review-request-by-changenum

2011-09-27 Thread Christian Hammond
In this case, it doesn't look as if you have any review requests with that changenum on that repository that are accessible via your user. This will by default only show ones that are opened for review and publicly viewable. If you want ones that are also closed, you can also pass &status=all. If

Re: JSON api get-review-request-by-changenum

2011-09-27 Thread jack jack
I generated the URL as http://ccsdtool.na.xware.com/ccsdtool/api/review-requests/?changenum=119631&repository=1/ self.myurl = self.rbserver + '/api/review-requests/?changenum=%s&repository=1/' % changenum And pass this url, if you look into the code snippet below once i get the self.myurl, t

Re: JSON api get-review-request-by-changenum

2011-09-27 Thread Christian Hammond
You don't want the / in the arguments to the URL. The HTTP Authorization Required means you need to supply the credentials for a user in the request. This uses the standard HTTP Basic Auth spec, of which there are many examples of uses in Python. See http://www.reviewboard.org/docs/manual/dev/web

Re: JSON api get-review-request-by-changenum

2011-09-27 Thread jack jack
Hi Chris, If i comments that part of the code i.e ( def get_review_request_by_changenum) the posting of review request is progressed ahead. But as i told i am trying to get the review id from the changenum, I am not able to do so as you told that API has changed. Also HTTP Basic Auth is being tak

Re: post-review svn - "Your review request still exists, but the diff is not attached"

2011-09-27 Thread Nelson Jones
That was it. Thank you Christian ! On Sep 26, 11:23 pm, Christian Hammond wrote: > What do you have in your .reviewboardrc file? > > Christian > > -- > Christian Hammond - chip...@chipx86.com > Review Board -http://www.reviewboard.org > VMware, Inc. -http://www.vmware.com > > > > On Mon, Sep 26,

Change email of users

2011-09-27 Thread Martin
Hi, I'm using RB 1.6.1 with LDAP authentication. I want to each user can modifiy his email address. I tried add the users a group with auth|user|Can change user, but it still not working... Does somebody can help me? Thanks in advance. Martin. -- Want to help the Review Board project? Donate

upgrade from 1.0.9 tp 1.6.1 failed

2011-09-27 Thread premod dev
Hi, I have been trying to upgrade my current reviewboard instance to 1.6.1 from the older 1.0.9. But when I try to upgrade the site, its giving me the following error and failing the upgrade. Any idea why? Creating tables ... Creating table accounts_localsiteprofile Creating table attachments_fi

Re: Change email of users

2011-09-27 Thread Chris Clark
Martin wrote: Hi, I'm using RB 1.6.1 with LDAP authentication. I want to each user can modifiy his email address. I tried add the users a group with auth|user|Can change user, but it still not working... NOTE I'm not using LDAP with RB :-) Usually when using LDAP it is a read only resource,

Re: upgrade from 1.0.9 tp 1.6.1 failed

2011-09-27 Thread Chris Clark
premod dev wrote: I have been trying to upgrade my current reviewboard instance to 1.6.1 from the older 1.0.9. But when I try to upgrade the site, its giving me the following error and failing the upgrade. Any idea why? ... No fixtures found. Registering new SCM Tool Plastic SCM (reviewboard.

Re: Change email of users

2011-09-27 Thread Martin
Thanks a lot Chris. I am managing my own instance of ReviewBoard, but, the LDAP is under the client control. So, the only chance to change the email is from the admin account. I will tell you if I can figure out an alternative way. Thanks you very much. Regards. MArtin. -- Want to help t

Re: API LDAP Authentiaction

2011-09-27 Thread Jeff Ward
Yes this is using post-review as the client. Here's the output when I disable anonymous reads: C:\Python27\Scripts\post-review --server=http://server: --debug -- repository=repo_name >>> RBTools 0.3.3 >>> Home = >>> hg showconfig >>> hg root >>> p4 info >>> repository info: Path: , Base path:

Re: Upgrading database/site from 1.0 to 1.6.1

2011-09-27 Thread insysion
It turns out that the reason Review Board thought my database hadn't been updated was because it hadn't: I had moved the site onto my new machine and not updated the relevant paths in /conf/*. My current status is that I now have two perfectly working sites, the legacy running off SQLite and pr

Re: upgrade from 1.0.9 tp 1.6.1 failed

2011-09-27 Thread Christian Hammond
Plastic's a red herring. It just happens to precede the evolution failure. For some reason, your evolution history for RB 1.0.9 doesn't include the "changedescs" table existing. However, that was added in 1.0.9. So something's a bit funky on that install. Did you make a backup of the database bef

Re: API LDAP Authentiaction

2011-09-27 Thread Christian Hammond
Hi Jeff, Thanks for sending me that info. So this actually has nothing to do with LDAP. I've seen this before once, on Python 2.7 only. My guess is there's something that prevented the right headers from reaching the server. Some sort of regression on their end (we already know they broke uploadi

Re: Upgrading database/site from 1.0 to 1.6.1

2011-09-27 Thread Christian Hammond
I'm happy to help, Sandeep :) I'm particularly glad it's working for you now. Merging databases is, well, nearly impossible right now. You'd end up with a broken mess, since entries in tables reference others by ID, and those IDs would change. A custom extraction/merging tool would need to be deve

Re: upgrade from 1.0.9 tp 1.6.1 failed

2011-09-27 Thread premod dev
Thanks Christian for the answer.. Sorry for not mentioning earlier mail, I was trying it on a test bed. The steps which I was following is 1. Installed a fresh copy of 1.0.9 on a new server. 2. Took backup of the db from the production server and restore on the test server of 1.0.9 3.

Re: JSON api get-review-request-by-changenum

2011-09-27 Thread jack jack
> > Hi Chris, > > If i comment out that part of the code i.e ( def > get_review_request_by_changenum) the posting of review request is progressed > ahead. But as i told i am trying to get the review id from the changenum, I > am not able to do so as you told that API has changed. > > Also HTTP Basi

Re: JSON api get-review-request-by-changenum

2011-09-27 Thread Christian Hammond
Not sure if this is a double posting or re-asking for help. Did you see my response about the "/" in the URL and the conditions on which the review request will be shown in the results? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. -

Re: Error of post-review: "The current directory does not contain a checkout from a supported source code repository."

2011-09-27 Thread James Tang
But the problem is I already tried post-review in checkout svn directory(which contains a .svn folder), it still give me same words. How to solve this? Isn't post-review support windows7? On 9月27日, 下午4时24分, Christian Hammond wrote: > Home has nothing to do with the checks. That's just for our ow

Re: JSON api get-review-request-by-changenum

2011-09-27 Thread jack jack
Hi Chris, Apologies for confusion. I did noticed your response and replied for help. Even after removing "/' from the URL i am not able to proceed further. The URL which i create i passed to "self.opener.open" self.myurl = self.rbserver + '/api/review-requests/?changenum=%s&repository=1' % chang

Re: Error of post-review: "The current directory does not contain a checkout from a supported source code repository."

2011-09-27 Thread Christian Hammond
To determine if you're in a valid checkout, we run "svn info" and look for the text "Repository Root". What do you see when you type that? If the output is localized, that's the problem, though we already try to force it to be English. Christian -- Christian Hammond - chip...@chipx86.com Review

Re: post-review new files with Perforce

2011-09-27 Thread Christian Hammond
Hi Ben, I'm going through a backlog of e-mail I missed. If you upgrade to the latest RBTools release, this should be fixed. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Wed, Sep 14, 2011 at 12:17 PM, Be