Thanks for the quick response.

Yes. I got it to work by changing the Clearcase retrieve function to return 1 if the $error_msg eq ""

return 1 if ( $error_msg eq "" ); --> added to the ClearcaseDynamic.pm
return $error_msg;

I am not sure if this is the correct way to handle this..I am seeing inconsistency in handling error condition and the return values in the repository implementations for retrieve function.

For eg:

in Perforce.pm, we call die in case of error and there is no eval block to catch the die
and in success scenario the function doesn't return any value to the caller. The return value is assumed to be 1

in ClearcaseDynamic.pm we call die/eval in some places, reuse $error_msg variable in case of error while closing the view and
mask any error when trying to retrieve the clearcase data.

I started looking at codestriker today and I am really very impressed. Thanks for the fantastic tool.


thanks
Ramesh


On Oct 5, 2009 10:00pm, David Sitsky <david.sitsky@gmail.com> wrote:
> > Shouldn't  $error_msg in lib/Codestriker/Repository/ClearCaseDynamic.pm
>
> > initialized to undef  instead of empty string?
>
>
>
> It appears that way.  Does this fix your issue if you make this change?
>
>
>
> Cheers,
>
> David
>