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/to your > 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 -~----------~----~----~----~------~----~------~--~---
