Re: Python API: Delete a user from a review group

2015-09-02 Thread Steven MacLeod
It looks like it's not actually possible to delete a user from a review
group using the Python API at the moment. Review Board seems to be
returning an incorrect url which points to the user resource itself, rather
than the review-group-user resource for the delete link. you can see that
by looking at the example here:
https://www.reviewboard.org/docs/manual/dev/webapi/2.0/resources/review-group-user/#example_application/vnd.reviewboard.org.user+json

"delete": {
> "href": "http://reviews.example.com/api/users/doc/;,
> "method": "DELETE"
> },


This is a bug in Review Board and will need to be fixed there to allow
this.


On Wed, Sep 2, 2015 at 12:29 PM, Steve  wrote:

> I'm trying to remove users from review groups using the Python API.  The
> docs indicate that the review group user resource has a delete method, but
> I can't figure out how to get it to work.  I can add a user to a group
> easily this way:
>
> g.get_users().create(username='steve')
>
> Unfortunately, this doesn't work:
>
> g.get_users().delete(username='steve')
>
> Nor does this:
>
> g.get_user()[0].delete()
>
> or any other incarnation I've tried.  Can somebody point me in the right
> direction?
>
> Thanks!
>
> --Steve
>
> --
> Supercharge your Review Board with Power Pack:
> https://www.reviewboard.org/powerpack/
> Want us to host Review Board for you? Check out RBCommons:
> https://rbcommons.com/
> Happy user? Let us know! https://www.reviewboard.org/users/
> ---
> You received this message because you are subscribed to the Google Groups
> "reviewboard" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: Recommended Approach To Add Review Diff Comments Through API

2015-08-12 Thread Steven MacLeod
The API is expecting you to pass in the row number which is you is the
number indicating which row of a side by side diff the comment is on. This
row number might not match the file line number if your diff contains adds
/ deletes on either side.

The Review Bot project has some MIT licensed code which is used to
translate from a file line number to a Review Board diff row number:
https://github.com/reviewboard/ReviewBot/blob/f5a77c039dd59a5880d334c08567af3a068bf933/bot/reviewbot/processing/review.py#L87-L102

On Wed, Aug 12, 2015 at 12:25 PM, jaylee108...@gmail.com wrote:

 Hi All,

 I have some automated machinery that creates diff comments based on the
 line numbers in the new diff.

 This creates an issue on the correct line number the majority of the time.
 However, there are cases when the line number passed through the API does
 not strictly map to the correct diff line number in the UI.

 The belief is that the line number expected through the API is not a 1:1
 mapping with the line number on the new diff.

 What is the recommended approach to translating the new diff line number
 to this API expected line number so that the issue shows up on the correct
 line of the diff?

 For reference, we are using:


 https://www.reviewboard.org/docs/manual/2.5/webapi/2.0/resources/review-diff-comment/

 on a 2.0.12 instance.

 Thanks!

 --
 Supercharge your Review Board with Power Pack:
 https://www.reviewboard.org/powerpack/
 Want us to host Review Board for you? Check out RBCommons:
 https://rbcommons.com/
 Happy user? Let us know! https://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.


-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: Plugin for reviewing .doc, .pdf and .xls in reviewboard

2014-07-16 Thread Steven MacLeod
The Review Board Power Pack provides support for reviewing .pdf
https://www.reviewboard.org/powerpack/

I don't currently know of an extension for reviewing .doc and .xls inside
Review Board, but you can certainly attach them and then download to review.


On Wed, Jul 16, 2014 at 6:37 AM, sharath.r...@gmail.com wrote:

 Looking for a plugin using which .doc, .pdf and .xls can be review similar
 to source code files in reviewboard

 --
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.


-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: Content-Disposition is inline instead of attachment in response to Download Diff

2014-05-22 Thread Steven MacLeod
Most sites seem to do the opposite of what I want about 50% of the time. I
think in this case though since Review Board gives you a diff viewer, you'd
generally want to download the diff when clicking Download Diff, so I
vote for changing it.

As an aside, here is a firefox add-on I use for dealing with the annoying
opposite case where things try to download and I want to view them in the
browser: https://addons.mozilla.org/en-US/firefox/addon/open-in-browser/


On Thu, May 22, 2014 at 9:41 PM, David Trowbridge trowb...@gmail.comwrote:

 Personally, if I click Download X and it displays in the browser, I get
 annoyed.

 -David


 On Thu, May 22, 2014 at 6:37 PM, Christian Hammond chip...@chipx86.comwrote:

 Hi Brett,

 It's actually never come up before, that I know of. It may be more useful
 to do that, but I think it's arguable. Certainly, I sometimes view the diff
 and am happy looking at it right in the browser, instead of having to save
 it somewhere and then open it.

 I'm interested in hearing what others have to say.

 Christian

 --
 Christian Hammond - chip...@chipx86.com
 Review Board - http://www.reviewboard.org
 Beanbag, Inc. - http://www.beanbaginc.com


 On Thu, May 22, 2014 at 6:25 PM, Brett Randall javabr...@gmail.comwrote:

 Hi,

 RB 1.7.21.

 I've noticed that the response to Download Diff includes a
 Content-Disposition: inline:

 GET /r/1234/diff/raw/ HTTP/1.1

 Content-Type: text/x-patch
 ...
 Content-Disposition: *inline*; filename=some.patch


 Shouldn't that be:

 Content-Disposition: *attachment*; filename=some.patch


  ?  Current Chrome version sees text/ MIME type and displays the content
 inline (in the current window) as suggested, instead of raising a download
 dialog.  I image most if not all folks clicking Download Diff want the
 patch as a file, otherwise they would click View Diff.

 Has this come up before?  Raise a bug?

 Thanks
 Brett

 --
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google
 Groups reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send
 an email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.


  --
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.


  --
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/d/optout.


-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Push Based Workflow

2014-02-12 Thread Steven MacLeod
Hi everyone,

Over the weekend Mike Conley and I hacked together a little project to
allow posting code for review using a push based workflow.

The idea is to allow for code to be pushed to a special reviews clone
which handles creating the review requests. This only took a few hours to
throw together and is mostly a prototype, but we think it's a pretty good
proof of concept.

We'd love to hear some feedback or ideas for continued development in this
space.

Demo video: https://vimeo.com/86448355
Code: https://github.com/mikeconley/rb-repo

(Cross posting to both lists as I believe both parties could be interested)

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Problem with signal handling in an extension

2013-12-04 Thread Steven MacLeod
Hi,

I'm actually the original author of the WebHooks extension. It was
originally written as more of a prototype, and probably has a few
bugs.

You appear to have hit one. What I believe the problem here is most
likely, is the webhooks extension doesn't unregister its signal
handlers when it shuts down. So, if you were to disable and then
re-enable the extension, it might have the handler registered twice.

If you'd like an example of an extension to work from with a little
more work put into it, check out
https://github.com/reviewboard/ReviewBot

It should have signal handlers that work properly.

On Wed, Dec 4, 2013 at 8:10 AM, Samuel Degrande
degrande.sam...@orange.fr wrote:
 Hi all.

 I use ReviewBoard 1.7.19, with mod_wsgi.

 I'm writing a rbirker extension, as a replacement to rbcia.

 I used rbwebhooks as an example to write my extension, so mainly
 only the notification code is different: it sends a notification
 to an irker relay through a TCP socket.

 But, the signal handler callbacks are called several times for each
 notification (the number of times varying...).

 I added some logging to my extension and to the mail notification
 code, and here is an example log of what I get:

 16:34:44 INFO - mail: review_published
 16:34:44 INFO - mail: sending to .
 16:34:44 INFO - irker: review_published

 which is fine, but I also sometime get:

 00:10:10 INFO - mail: review_published
 00:10:10 INFO - mail: sending to .
 00:10:10 INFO - irker: review_published
 00:10:10 INFO - irker: review_published
 00:10:10 INFO - irker: review_published
 00:10:10 INFO - irker: review_published

 which is not fine.

 I also logged the call to the extension __init__ (which connect
 callbacks to signals), and I can see lot of traces:

 19:21:06 INFO - Connecting rbiker
 19:26:09 INFO - Connecting rbiker
 19:31:10 INFO - Connecting rbiker
 19:39:06 INFO - Connecting rbiker
 19:44:07 INFO - Connecting rbiker
 19:57:06 INFO - Connecting rbiker

 My guess (but I may be totally wrong, I'm a python newbie and I know
 nothing of django) is that the callbacks could be registered several
 times ?

 Could someone help me solving that issue ?

 Thanks.

 --
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://www.reviewboard.org/users/
 --- You received this message because you are subscribed to the Google
 Groups reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Reviewbot: Should I see cppcheck and pep8 in my extension list?

2013-09-25 Thread Steven MacLeod
Right after the section you've pasted, you'll see this:

After a tool has been installed on a worker, it must be registered with the
Review Bot extension, and configured in the admin panel. Registering tools
is accomplished in the following manner:

   1. Go to the extension list in the admin panel.
   2. Click the 'DATABASE' button for the 'Review-Bot-Extension'.
   3. Click the 'Review bot tools' link.
   4. Click 'REFRESH INSTALLED TOOLS' in the upper right of the page.

This will trigger tool registration for all of the currently running
workers, and refresh the page. You will now see the list of installed tools
and may configure them using this admin panel.



On Wed, Sep 25, 2013 at 2:21 PM, Steve seide.al...@gmail.com wrote:

 I installed reviewbot on our 1.7.14 server using these instructions:

 https://github.com/reviewboard/ReviewBot

 It looks like pep8 and cppcheck are automatically installed with
 reviewbot, but I don't see them listed in the extension list. I'm looking
 for them under

 *Extensions - Database - Review bot tools*

 That page says *'0 review bot tools'*.  Is some additional configuration
 needed?  The instructions say:

 Workers are able to find installed tools using Entry 
 Pointshttp://packages.python.org/distribute/pkg_resources.html#entry-points.
 New tool classes should add a reviewbot.toolsentry point. The entry point
 for the pep8 tool is part of the review bot installation, here is an
 example showing its definition:

 'reviewbot.tools': [
 'cppcheck = reviewbot.tools.cppcheck:CPPCheckTool',
 'cpplint = reviewbot.tools.cpplint:CPPLintTool',
 'pep8 = reviewbot.tools.pep8:PEP8Tool',
 ],


 But it's not clear to me where that setting should be.

 --Steve

 --
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.


-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: listing all review groups and users within the group using a python script

2013-09-11 Thread Steven MacLeod
Yes, this can be done using the API. Assuming you have rbtools 5.x
installed, the following script should work. Modify for your desired
output. It should be noted that this will make a large number of HTTP
requests, and is not optimized.

from rbtools.api.client import RBClient

client = RBClient(http://your-rb-server-here)
root = client.get_root()

groups = root.get_groups()

while True:
for group in groups:
print group.name
users = group.get_users()

while True:
for user in users:
print \t%s % user.username

try:
users = users.get_next()
except StopIteration:
break

try:
groups = groups.get_next()
except StopIteration:
break


On Wed, Sep 11, 2013 at 12:46 PM, Steve seide.al...@gmail.com wrote:

 Better yet, is there a way to do this with the new RBTools 5.x API?

 --Steve


 On Wednesday, September 11, 2013 8:54:07 AM UTC-7, Steve wrote:

 I can list all RB users this way:

  from django.contrib.auth.models import User
  for user in User.objects.all():
 print user

 That correctly prints out all of my users. But the same doesn't work for
 Group

  from django.contrib.auth.models import Group
  for group in Group.objects.all():
 print group
 default reviewer

 I have about 30 groups defined in RB, but I only get 'default reviewer'.
  What I'd like to do is list out all the groups, or more specifically, for
 each group, list all the user in the group.  There's probably a set of
 django functions that will do that, but I'm having trouble finding them.
  Can someone point me in the right direction?

 Thanks

 --Steve

  --
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://www.reviewboard.org/users/
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.


-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: post-review: Error uploading diff

2013-08-20 Thread Steven MacLeod
The minimum Review Board version which fully supports the rbt commands is RB
1.6.12. There were some changes to the web API in this version which rbt
relies on. Unfortunately, you won't be able to make use of most of rbt
without a newer version of Review Board.

Admittedly, the error reporting here was terrible, but is something I'm
working on imporoving.


On Tue, Aug 20, 2013 at 8:43 AM, Akshaya Samantaray 
akshaya.samanta...@gmail.com wrote:

 Thank for the response David .

 I had rbtool 0.4.1 so rbt was not there so i installed 0.5.2 and got rbt
 script  .

 To attach a arbitary file to existing review ,as per you i am using *rbt
 attach 359 --filename=diff* (where 359 is ReviewID), it giving below
 usage messge


 hostname:/vobs/ms 167 % *rbt attach 359 --filename=diff*
 *Usage: rbt attach [options] review-request-id file*
 *
 *
 *rbt: error: Invalid number of arguments provided*


 So i tried the command as *rbt attach 359 diff* (where 359 is the
 reviewID and diff is the filename) but it giving error * CRITICAL:root:
 .Please have look and let me know what could be the issue .*
 *
 *
 *Note:The reviewBoard server version 1.6.3 and Rbtools is 0.5.2 ,Python
 2.7 *

 rbt attach 359 diff
 CRITICAL:root:

 *below is the debug output :*

  169 % rbt attach 359 diff -d
 DEBUG:root:Checking for a Subversion repository...
 DEBUG:root:Checking for a Git repository...
 DEBUG:root:Checking for a Mercurial repository...
 DEBUG:root:Checking for a CVS repository...
 DEBUG:root:Checking for a Perforce repository...
 DEBUG:root:Checking for a Plastic repository...
 DEBUG:root:Checking for a ClearCase repository...
 DEBUG:root:Running: cleartool pwv -short
 DEBUG:root:Running: diff --version
 DEBUG:root:Running: cleartool lsview -full -properties -cview
 DEBUG:root:Running: cleartool describe -short vob:.
 DEBUG:root:Running: cleartool pwv -root
 DEBUG:root:repository info: Path: /vobs/ms-test, Base path: /vobs/ms-test,
 Supports changesets: False
 DEBUG:root:Making HTTP GET request to https://ppsappdev02:443/api/
 Traceback (most recent call last):
   File /clearcase/tools/bin/rbt, line 9, in module
 load_entry_point('RBTools==0.5.2', 'console_scripts', 'rbt')()
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/main.py,
 line 106, in main
 command.run_from_argv([RB_MAIN] + args)
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/__init__.py,
 line 157, in run_from_argv
 exit_code = self.main(*args) or 0
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/attach.py,
 line 67, in main
 request = self.get_review_request(request_id, api_root)
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/attach.py,
 line 55, in get_review_request
 request = api_root.get_review_request(review_request_id=request_id)
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/api/resource.py,
 line 278, in __getattr__
 raise AttributeError
 AttributeError











 On Tuesday, August 20, 2013 1:01:04 AM UTC+5:30, David Trowbridge wrote:

 You can't use post-review --diff-filename to upload arbitrary files.
 The newer rbt commands will let you upload file attachments with rbt
 attach review request ID --filename=filename

 For your existing



 On Mon, Aug 19, 2013 at 4:35 AM, Akshaya Samantaray 
 akshaya.s...@gmail.com wrote:

 Hi David ,

 My object is just to upload a file which  can be any file (*.gif  or
 *.bmp ) (need not be a diff file) using commandline post-review
 --diff-filename=file_name  to review-board server in our clear-case
 environment

 How to do it using post-review in clearcase environment ? let me know if
  any other option to post-review  is there ?

 Thanks



 On Monday, August 19, 2013 4:39:59 PM UTC+5:30, Akshaya Samantaray wrote:

 HI David ,

 Thanks for the response .

 d1.diff is just diff file i have created manually i.e the diff between
 my dev branch changes  Integration branch changes redirected to file
 called  d1.diff and i am trying to upload this manually created diff file
 d1.diff using  *post-review --diff-filename=d1.diff --server=
 https://MyRbServer https://myrbserver/ .*
 *
 *
 I can upload this manually created diff file d1.diff file from GUI in
 reviewboard  page  but not sure why its not uploading using  from command
 line using  post-review diff-**filen**ame=d1.diff --server=
 https://MyRbServer https://myrbserver/ .I am not getting any clue
 from the post-review debug log or reviewboard server log .

 Do i need to have any setup for apache or reviewboard server side
  configuration ?where can i check to get the exact clue for this error ?

 Please have a look and get back to me .I am waiting for your response .

 Thanks

  HTTP POSTing 

Re: Bitnami Review Board Stack-Issue while running post-review with perforce on windows

2013-07-25 Thread Steven MacLeod
Try changing the REPOSITORY field in your .reviewboardrc to be the name
of the repository given in the RB admin panel. This is the name you give
the repository when configuring it for the first time.


On Thu, Jul 25, 2013 at 2:17 AM, RB lanka.a...@gmail.com wrote:

 I have set-up Bitnami Review Board stack on windows 64-bit. Later i am
 able to log-in to Review Board url and i was able to add a Perforce
 repository by clicking the Add Repository link. When i try to submit a
 review request for review using post-review command *post-review 
 12345*(changelist
 number) first it will ask for credentials and after submitting the
 credentials i am getting the following error:

 *There was an error creating this review request.*

 *The repository path public.perforce.com:1666 is not in the list of
 known repositories on the server.*

 *Ask the administrator to add this repository to the Review Board server.
 For information on adding repositories, please read*

 *
 http://www.reviewboard.org/docs/manual/dev/admin/configuration/repositories/
 *

 Replaced my perforce path with public.perforce.com:1666 for posting
 question in this forum.

 I have created .reviewboardrc file with following values:

 REPOSITORY = 'public.perforce.com:1666'
 REVIEWBOARD_URL = 'http://XX.XX.XX.XX:/reviewboard'.

 My question here is though i have added perforce repository on Review
 Board why it is still throwing the error: The repository path 
 public.perforce.com:1666 is not in the list of known repositories on the
 server

 Also am i missing something in .reviewboardrc file.

 Please provide your inputs for resolving this issue.

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: ReviewBot Projects

2013-06-28 Thread Steven MacLeod
Hey Ross,

Currently this is unsupported. I'm working on a large patch to Review Bot
which will include this, along with a number of new features:

- Multiple configurations per tool.
- Manual execution of tools on review requests using a drop down on the
request page.
- Better logging of tool execution.
- Option to consolidate multiple tool reviews into a single review before
it is posted.

I've been pretty busy though, so progress is a little slower than I'd like.
Features are coming.

- Steve


On Fri, Jun 28, 2013 at 2:12 PM, Ross Guarino rssg...@gmail.com wrote:

 Hi,

 Does anyone know if there is a way to set up reviewbot settings for
 different project/repositories? I have different repos that are all on the
 same reviewboard instance but they need different settings for them. Is
 there a way to do this?

 --
 Ross

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: post-review with git

2013-05-29 Thread Steven MacLeod
You can create a '.reviewboardrc' file in the top-level of the repository,
and add the following line:

REPOSITORY = 'repository-name'

where repository-name is the name you gave the repository in the Review
Board admin panel. This should make things work.


On Wed, May 29, 2013 at 5:23 PM, Peter Snelling
usenet.snell...@gmail.comwrote:

 I posted previously that I've installed ReviewBoard 1.7.7.1 on a Suse
 linux server (with git, apache, postgresql, modswgi, memcached, ...). The
 git repository we all use is 172.16.27.63. To get post-review to work
 properly, I had to add the following repository:
Path: snelling@172.16.27.63:/**localdisk/git/repositories/**TCD.git
Raw File URL: http://172.16.27.63/cgi-bin/**
 gitweb/gitweb.cgi?p=TCD.git;a=**blob_plain;f=filename;h=**revision

 and to have review board accept that repository, I had to setup ssh keys
 from the review board server to 172.16.27.63. Now that all works fine for
 me.

 The problem now is when I tried to get another user to use post-review,
 they failed. Their repository was:
 someuser@172.16.27.63:/**localdisk/git/repositories/**TCD.git
 and post-review wouldn't accept that. I had to create another repository
 for them, and add their ssh keys to the git server. That worked, but it
 seems like too much pain for dozens of users.

 If users upload their diff using the ReviewBoard web page and my
 repository (snelling@172.16.27.63:/**localdisk/git/repositories/**TCD.git)
 on the New Review Request screen that works fine. But I'd like to get
 post-review work without having to create a specific repository for each
 user. Does anyone know if this can be done?

 - Peter

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: Creating a review with Python API

2013-05-20 Thread Steven MacLeod
Hi,

'root.get_review_requests().get_self(id='15583', repository=repository)'
will result in a
GET http://.../api/review_requests/?id=15583repository=id

What you're looking for is 'root.get_review_requests().get_item(15583)', or
alternatively 'root.get_review_request(review_request_id=15583). The former
traverses the tree of resources, while the latter uses the root resource's
uri-template for 'review-request'.

Additionally, retrieving the id of the repository is unnecessary to find
the review request with the ID you are looking for.


On Mon, May 20, 2013 at 8:50 AM, Dave Johnston dave.johns...@me.com wrote:

 Hi,

 I've been trying to following the documentation here:
 http://www.reviewboard.org/docs/rbtools/0.5/api/tutorial/

 To create a review for a existing review request, and then add a comment
 to the review.  Unfortunately I'm not getting far.
 I've tried the following:

 root = client.get_root()
 repos = root.get_repositories()
 if repos.num_items  1:
 raise Exception('No valid repositories.')

 repository = repos[0].id
 review_request = root.get_review_requests().get_self(id='15583',
 repository=repository)
 review = review_request.get_reviews().create()
 filediff_id = review.get_diffs()[0].get_files()[0].id
 review.get_diff_comments().create(
 filediff_id=1,
 first_line=131,
 num_lines=2,
 text='This is a diff comment!')

 review.update(body_top='Test Review Comment - please ignore', public=True)

 I get the error: AttributeError: 'ResourceList' object has no attribute
 'get_reviews'

 So it looks like  root.get_review_requests().get_self(id='15583',
 repository=repository) isn't the right request to return an existing review
 request ?
 Any help would be much appreciated.

 Cheers

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: Problem with RBTools 0.5

2013-04-08 Thread Steven MacLeod
You can download curl for windows here: http://curl.haxx.se/download.html


On Mon, Apr 8, 2013 at 1:34 PM, Robert Dailey rcdailey.li...@gmail.comwrote:

 I'm not on Linux, how can I get you what you want on Windows 7?

 On Fri, Apr 5, 2013 at 12:28 AM, Steven MacLeod ste...@smacleod.ca
 wrote:
  Hi Robert,
 
  Sorry for the delay in response, this slipped by in my inbox.
 
  Could you please send me the output of:
 
  $ curl
 http://reviewboard.corp.good.com/api/review-requests/44580/diffs/
  -I
 
  Basically I need to take a look at the headers being returned in the HTTP
  request to RB.
 
 
 
  On Fri, Mar 29, 2013 at 1:40 PM, Robert Dailey rcdailey.li...@gmail.com
 
  wrote:
 
  Using RB version 1.6.3. Output of the command you requested is below:
 
  DEBUG:root:Checking for a Bazaar repository...
 
  DEBUG:root:Checking for a CVS repository...
 
  DEBUG:root:Checking for a ClearCase repository...
 
  DEBUG:root:Checking for a Git repository...
 
  DEBUG:root:Running: git rev-parse --git-dir
 
  DEBUG:root:Command exited with rc 128: ['git', 'rev-parse', '--git-dir']
 
  fatal: Not a git repository (or any of the parent directories): .git
 
  ---
 
  DEBUG:root:Checking for a Mercurial repository...
 
  DEBUG:root:Checking for a Perforce repository...
 
  DEBUG:root:Running: p4 info
 
 
  DEBUG:root:Running: diff --version
 
  DEBUG:root:repository info: Path: perforce-rws2.corp.good.com:3666,
  Base path: None, Supports changesets: True
 
  DEBUG:root:Making HTTP GET request to
  http://reviewboard.corp.good.com/api/
 
 
  DEBUG:root:Making HTTP GET request to
  http://reviewboard.corp.good.com/api/info/
 
 
  INFO:root:Generating diff for changenum 303396
 
  DEBUG:root:Running: p4 describe -s 303396
 
 
  DEBUG:root:Processing delete of
  //depot/dev/DominoProcessDecomp/DominoProcessDecomp.sln
 
  DEBUG:root:Writing
  //depot/dev/DominoProcessDecomp/DominoProcessDecomp.sln#4 to
  c:\users\rdailey\appdata\local\temp\tmpivyvra
 
  DEBUG:root:Running: p4 print -o
  c:\users\rdailey\appdata\local\temp\tmpivyvra -q
  //depot/dev/DominoProcessDecomp/DominoProcessDecomp.sln#4
 
 
  DEBUG:root:Running: diff -urNp
  c:\users\rdailey\appdata\local\temp\tmpivyvra
  c:\users\rdailey\appdata\local\temp\tmpljgwo3
 
  DEBUG:root:Command exited with rc 1: ['diff', '-urNp',
  'c:\\users\\rdailey\\appdata\\local\\temp\\tmpivyvra',
  'c:\\users\\rdailey\\appdata\\local\\temp\\tmpljgwo3']
 
  --- c:\users\rdailey\appdata\local\temp\tmpivyvra 2013-03-29
  12:39:00.971865100 -0500
 
 
  +++ c:\users\rdailey\appdata\local\temp\tmpljgwo3 2013-03-29
  12:39:00.798330400 -0500
 
 
  @@ -1,44 +0,0 @@
 
 
  -
 
 
  -Microsoft Visual Studio Solution File, Format Version 9.00
 
 
  -# Visual Studio 2005
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = services,
  server\gmmserver\domino\server\services\services.vcproj,
  {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}
 
 
  -EndProject
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = interface,
  server\gmmserver\domino\server\interface\interface.vcproj,
  {EADFA55E-6196-44CC-A496-1FDEC07DDE54}
 
 
  -EndProject
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = groupware,
  server\gmmserver\domino\server\groupware\groupware.vcproj,
  {C4D3A251-B853-43E2-84C8-5501FE40EEE8}
 
 
  -EndProject
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = gwserver,
  server\gmmserver\domino\server\gwserver\gwserver.vcproj,
  {0B59A140-C1D4-4A06-ACD8-5A78745D3C9F}
 
 
  -EndProject
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = gwclient,
  server\gmmserver\domino\server\gwclient\gwclient.vcproj,
  {933240B0-F445-49E7-AB25-7BAE6B3C458C}
 
 
  -EndProject
 
 
  -Global
 
 
  - GlobalSection(SolutionConfigurationPlatforms) = preSolution
 
 
  - Debug|Win32 = Debug|Win32
 
 
  - Release|Win32 = Release|Win32
 
 
  - EndGlobalSection
 
 
  - GlobalSection(ProjectConfigurationPlatforms) = postSolution
 
 
  - {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}.Debug|Win32.ActiveCfg =
  Debug|Win32
 
 
  - {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}.Debug|Win32.Build.0 =
 Debug|Win32
 
 
  - {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}.Release|Win32.ActiveCfg =
  Release|Win32
 
 
  - {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}.Release|Win32.Build.0 =
  Release|Win32
 
 
  - {EADFA55E-6196-44CC-A496-1FDEC07DDE54}.Debug|Win32.ActiveCfg =
  Debug|Win32
 
 
  - {EADFA55E-6196-44CC-A496-1FDEC07DDE54}.Debug|Win32.Build.0 =
 Debug|Win32
 
 
  - {EADFA55E-6196-44CC-A496-1FDEC07DDE54}.Release|Win32.ActiveCfg =
  Release|Win32
 
 
  - {EADFA55E-6196-44CC-A496-1FDEC07DDE54}.Release|Win32.Build.0 =
  Release|Win32
 
 
  - {C4D3A251-B853-43E2-84C8-5501FE40EEE8}.Debug|Win32.ActiveCfg =
  Debug|Win32
 
 
  - {C4D3A251-B853-43E2-84C8-5501FE40EEE8}.Debug|Win32.Build.0 =
 Debug|Win32
 
 
  - {C4D3A251-B853-43E2-84C8-5501FE40EEE8}.Release|Win32.ActiveCfg =
  Release|Win32
 
 
  - {C4D3A251-B853-43E2-84C8-5501FE40EEE8}.Release|Win32.Build.0 =
  Release|Win32
 
 
  - {0B59A140-C1D4-4A06-ACD8-5A78745D3C9F}.Debug

Re: Problem with RBTools 0.5

2013-04-08 Thread Steven MacLeod
It appears your Apache server won't allow a HEAD request.

Can I see the output of:

$ curl http://reviewboard.corp.good.com/api/review-requests/44580/diffs/
 -i




On Mon, Apr 8, 2013 at 4:27 PM, Robert Dailey rcdailey.li...@gmail.comwrote:

 Here you go:

 C:\workcurl
 http://reviewboard.corp.good.com/api/review-requests/44580/diffs/
 -I
 HTTP/1.1 405 METHOD NOT ALLOWED
 Date: Mon, 08 Apr 2013 20:24:40 GMT
 Server: Apache/2.2.21 (Unix) mod_wsgi/3.3 Python/2.7 mod_ssl/2.2.17
 OpenSSL/1.0.0c
 Content-Language: en-us
 Expires: Mon, 08 Apr 2013 20:24:40 GMT
 Vary: Accept,Cookie,Accept-Language
 Last-Modified: Mon, 08 Apr 2013 20:24:40 GMT
 Allow: GET, POST
 Cache-Control: max-age=0
 Content-Type: text/html; charset=utf-8

 On Fri, Apr 5, 2013 at 12:28 AM, Steven MacLeod ste...@smacleod.ca
 wrote:
  Hi Robert,
 
  Sorry for the delay in response, this slipped by in my inbox.
 
  Could you please send me the output of:
 
  $ curl
 http://reviewboard.corp.good.com/api/review-requests/44580/diffs/
  -I
 
  Basically I need to take a look at the headers being returned in the HTTP
  request to RB.
 
 
 
  On Fri, Mar 29, 2013 at 1:40 PM, Robert Dailey rcdailey.li...@gmail.com
 
  wrote:
 
  Using RB version 1.6.3. Output of the command you requested is below:
 
  DEBUG:root:Checking for a Bazaar repository...
 
  DEBUG:root:Checking for a CVS repository...
 
  DEBUG:root:Checking for a ClearCase repository...
 
  DEBUG:root:Checking for a Git repository...
 
  DEBUG:root:Running: git rev-parse --git-dir
 
  DEBUG:root:Command exited with rc 128: ['git', 'rev-parse', '--git-dir']
 
  fatal: Not a git repository (or any of the parent directories): .git
 
  ---
 
  DEBUG:root:Checking for a Mercurial repository...
 
  DEBUG:root:Checking for a Perforce repository...
 
  DEBUG:root:Running: p4 info
 
 
  DEBUG:root:Running: diff --version
 
  DEBUG:root:repository info: Path: perforce-rws2.corp.good.com:3666,
  Base path: None, Supports changesets: True
 
  DEBUG:root:Making HTTP GET request to
  http://reviewboard.corp.good.com/api/
 
 
  DEBUG:root:Making HTTP GET request to
  http://reviewboard.corp.good.com/api/info/
 
 
  INFO:root:Generating diff for changenum 303396
 
  DEBUG:root:Running: p4 describe -s 303396
 
 
  DEBUG:root:Processing delete of
  //depot/dev/DominoProcessDecomp/DominoProcessDecomp.sln
 
  DEBUG:root:Writing
  //depot/dev/DominoProcessDecomp/DominoProcessDecomp.sln#4 to
  c:\users\rdailey\appdata\local\temp\tmpivyvra
 
  DEBUG:root:Running: p4 print -o
  c:\users\rdailey\appdata\local\temp\tmpivyvra -q
  //depot/dev/DominoProcessDecomp/DominoProcessDecomp.sln#4
 
 
  DEBUG:root:Running: diff -urNp
  c:\users\rdailey\appdata\local\temp\tmpivyvra
  c:\users\rdailey\appdata\local\temp\tmpljgwo3
 
  DEBUG:root:Command exited with rc 1: ['diff', '-urNp',
  'c:\\users\\rdailey\\appdata\\local\\temp\\tmpivyvra',
  'c:\\users\\rdailey\\appdata\\local\\temp\\tmpljgwo3']
 
  --- c:\users\rdailey\appdata\local\temp\tmpivyvra 2013-03-29
  12:39:00.971865100 -0500
 
 
  +++ c:\users\rdailey\appdata\local\temp\tmpljgwo3 2013-03-29
  12:39:00.798330400 -0500
 
 
  @@ -1,44 +0,0 @@
 
 
  -
 
 
  -Microsoft Visual Studio Solution File, Format Version 9.00
 
 
  -# Visual Studio 2005
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = services,
  server\gmmserver\domino\server\services\services.vcproj,
  {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}
 
 
  -EndProject
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = interface,
  server\gmmserver\domino\server\interface\interface.vcproj,
  {EADFA55E-6196-44CC-A496-1FDEC07DDE54}
 
 
  -EndProject
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = groupware,
  server\gmmserver\domino\server\groupware\groupware.vcproj,
  {C4D3A251-B853-43E2-84C8-5501FE40EEE8}
 
 
  -EndProject
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = gwserver,
  server\gmmserver\domino\server\gwserver\gwserver.vcproj,
  {0B59A140-C1D4-4A06-ACD8-5A78745D3C9F}
 
 
  -EndProject
 
 
  -Project({8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}) = gwclient,
  server\gmmserver\domino\server\gwclient\gwclient.vcproj,
  {933240B0-F445-49E7-AB25-7BAE6B3C458C}
 
 
  -EndProject
 
 
  -Global
 
 
  - GlobalSection(SolutionConfigurationPlatforms) = preSolution
 
 
  - Debug|Win32 = Debug|Win32
 
 
  - Release|Win32 = Release|Win32
 
 
  - EndGlobalSection
 
 
  - GlobalSection(ProjectConfigurationPlatforms) = postSolution
 
 
  - {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}.Debug|Win32.ActiveCfg =
  Debug|Win32
 
 
  - {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}.Debug|Win32.Build.0 =
 Debug|Win32
 
 
  - {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}.Release|Win32.ActiveCfg =
  Release|Win32
 
 
  - {40E4563C-EDCE-4F19-8A3F-F9497265ACEE}.Release|Win32.Build.0 =
  Release|Win32
 
 
  - {EADFA55E-6196-44CC-A496-1FDEC07DDE54}.Debug|Win32.ActiveCfg =
  Debug|Win32
 
 
  - {EADFA55E-6196-44CC-A496-1FDEC07DDE54}.Debug|Win32.Build.0 =
 Debug|Win32
 
 
  - {EADFA55E-6196-44CC-A496-1FDEC07DDE54}.Release

Re: Problem with RBTools 0.5

2013-04-04 Thread Steven MacLeod
|Win32


 - {933240B0-F445-49E7-AB25-7BAE6B3C458C}.Release|Win32.Build.0 =
 Release|Win32


 - EndGlobalSection


 - GlobalSection(SolutionProperties) = preSolution


 - HideSolutionNode = FALSE


 - EndGlobalSection


 -EndGlobal


 ---

 DEBUG:root:Making HTTP GET request to
 http://reviewboard.corp.good.com/api/review-requests/


 DEBUG:root:Making HTTP POST request to
 http://reviewboard.corp.good.com/api/review-requests/


 DEBUG:root:Making HTTP GET request to
 http://reviewboard.corp.good.com/api/review-requests/44580/diffs/


 Traceback (most recent call last):

 File C:\Python27\Scripts\rbt-script.py, line 8, in module

 load_entry_point('RBTools==0.5', 'console_scripts', 'rbt')()

 File
 C:\Python27\lib\site-packages\rbtools-0.5-py2.7.egg\rbtools\commands\main.py,
 line 99, in main

 File
 C:\Python27\lib\site-packages\rbtools-0.5-py2.7.egg\rbtools\commands\__init__.py,
 line 150, in run_from_argv

 File
 C:\Python27\lib\site-packages\rbtools-0.5-py2.7.egg\rbtools\commands\post.py,
 line 450, in main

 File
 C:\Python27\lib\site-packages\rbtools-0.5-py2.7.egg\rbtools\commands\post.py,
 line 324, in post_request

 File
 C:\Python27\lib\site-packages\rbtools-0.5-py2.7.egg\rbtools\api\transport\sync.py,
 line 289, in __getattr__

 AttributeError: 'ResourceList' object has no attribute 'upload_diff'


 On Fri, Mar 29, 2013 at 12:20 PM, Steven MacLeod ste...@smacleod.ca
 wrote:
  Hi Robert,
 
  What version of Review Board is there server running? Could you also
 provide
  the output of rbt post -d --server=http://my.review.board.url--p4-client=$c
  --p4-port=$p --username=myuser --password=mypassword %C
 
  The post-review script is still part of rbtools 0.5, so you can use that
 as
  a backup until we have this figured out.
 
  On Mar 29, 2013 11:01 AM, Robert Dailey rcdailey.li...@gmail.com
 wrote:
 
  No one will help?
 
  On Thu, Mar 28, 2013 at 3:54 PM, Robert Dailey 
 rcdailey.li...@gmail.com
  wrote:
   I am getting the following error uploading my review from a Perforce
   changelist. This impedes my day to day work, so I hope that someone
   can help me quickly. The error I get while trying to run 'rbt post'
   is:
  
   CRITICAL:root:'ResourceList' object has no attribute 'upload_diff'
  
   I have only transitioned from review-board to RBT 0.5, most of the
   commands are the same so I am only invoking a different process. I am
   using Python 2.7.3 on Windows 7 x64.
  
   Here is the command I'm running:
  
   rbt post --server=http://my.review.board.url --p4-client=$c
   --p4-port=$p --username=myuser --password=mypassword %C
  
   Note that the previous command I used was:
  
   post-review %C --server=http://my.review.board.url --p4-client=$c
   --p4-port=$p --username=myuser --password=mypassword
  
   The post-review command always worked fine for me. I installed RBTools
   0.5 using easy_install (logs below). Note that I run this as a custom
   tool in P4V, so %C actually maps to the Changelist number. $c maps to
   client name, $p is port, and so on.
  
   C:\Users\rdaileyeasy_install -U RBTools
   Searching for RBTools
   Reading http://pypi.python.org/simple/RBTools/
   Reading http://www.reviewboard.org/
   Reading http://downloads.reviewboard.org/releases/RBTools/0.3/
   Reading http://downloads.reviewboard.org/releases/RBTools/0.4/
   Reading http://downloads.reviewboard.org/releases/RBTools/0.5/
   Reading http://downloads.reviewboard.org/releases/RBTools/0.2/
   Reading http://www.review-board.org/
   Reading http://downloads.review-board.org/releases/
   Reading http://downloads.review-board.org/releases/RBTools/0.2/
   Best match: RBTools 0.5
   Downloading
  
 http://downloads.reviewboard.org/releases/RBTools/0.5/RBTools-0.5-py2.7.egg
   Processing RBTools-0.5-py2.7.egg
   Moving RBTools-0.5-py2.7.egg to c:\python27\lib\site-packages
   Removing rbtools 0.4.1 from easy-install.pth file
   Adding RBTools 0.5 to easy-install.pth file
   Installing post-review-script.py script to C:\Python27\Scripts
   Installing post-review.exe script to C:\Python27\Scripts
   Installing post-review.exe.manifest script to C:\Python27\Scripts
   Installing rbt-script.py script to C:\Python27\Scripts
   Installing rbt.exe script to C:\Python27\Scripts
   Installing rbt.exe.manifest script to C:\Python27\Scripts
  
   Installed c:\python27\lib\site-packages\rbtools-0.5-py2.7.egg
   Processing dependencies for RBTools
   Finished processing dependencies for RBTools
 
  --
  Want to help the Review Board project? Donate today at
  http://www.reviewboard.org/donate/
  Happy user? Let us know at http://www.reviewboard.org/users/
  -~--~~~~--~~--~--~---
  To unsubscribe from this group, send email to
  reviewboard+unsubscr...@googlegroups.com
  For more options, visit this group at
  http://groups.google.com/group/reviewboard?hl=en
  ---
  You received this message because you are subscribed to the Google
 Groups
  reviewboard group.
  To unsubscribe from this group

Re: Problem with RBTools 0.5

2013-03-29 Thread Steven MacLeod
Hi Robert,
What version of Review Board is there server running? Could you also
provide the output of rbt post -d --server=http://my.review.board.url
--p4-client=$c
--p4-port=$p --username=myuser --password=mypassword %C

The post-review script is still part of rbtools 0.5, so you can use that as
a backup until we have this figured out.

On Mar 29, 2013 11:01 AM, Robert Dailey rcdailey.li...@gmail.com wrote:

 No one will help?

 On Thu, Mar 28, 2013 at 3:54 PM, Robert Dailey rcdailey.li...@gmail.com
 wrote:
  I am getting the following error uploading my review from a Perforce
  changelist. This impedes my day to day work, so I hope that someone
  can help me quickly. The error I get while trying to run 'rbt post'
  is:
 
  CRITICAL:root:'ResourceList' object has no attribute 'upload_diff'
 
  I have only transitioned from review-board to RBT 0.5, most of the
  commands are the same so I am only invoking a different process. I am
  using Python 2.7.3 on Windows 7 x64.
 
  Here is the command I'm running:
 
  rbt post --server=http://my.review.board.url --p4-client=$c
  --p4-port=$p --username=myuser --password=mypassword %C
 
  Note that the previous command I used was:
 
  post-review %C --server=http://my.review.board.url --p4-client=$c
  --p4-port=$p --username=myuser --password=mypassword
 
  The post-review command always worked fine for me. I installed RBTools
  0.5 using easy_install (logs below). Note that I run this as a custom
  tool in P4V, so %C actually maps to the Changelist number. $c maps to
  client name, $p is port, and so on.
 
  C:\Users\rdaileyeasy_install -U RBTools
  Searching for RBTools
  Reading http://pypi.python.org/simple/RBTools/
  Reading http://www.reviewboard.org/
  Reading http://downloads.reviewboard.org/releases/RBTools/0.3/
  Reading http://downloads.reviewboard.org/releases/RBTools/0.4/
  Reading http://downloads.reviewboard.org/releases/RBTools/0.5/
  Reading http://downloads.reviewboard.org/releases/RBTools/0.2/
  Reading http://www.review-board.org/
  Reading http://downloads.review-board.org/releases/
  Reading http://downloads.review-board.org/releases/RBTools/0.2/
  Best match: RBTools 0.5
  Downloading
 http://downloads.reviewboard.org/releases/RBTools/0.5/RBTools-0.5-py2.7.egg
  Processing RBTools-0.5-py2.7.egg
  Moving RBTools-0.5-py2.7.egg to c:\python27\lib\site-packages
  Removing rbtools 0.4.1 from easy-install.pth file
  Adding RBTools 0.5 to easy-install.pth file
  Installing post-review-script.py script to C:\Python27\Scripts
  Installing post-review.exe script to C:\Python27\Scripts
  Installing post-review.exe.manifest script to C:\Python27\Scripts
  Installing rbt-script.py script to C:\Python27\Scripts
  Installing rbt.exe script to C:\Python27\Scripts
  Installing rbt.exe.manifest script to C:\Python27\Scripts
 
  Installed c:\python27\lib\site-packages\rbtools-0.5-py2.7.egg
  Processing dependencies for RBTools
  Finished processing dependencies for RBTools

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: python API: target_people access fails

2013-03-26 Thread Steven MacLeod
Hi,

This is an issue with Python versions  2.6.5. I have a fix for it which
will be part of the RBTools 0.5.1 release which is coming soon.


On Fri, Mar 22, 2013 at 6:13 PM, Steven MacLeod ste...@smacleod.ca wrote:

 What version of python are you running?


 On Fri, Mar 22, 2013 at 3:54 PM, A.M. age...@themactionfaction.comwrote:

 On Fri, 22 Mar 2013 15:25:48 -0400, Steven MacLeod ste...@smacleod.ca
 wrote:
  That looks correct. I'm not quire sure what's going wrong.
 
  Could you provide me with the JSON payload being used to create the
 review
  request resource. You can get this by running:
 
 
  review_requests = rb_conn.get_root().get_review_
  requests(time_added_from=time_added, status='pending', max_results=1)
 
  for rr in review_requests:
  print rr._resource._payload

 Sure- here is one sanitized parsed-to-dict JSON structure:

 {u'status': u'pending', u'last_updated': u'2013-03-22T18:39:57.056Z',
 u'description': u_, u'links': {u'diffs': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/diffs/', u'method':
 u'GET'}, u'repository': {u'href':
 u'https://reviewboard._.com/api/repositories/9/', u'method': u'GET',
 u'title': u'_'}, u'changes': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/changes/',
 u'method':
 u'GET'}, u'self': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/', u'method':
 u'GET'},
 u'update': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/', u'method':
 u'PUT'},
 u'last_update': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/last-update/',
 u'method': u'GET'}, u'reviews': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/reviews/',
 u'method':
 u'GET'}, u'draft': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/draft/', u'method':
 u'GET'}, u'file_attachments': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/file-attachments/',
 u'method': u'GET'}, u'submitter': {u'href':
 u'https://reviewboard._.com/api/users/_/', u'method': u'GET', u'title':
 u'_'}, u'screenshots': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/screenshots/',
 u'method': u'GET'}, u'delete': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/', u'method':
 u'DELETE'}}, u'target_groups': [], u'bugs_closed': [u'_'], u'changenum':
 None, u'target_people': [{u'href':
 u'https://reviewboard._.com/api/users/_/', u'method': u'GET', u'title':
 u'_'}, {u'href': u'https://reviewboard._.com/api/users/_/', u'method':
 u'GET', u'title': u'_'}, {u'href':
 u'https://reviewboard._.com/api/users/_/', u'method': u'GET', u'title':
 u'_'}], u'testing_done': u_, u'branch': u'2013-1', u'id': 2727,
 u'time_added': u'2013-03-22T16:49:32.368Z', u'summary': u'_', u'public':
 True}

 Company-specific information has been redacted with underscores.

 We are using ReviewBoard 1.7.5, in case that matters.

 Cheers,
 M

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.





-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: python API: target_people access fails

2013-03-22 Thread Steven MacLeod
That looks correct. I'm not quire sure what's going wrong.

Could you provide me with the JSON payload being used to create the review
request resource. You can get this by running:


review_requests = rb_conn.get_root().get_review_
requests(time_added_from=time_added, status='pending', max_results=1)

for rr in review_requests:
print rr._resource._payload


On Fri, Mar 22, 2013 at 1:55 PM, A.M. age...@themactionfaction.com wrote:

 Hello,

 I am trying to extract user names from the review_request.target_people but
 I receive an unexpected exception.

 review_requests =
 rb_conn.get_root().get_review_requests(time_added_from=time_added,

 status='pending',max_results=1)
 for rr in review_requests:
 for reviewer in rr.target_people:
 print reviewer

 Traceback (most recent call last):
   File rbpoller.py, line 129, in module
 send_rb_fresh_emails()
   File rbpoller.py, line 97, in send_rb_fresh_emails
 for reviewer in rr.target_people:
   File build/bdist.linux-x86_64/egg/rbtools/api/transport/sync.py, line
 142, in next
   File build/bdist.linux-x86_64/egg/rbtools/api/transport/sync.py, line
 162, in __getitem__
   File build/bdist.linux-x86_64/egg/rbtools/api/transport/sync.py, line
 71, in wrap
 TypeError: __init__() keywords must be strings


 My understanding is that the ResourceListField should be iterable. Am I
 doing something incorrectly?

 Cheers,
 M

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: python API: target_people access fails

2013-03-22 Thread Steven MacLeod
What version of python are you running?


On Fri, Mar 22, 2013 at 3:54 PM, A.M. age...@themactionfaction.com wrote:

 On Fri, 22 Mar 2013 15:25:48 -0400, Steven MacLeod ste...@smacleod.ca
 wrote:
  That looks correct. I'm not quire sure what's going wrong.
 
  Could you provide me with the JSON payload being used to create the
 review
  request resource. You can get this by running:
 
 
  review_requests = rb_conn.get_root().get_review_
  requests(time_added_from=time_added, status='pending', max_results=1)
 
  for rr in review_requests:
  print rr._resource._payload

 Sure- here is one sanitized parsed-to-dict JSON structure:

 {u'status': u'pending', u'last_updated': u'2013-03-22T18:39:57.056Z',
 u'description': u_, u'links': {u'diffs': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/diffs/', u'method':
 u'GET'}, u'repository': {u'href':
 u'https://reviewboard._.com/api/repositories/9/', u'method': u'GET',
 u'title': u'_'}, u'changes': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/changes/', u'method':
 u'GET'}, u'self': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/', u'method':
 u'GET'},
 u'update': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/', u'method':
 u'PUT'},
 u'last_update': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/last-update/',
 u'method': u'GET'}, u'reviews': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/reviews/', u'method':
 u'GET'}, u'draft': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/draft/', u'method':
 u'GET'}, u'file_attachments': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/file-attachments/',
 u'method': u'GET'}, u'submitter': {u'href':
 u'https://reviewboard._.com/api/users/_/', u'method': u'GET', u'title':
 u'_'}, u'screenshots': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/screenshots/',
 u'method': u'GET'}, u'delete': {u'href':
 u'https://reviewboard._.com/api/review-requests/2727/', u'method':
 u'DELETE'}}, u'target_groups': [], u'bugs_closed': [u'_'], u'changenum':
 None, u'target_people': [{u'href':
 u'https://reviewboard._.com/api/users/_/', u'method': u'GET', u'title':
 u'_'}, {u'href': u'https://reviewboard._.com/api/users/_/', u'method':
 u'GET', u'title': u'_'}, {u'href':
 u'https://reviewboard._.com/api/users/_/', u'method': u'GET', u'title':
 u'_'}], u'testing_done': u_, u'branch': u'2013-1', u'id': 2727,
 u'time_added': u'2013-03-22T16:49:32.368Z', u'summary': u'_', u'public':
 True}

 Company-specific information has been redacted with underscores.

 We are using ReviewBoard 1.7.5, in case that matters.

 Cheers,
 M

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: Creating a new comment, first_line not being used correctly?

2012-09-17 Thread Steven MacLeod
Hi,

first_line is in reference to the line numbers of the file diff resource (
http://www.reviewboard.org/docs/manual/1.6/webapi/2.0/resources/file-diff/).
This number doesn't correspond to either the left or right pane of the
diff, but a combination of the two. If you were to count down the rows of
the diff simultaneously, the row number the line of code falls on is the
value of first_line you are looking for. (i.e. in the file-diff resource,
look at the description of the lines field inside a chunk, the value
you are looking for is 1.Row number of the line in the combined
side-by-side diff. ).

You can find an example function which converts the line number from the
file to the row number here:
https://github.com/smacleod/ReviewBot/blob/490522cfcd2ec8ec3d5a953d0b225344e60e9ad3/bot/reviewbot/processing/review.py#L84

It uses a very slow algorithm though, so I only recommend looking at it to
make things clearer.

On Mon, Sep 17, 2012 at 7:47 AM, soorajchirag contact.sund...@gmail.comwrote:

 Hi,

 I am looking at the web page:
 http://www.reviewboard.org/docs/manual/1.6/webapi/2.0/resources/review-diff-comment-list/#webapi2.0-review-diff-comment-list-resource
  to
 know how to post comments using the API on an uploaded diff.

 The comment is getting posted by using the parameters specified on the
 page, but the first_line parameter doesn't seem to be working as expected.
 From the web page quoted above:

 first_line (required)  Integer  The line number the comment
 starts at.

 The issue:
 I am setting the first_line as the one where I want the comment to appear
 on the updated content (right hand column in the browser). I have verified
 that the request being sent to the reviewboard server has the correct
 first_line value. But when I view the review using the Web browser it
 appears at a different line number.

 The line number is not correct even if I assume that the first_line is
 based on the line numbers of the original content.

 Eg:
 Posted on line number 1594, displayed on:
 1. New content line number 1468
 2. Old content line number 1505

 If I use the Web browser to post the comment on the intended line (i.e.
 1594), I see that it uses first_line as 1720!

 I am running version 1.6.3.

 Is my understanding of the description of first_line incorrect? Any
 suggestions on how to go about figuring out the cause?

 Thanks,
 Sundeep

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: install RB extension

2012-09-13 Thread Steven MacLeod
Hi,

Extensions are not supported in version 1.6.1, support starts in version
1.7 which is currently in beta. Note that 1.7 is still unstable and not
recommended for production use yet.

Steve

On Thu, Sep 13, 2012 at 7:28 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Hi All,

 Presently i am using RB server 1.6.1 and I want to install reviewboard
 extension explicitly, is it recommended or supported for this version of RB
 server.

 Regards,
 Nilesh J

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: When i use post-review, always get the No supported repository could be accessed at the supplied url.

2012-08-07 Thread Steven MacLeod
It appears that post-review does not support Bazaar yet.

There is an issue for this in this tracker:
http://code.google.com/p/reviewboard/issues/detail?id=773

And some discussion about working on a patch (From May) can be found in
this mailing list thread:
https://groups.google.com/d/topic/reviewboard/lnaB7bnfc34/discussion

On Tue, Aug 7, 2012 at 9:41 PM, Kyoyeol Kim kimkyo...@gmail.com wrote:

 Hi, My name is Kyoyeol Kim. I'm from South Korean. I think ReviewBoard
 is amazing! but I got a problem.
 When i use post-review, always get the No supported repository could be
 accessed at the supplied url.
 Please help me.



 Problem
 (269)$ post-review
 No supported repository could be accessed at the supplied url.

 Infomation
 ReviewBoard : ver.1.6.1, http://10.90.38.82/reviews/
 RBTools : ver.0.4.1
 Bazaar : ver.2.1.4

 /bazaar/sample1 =repository

 /bazaar/sample1/trunk =branch

 /bazaar/sample1/.reviewboardrc

 (270)$ cat .reviewboardrc
 REPOSITORY= 'sample1/trunk'
 REVIEWBOARD_URL= 'http://10.90.38.82/reviews/'

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Installing the Web Hooks extension

2012-06-25 Thread Steven MacLeod
Hey Rodrigo,

I'm the author of the Web Hooks extension. I believe it should
be compatible with the latest 1.7 dev code, as I don't remember any changes
which would have broken it since it was committed.

As for not running a production site off of 1.7, the Web Hooks extension is
even less ready for production. To be honest, it is more of a prototype
which has had only minimal testing. Working to improve this extension isn't
currently one of my priorities, but I'd be more than happy to offer support
if you are interested in tinkering with it.

Steve

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Post-Commit in server side?

2012-03-08 Thread Steven MacLeod
I've never done this myself, but I think it's probably possible. SVN has a 
number of hook points where you can run scripts, including a post-commit 
hook. After a quick search I found this:

http://svnbook.red-bean.com/en/1.6/svn.reposadmin.create.html#svn.reposadmin.create.hooks

On Thursday, March 8, 2012 4:56:14 AM UTC-5, brucenan wrote:

 Does any solutions for post-commit in server? 
 I do not want to make developer commit codes manually. Does any hooks 
 in server's SVN can implement this function. 

 My steps: 

 1.Dev commit codes to SVN in sever 
 2.SVN automatically commit review request to Reviewboard 
 3.All can view request in Reviewboard Web.

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Extension's ModelAdmin and AdminSite

2012-02-12 Thread Steven MacLeod
After a design discussion with ChipX86, the plans have changed a little. 
Instead of allowing extensions to register with the Review Board admin 
site, I'm going to add support for extensions to have their own admin site 
instance.

By specifying a flag in your extension (similar to how you may set 
is_configurable = true), a separate admin site instance will be created for 
you, which you may register your ModelAdmins to. This will work by calling 
something like extension.admin_site.register(...)

I think this will cover what you are looking to do. I'll keep you posted

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Extension's ModelAdmin and AdminSite

2012-02-11 Thread Steven MacLeod
I came up against this issue myself recently.

Currently Djblets has nothing implemented to import the admin.py files
from extensions.

ChipX86 django has a function that scans all admin.py files in
INSTALLED_APPS and imports them, causing registers to happen

I've started adding something similar to Djblets extensions. I'll let
you know when it's up for review.

On Feb 9, 3:15 pm, Bart Celary bartlomiej.cel...@gmail.com wrote:
 Hello,
 Could you tell me what is the proper way of registering ModelAdmin
 classes for extensions. Should I create extra AdminSite instance for
 this purpose or use django.contrib.admin.site? I thought that just
 registering ModelAdmin classes in myext.admin module would be OK but
 it is not picked up by Django. I have added explicit import of admin
 module in urls:

 from django.conf.urls.defaults import include, patterns, url

 from myext.admin import admin

 urlpatterns = patterns('myext.views',
     url(r'^$', 'dashboard'),
     (r'^db/', include(admin.site.urls)),
 )

 and although it is working, there's always a list of all applications
 in both myext/db and admin/db but in admin/db the myext application
 urls are not valid. This doesn't look to good... Could you suggest a
 right way of doing this? Any help would be greatly appreciated.

 Thanks,
 Bartek

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en