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
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.
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.
On Oct 5, 2009 10:00pm, David Sitsky <firstname.lastname@example.org> 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?