Share

Codestriker: collaborative code reviewer

Tracker: Patches

5 Tag support for Clearcase dynamic view - ID: 2825336
Last Update: Comment added ( tobimie )

The Tag support clearcase dynamic view support has been added.
The start-tag and end-tag can be provided to fetch the file info directly
from clearcase.
No need for getting the diff of two version and then uploading it while
creating Topic.

Changes done: ClearCaseDynamic.pm
Added functionality for getDiff()


Lakshmivaragan ( lakshmivaragan ) - 2009-07-22 12:10

5

Open

None

Nobody/Anonymous

None

None

Public


Comments ( 2 )




Date: 2009-11-04 09:23
Sender: tobimie

Hi Lakshmivaragan,

thanks for this useful contribution. However, it appears to me that it has
some minor quirks.

1.) getDiff() in ClearCaseDynamic.pm should use CodeStriker::isWindows()
in order to determine if its running on windows, because CtCmd::setview
doesn't work on windows (see retrieve() on how it is done there)

2.) if only on tag is specified the getDiff() method should not return a
diff to the elements predecessor instead it should pull the entire element
from clearcase (this guarantees the same behaviour as in Subversion.pm for
example)

Thanks!

Tobias


Date: 2009-10-17 23:34
Sender: sitsProject Admin

Hi Lakshmivaragan,

Thanks for your contribution, and my apologies for the delay in replying
to you. I have put this into Codestriker now - with some minor
modifications.

Could you download 1.9.10 test 1, and update the ClearcaseDynamic.pm
module from here:

http://codestriker.cvs.sourceforge.net/viewvc/*checkout*/codestriker/codestriker/lib/Codestriker/Repository/ClearCaseDynamic.pm?revision=1.9

Please let me know how you go. I have updated the CHANGELOG file as well
with your contribution (that's not in the test 1 release though),

Thanks.

Cheers,
David


Log in to comment.

Attached File ( 1 )

Filename Description Download
ClearCaseDynamic.pm Tag support for Clearcase dynamic view Download

Change ( 1 )

Field Old Value Date By
File Added 336081: ClearCaseDynamic.pm 2009-07-22 12:10 lakshmivaragan