1516: ClearCase configuration in post-review should move to .reviewboardrc

locomo******@gmai***** (Google Code) (Is this you? Claim this profile.)
April 4, 2011
1929
What's the URL of the page containing the problem?
http://www.reviewboard.org/docs/manual/dev/users/tools/post-review/#clearcase

Describe the problem and any recommendations below.
The link suggests that clearcase support "requires some initial setup to be
used. Edit the cc_rootpath variable in the ClearcaseClient class to point
to a view from which you want to create a review request. "

However, upon installing post-review to my server, I get
/usr/bin/post-review and an egg (zipped) file at
/usr/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg. It's not clear
to me where I edit the cc_rootpath variable. Should I unzip the egg file
and do my edit there?

Please provide any additional information below.
Similarly, if I want to do "internal post-review configuration", as from
the link below:
http://www.reviewboard.org/docs/manual/dev/users/tools/post-review/#internal-post-review-configuration
I believe I need to modify the source. Should I unzip the egg file and edit
there?

Thanks for clarification.
#1 locomo******@gmai***** (Google Code) (Is this you? Claim this profile.)
Ok, I can install RBTools with -Z option. So, should I edit the python code in the
egg directory to set up clearcase and reviewboard server configuration?
chipx86
#2 chipx86
Right now, yeah, that's what you need to do. By 0.2 final, I'd like this to move to
.reviewboardrc.
  • +Confirmed
  • -Component-Docs
    +Component-RBTools
    +Milestone-RBTools-Release1.0
    +Priority-High
  • +ClearCase configuration in post-review should move to .reviewboardrc
david
#4 david
Fixed in reviewboard 7553ec1 and rbtools 47fb2ce. Thanks!
  • -Confirmed
    +Fixed