The requirements to use post-review with clearcase are the same as with any other version control system. The only difference is you pass a list of filenames as arguments to the script that you want to review:
python post-review --server=http://example.com/ file1.c file2.c file3.c If post-review doesn't work, easy_install rbtools should install the python dependencies. Dan On Jul 17, 8:11 pm, 吴开春 <wukaic...@gmail.com> wrote: > 请教一下，你装起来用了没有？ > > On Jul 17, 10:23 pm, Dan Savilonis <d...@n-cube.org> wrote: > > > > > It is not required. post-review simply includes support for cygwin > > (i.e. path translation since cleartool does not understand cygwin > > paths). My example used sed and xargs to make it easier, so that's the > > only reason I suggested having the tools in your path. This feature > > can eventually be added to post-review to automatically collect > > checked out files. > > > Dan > > > On Jul 17, 6:50 am, 夏勇杰 <notox...@gmail.com> wrote: > > > > Hi Dan, > > > > I have some questions about post review. > > > > Should we install cygwin for post-review in order to make it work with > > >Clearcase? > > > > On Jul 17, 9:32 am, Dan Savilonis <d...@n-cube.org> wrote: > > > > > I went through a fresh installation of reviewboard withClearcase > > > > today and recorded the steps. > > > > > Here's what you need to do to get up and running. For the purpose of > > > > this test, I used a dynamic view and was running the server on the > > > > same machine as I had the view and did a review of checked out files. > > > > > 1) Add reviewboard.scmtools.clearcase.ClearCaseTool in the admin > > > > interface (Home > Scmtools > Tools), if using ReviewBoard 1.0. This > > > > step is not necessary if you're running svn revision >= r2041. > > > > 2) Add aClearCaserepository via the admin interface, and specify a > > > > path to your view (I used the top of the view, not the vob). > > > > 3) Apply the patch fromhttp://reviews.review-board.org/r/906/toyour > > > > post-review script. > > > > 4) Edit post-review ClearCaseClient class. Change cc_rootpath to be > > > > identical to what you typed into the admin interface, change viewtype > > > > to 'dynamic' (if you're trying this with a dynamic view, of course) > > > > 4) Check out a file in the view and make a change > > > > 4) Run post-review in your view and pass a list of the files that > > > > you've checked out after the options. Alternatively if you're lazy > > > > like me, you can do something like (on windows, with cygwin, msys, > > > > etc, in your vob directory): > > > > > cleartool lsco -cview -me -recurse -short | sed "s/\\\\/\//g" | xargs > > > > python post-review --server=http://example.com/ > > > > > I tried using a snapshot view but it didn't work immediately for me > > > > and I haven't debugged it. It looked like it was trying to use a > > > >clearcaseextended path. I assumed you could only use that notation > > > > within a dynamic view, but I'm probably wrong. > > > > > Dan > > > > > On Jul 10, 5:38 pm, Dan Savilonis <d...@n-cube.org> wrote: > > > > > > I will try to write up my experience as I can. (Un)fortunately, we're > > > > > not using RB withClearcaseright now, as I run a rogueClearcase-to- > > > > > git system, and use RB with git :) > > > > > > At minimum right now, we need to get the the line from Bartek's patch > > > > > to add split_line=TRUE committed or post-review will just fail for end > > > > > users. If you can get theClearcaseSCMTool to be added to the list of > > > > > scmtools by default, that would be great, too. I have no idea how to > > > > > do that, though. > > > > > > Dan > > > > > > On Jul 10, 3:29 pm, Christian Hammond <chip...@chipx86.com> wrote: > > > > > > > Hi djs, > > > > > > > Since David and I have noClearCasesupport or really test > > > > > > infrastructure, > > > > > > would I be able to convince you to write some entries for our docs > > > > > > on > > > > > > setting it up and using it? Nothing major, but it would be good if > > > > > > people > > > > > > were able to get it working easier than they are. > > > > > > > Christian > > > > > > > -- > > > > > > Christian Hammond - chip...@chipx86.com > > > > > > Review Board -http://www.review-board.org > > > > > > VMware, Inc. -http://www.vmware.com > > > > > > > On Fri, Jul 10, 2009 at 7:22 AM, djs <d...@n-cube.org> wrote: > > > > > > > > Hi Oliver, > > > > > > > > Yes,Clearcaseis in fact supported. We have tested out setting it > > > > > > > up > > > > > > > running on Windows here and were able to get it to working. The > > > > > > > path > > > > > > > should be the location of a dynamic view and you don't need > > > > > > > cygwin to > > > > > > > use it. There is code in post-review to support using cygwin or > > > > > > > the > > > > > > > regular cmd.exe on Windows. > > > > > > > > We did have to modify one line of code in post-review to get it > > > > > > > to run > > > > > > > successfully, at which point we were able to run post-review and > > > > > > > pass > > > > > > > a list of files are arguments. This uploaded a diff of the > > > > > > > checked out > > > > > > > files against their base revisions. I believe you also need to > > > > > > > modify > > > > > > > the hard-coded location in the post-review script for it to work. > > > > > > > > I will try to submit a patch in a few days to fix post-review. > > > > > > > Let me > > > > > > > know if you have any luck setting it up. > > > > > > > > Dan > > > > > > > > On Jul 10, 1:24 am, Oliver Jones <orjo...@gmail.com> wrote: > > > > > > > > According to the ReviewBoard 1.0 release notes ReviewBoard is > > > > > > > > supposed > > > > > > > > to supportClearCase"out of the box". > > > > > > > > > However once I had ReviewBoard 1.0 installed on my Windows > > > > > > > > server the > > > > > > > > config section's Scmtools area didn't mentionClearCase. > > > > > > > > > After a little Googling I found that you need to add > > > > > > > > theClearCase > > > > > > > > SCMTool manually into the DB. That I have done. > > > > > > > > > Now I assume I need to add a repository. However I have no > > > > > > > > idea what > > > > > > > > to enter into the "Path" configuration item when adding the > > > > > > > > repo. > > > > > > > > > This page contains info on determining the path with SVN, Git, > > > > > > > > etc, > > > > > > > > but notClearCase: > > > > > > >http://www.review-board.org/docs/manual/dev/admin/management/reposito... > > > > > > > > > IsClearCaseactually supported?? > > > > > > > > > Reading the post-review script it also appears that > > > > > > > > theclearcasecode > > > > > > > > assumes you're using Cygwin. We do not have Cygwin installed > > > > > > > > here. > > > > > > > > And getting it installed could be difficult. > > > > > > > > > It also seems to hard code the vob location. > > > > > > > > > Can someone who has actually gottenClearCaseand ReviewBoard to > > > > > > > > play > > > > > > > > nice with each other in a Windows (non-Cygwin) environment help > > > > > > > > me out > > > > > > > > here. > > > > > > > > > Thanks- Hide quoted text - > > > - Show quoted text - --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "reviewboard" group. To post to this group, send email to reviewboard@googlegroups.com 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 -~----------~----~----~----~------~----~------~--~---
