forcing post-review to disregard current working directory

2013-03-19 Thread Kudrettin Güleryüz
Hello, I have cloned the latest RBTools from github. After I make changes to postreview.py regarding perforce, I'd like to test my changes but postreview.py assumes I'll be submitting reviews regarding RBTools itself. Can I override this behavior? If not please suggest your development workflows.

Re: post-review created 2 reviews for the same change list number

2013-03-19 Thread Steve
Any thoughts on the duplicate review problem? It's a concern because we're currently using RB 1.5.5 with an in-house perl script for posting reviews. We plan to upgrade soon to 1.7.6 so I'm migrating our users to the post-review script. This duplicate review occurred within a few days of

Re: post-review created 2 reviews for the same change list number

2013-03-19 Thread Kudrettin Güleryüz
Hi, Can there be two different Perforce servers involved with the two submissions? one submission using perforce-main vs another using perforce-backup for instance... Kudret On Tue, Mar 19, 2013 at 10:45 AM, Steve seide.al...@gmail.com wrote: Any thoughts on the duplicate review problem?

Re: post-review created 2 reviews for the same change list number

2013-03-19 Thread Steve
That may be it. I forgot we added a perforce sandbox server to the repository list for testing and they are at the top of the repository list. I just removed them. Thanks! --Steve On Tuesday, March 19, 2013 8:17:17 AM UTC-7, Kudret Güler wrote: Hi, Can there be two different Perforce

Re: RBTools 0.5 released!

2013-03-19 Thread Christian Hammond
On Tue, Mar 19, 2013 at 6:35 AM, Stephen Gallagher step...@gallagherhome.com wrote: On 03/19/2013 04:43 AM, Christian Hammond wrote: Hi everyone, The new RBTools 0.5 is out. This is a major release, led by our own Steven MacLeod, and is, to me, the most exciting release of RBTools

Re: RBTools 0.5 released!

2013-03-19 Thread Matthew Woehlke
On 2013-03-19 04:43, Christian Hammond wrote: The new RBTools 0.5 is out. This is a major release, led by our own Steven MacLeod, and is, to me, the most exciting release of RBTools we've put out. There's a bunch of new command line scripts and a brand new Python API that you can use to talk to

Re: RBTools 0.5 released!

2013-03-19 Thread Christian Hammond
No, we don't have that yet, but I think that'd be a great addition, and it's something I certainly want. Something that can pull out data, serialize it in some form for easy parsing. We started with a small set of tools for the initial release. I expect that to grow. Christian -- Christian

Re: forcing post-review to disregard current working directory

2013-03-19 Thread Matthew Woehlke
On 2013-03-19 10:16, Kudrettin Güleryüz wrote: I have cloned the latest RBTools from github. After I make changes to postreview.py regarding perforce, I'd like to test my changes but postreview.py assumes I'll be submitting reviews regarding RBTools itself. Can I override this behavior? If not

Re: forcing post-review to disregard current working directory

2013-03-19 Thread Christian Hammond
You can also run sudo setup.py develop to get your custom tree's versions to be in your path, and then cd to where you want to actually post the review request. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. -

Issue 2933 in reviewboard: Trim blank lines from the end (and beginning?) of review comments

2013-03-19 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 2933 by jamesd...@gmail.com: Trim blank lines from the end (and beginning?) of review comments http://code.google.com/p/reviewboard/issues/detail?id=2933 *** For customer support, please post to

Issue 2934 in reviewboard: demo.reviewboard.org/reviews.reviewboard.org require email addresses but don't hide them by default

2013-03-19 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 2934 by jamesd...@gmail.com: demo.reviewboard.org/reviews.reviewboard.org require email addresses but don't hide them by default http://code.google.com/p/reviewboard/issues/detail?id=2934 What version are you