Re: permission issue - new users

2009-11-14 Thread Christian Hammond
The password field in Admin UI expects an encrypted password, so setting something there won't actually give them a password they can use. There's a way to do it from a Python shell, if you wanted to go that route: $ rb-site manage /path/to/site shell >>> from django.contrib.auth.models im

Re: Active Directory Settings and ReviewBoard

2009-11-14 Thread Christian Hammond
Hi Kunjal, There's no good way for this, short of writing custom code to automate it. You'd need to loop through every user in AD and call reviewboard.webapi.json.find_user("username"). I can't really help with the part about looping through AD users, though. That being said, you should be able t

RE: problem with alias lists to the perforce server name

2009-11-14 Thread Paul Scott
Hi Ravi, I'm a little curious as to what your setup is here. There are only a few cases where I'd expect folks to be using round-robin DNS for Perforce servers. Are these p4p (Perforce proxy) instances? -- Paul From: reviewboard@googlegroups.com [review

Re: API to submit single comments within a review?

2009-11-14 Thread Christian Hammond
Hi, This is definitely doable. The diff viewer actually uses the API for posting comments and reviews. The docs are in need of a full rewrite (slowly getting there), but you can sort of see an example of how to do this at: http://github.com/reviewboard/reviewboard/blob/master/reviewboard/htdocs/

Re: problem with alias lists to the perforce server name

2009-11-14 Thread Christian Hammond
Hi Ravi, There's no good solution to this today. Right now, it needs one perforce repository it can rely upon, and if you were to add multiple repositories, it would not work. In the future I really would like to make our repository matching more flexible, but it's not trivial. There are two thin

Re: problem with alias lists to the perforce server name

2009-11-14 Thread Ravi Kondamuru
Thanks Christian for that info. I was able to patch the script and verify it works. I am putting the changes inline here for your review: diff post-review.orig.py post-review.py 19a20 > from types import ListType 366a368,378 > > # If repository_path is a list, get the name in list registered

Re: problem with alias lists to the perforce server name

2009-11-14 Thread Christian Hammond
Can you put this up on http://reviews.reviewboard.org/ ? You should just use post-review for this, as it'll pass some parameters needed in the diff generation. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On