Re: post-review failing to work on CVS because I have cleartool with a view set

2010-01-19 Thread Chris Clark
Christian Hammond wrote: What you could do now is modify postreview.py so that the CVSClient instance is created before the ClearCaseClient. That would guarantee that CVS would check first. An easy technique for dealing with this is to add another definition for SCMCLIENTS, this way if the

Re: post-review failing to work on CVS because I have cleartool with a view set

2010-01-19 Thread Chris Clark
David McCloskey wrote: But Another problem is appearing now. Once it tries to do the post, I get an error that the connection was refused on port 2401. I'm guessing this is because we're using ssh tunnels only to connect to our CVS servers. Is there some special way to configure my repository

post-review failing to work on CVS because I have cleartool with a view set

2010-01-18 Thread David McCloskey
I'm in a directory with a checkout trying to do a post-review with CVS, but post-review is tripping up on a clearcase check: svn info git rev-parse --git-dir cleartool pwv -short repository info: Path: /view/DEFAULT/vobs/, Base path: /view/DEFAULT/vobs/, Supports changesets: False Traceback

Re: post-review failing to work on CVS because I have cleartool with a view set

2010-01-18 Thread Christian Hammond
This is a large part of why we need to move the clearcase configuration out into a .reviewboardrc... What you could do now is modify postreview.py so that the CVSClient instance is created before the ClearCaseClient. That would guarantee that CVS would check first. Christian -- Christian