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 with Clearcase > > 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 a ClearCase repository 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 > > clearcase extended 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 with Clearcase right now, as I run a rogue Clearcase-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 the Clearcase SCMTool 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 no ClearCase support 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, Clearcase is 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 support ClearCase "out of the box". > > > > > > > However once I had ReviewBoard 1.0 installed on my Windows server > > > > > > the > > > > > > config section's Scmtools area didn't mention ClearCase. > > > > > > > After a little Googling I found that you need to add the ClearCase > > > > > > 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 not ClearCase: > > > > >http://www.review-board.org/docs/manual/dev/admin/management/reposito... > > > > > > > Is ClearCase actually supported?? > > > > > > > Reading the post-review script it also appears that the clearcase > > > > > > code > > > > > > 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 gotten ClearCase and ReviewBoard to > > > > > > play > > > > > > nice with each other in a Windows (non-Cygwin) environment help me > > > > > > out > > > > > > here. > > > > > > > Thanks > > --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---
