Hi Dan,
I have done the changes to make it work with Clearcase. I have
followed all the steps you have mentioned. I am using dynamic
clearcase view only. When I run the post-review script, it hangs when
it tries to post the diff content. But I can see the diff content if I
use output-diff option for post-review. Following are the output taken
from post-review script.
karthik-wx> ct lsco -cview -me -short -rec | sed "s/\\\\/\//g" | xargs
python C:/Python25/Scripts/post-review --server="http://localhost:
8082" --submit-as="karthik" --password="mywork" –d
C:\Python25\lib\site-packages\rbtools-0.2beta1-py2.5.egg\EGG-INFO
\scripts\post-review
>>> cleartool pwv -short
>>> repository info: Path: /sw0, Base path: /sw0, Supports changesets: False
>>> cleartool desc -pre K:\sw0\software\code\include\techMgr.h
>>> Looking for 'localhost /' cookie in C:\Documents and Settings\karthik\Local
>>> Settings\Application Data\.post-review-cookies.txt
>>> Cookie file loaded, but no cookie for this server
==> Review Board Login Required
Enter username and password for Review Board at http://localhost:8082/
>>> Logging in with username "karthik"
>>> HTTP POSTing to http://localhost:8082/api/json/accounts/login/:
>>> {'username': 'karthik', 'password': '**************'}
>>> Logged in.
>>> Attempting to create review request for None
>>> Submitting the review request as karthik
>>> HTTP POSTing to http://localhost:8082/api/json/reviewrequests/new/:
>>> {'repository_path': '/sw0',
'submit_as': 'karthik'}
>>> Review request created
>>> Attempting to set field 'description' to 'karthik_view_9.0BugFix
VIEWTYPE: dynamic
' for review request '6'
>>> HTTP POSTing to http://localhost:8082/api/json/reviewrequests/6/draft/set/:
>>> {'description': 'karthik_view_9.0BugFix\nVIEWTYPE: dynamic\n'}
>>> HTTP POSTing to
>>> http://localhost:8082/api/json/reviewrequests/6/draft/save/: {}
>>> Review request draft saved
>>> Uploading diff, size: 548
>>> HTTP POSTing to http://localhost:8082/api/json/reviewrequests/6/diff/new/:
>>> {'basedir': '/sw0'}
---- It hangs after this line ---
Please let me know If I miss any configuration or something.
Thanks,
Karthik
On Jul 18, 6:05 am, Dan Savilonis <d...@n-cube.org> wrote:
> 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
-~----------~----~----~----~------~----~------~--~---
