I would love a patch to move that functionality out into a .reviewboardrc. I haven't done it myself due to lack of a Clearcase server to test with (though if anybody wants to donate a license, we can get some automated tests for stuff going :).
Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Jan 15, 2010 at 7:37 AM, Dan Savilonis <d...@n-cube.org> wrote: > Hi Sassan, > > The latest docs have some instructions on how to use post-review with > Clearcase. In fact, it's the only way to post a review with Clearcase. > > http://www.reviewboard.org/docs/manual/dev/users/tools/post-review/ > > I wrote it using some unix-ish examples, but I actually tested the > instructions under Windows. post-review's Clearcase support is fairly > crude, but it will work for reviewing modified code in a view that has > not been checked in. > > Note that you actually need to modify post-review.py to make it work > in your environment, so I recommend pulling it from git and running > setup.py develop. If someone wants to just add these hard-coded values > as options, this won't be necessary anymore. > > Dan > > On Jan 14, 7:07 pm, Sassan <sassan...@verifone.com> wrote: > > Does anyone know what is and is not currently supported for ClearCase > > as an SCM tool on Windows? > > > > For example I see instructions for posting commited code reviews using > > post-review.exe under ClearCase section inhttp:// > www.reviewboard.org/docs/manual/1.0/users/tools/post-review/ > > > > Does the "Todo" mean it is only on the wish list and not yet > > implemented? > > > > I also see a --label=<LABEL> under ClearCase options for post- > > review.exe but I am not clear as to how a single label can designate a > > diff of two distinct revisions. > > > > I tried this anyway and I got:: > > > > Traceback (most recent call last): > > File "C:\Python25\Scripts\post-review-script.py", line 8, in > > <module> load_entry_point('RBTools==0.2beta2', 'console_scripts', > > 'post-review')() > > File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg > > \rbtools\postreview.py", line 2509, in main > > File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg > > \rbtools\postreview.py", line 954, in diff_label > > File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg > > \rbtools\postreview.py", line 947, in diff > > File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg > > \rbtools\postreview.py", line 811, in get_extended_namespace > > File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg > > \rbtools\postreview.py", line 790, in get_previous_version > > ValueError: too many values to unpack > > > > Do you know how I get to the postreview.py script inside the rbtools > > egg to modify it? > > > > On the other hand it is easy enough for me to generate the diffs > > myself out of ClearCase, but is there any way for me post these > > directly into the RB server ? > > > > Maybe by passing them to post-review.exe but only to be posted to the > > RB server ? > > > > Thanks, > > > > Sassan > > -- > 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<reviewboard%2bunsubscr...@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
