Summary:

comparing clearcase label is not supported

Review Request #5155 — Created Dec. 23, 2013 and discarded April 27, 2014, 2:44 p.m.

Information
Owner:	delyn
Repository:	RBTools
Branch:	master
Bugs:	3174
Depends On:
Reviewers
Groups:	rbtools
People:	jan.koprowski

Description

currently tracking changes in a branch or several revisions are supported. However since other SCM, clearcase proposes a system of label/tag ie lbtype
It is very useful to compare a clearcase directory between two labels or between one label and current checked in code

this has been implemented through --revision-range option. When it does not contains clearcase revision token "@@", we consider requesting a diff between labels
following are supported:
--revision-range=label1 # compares recursively current directory ie .@@/label1 with current checked in files ie .@@.../LATEST (a temporary lbtype is created to perform this diff efficiently
--revision-range="label1;label2" # compares recursively current directory ie .@@/label1 with .@@/label2

co-author:
Christophe RABU

Testing Done

$ cd /vobs/vobtag/subdir or /vobs/vobtag

  post-review --revision-range=label1

OK

$ post-review --revision-range="label1;label2"

OK

$ post-review --revision-range="label1;label2;something"

KO

Change Summary:

add co-author

Description:

		currently tracking changes in a branch or several revisions are supported. However since other SCM, clearcase proposes a system of label/tag ie lbtype
		It is very useful to compare a clearcase directory between two labels or between one label and current checked in code

		this has been implemented through --revision-range option. When it does not contains clearcase revision token "@@", we consider requesting a diff between labels
		following are supported:
		--revision-range=label1 # compares recursively current directory ie .@@/label1 with current checked in files ie .@@.../LATEST (a temporary lbtype is created to perform this diff efficiently
		--revision-range=label1:label2 # compares recursively current directory ie .@@/label1 with .@@/label2

		there is a known issue when comparing rootdir of a vobtag, .@@/ is required for 2 parts of this option
		--revision-range=.@@/label1:.@@/label2
	+
	+	co-author:
	+	Christophe RABU

Depends On:

-	~~5154 - cannot compare several couple of clearcase file revision using --revision-range~~

Change Summary:

add Jan

People:

+	jan.koprowski

Change Summary:

restore ";" as default delimiter see https://reviews.reviewboard.org/r/5158

check checkedout files before trying to set a label

return an error id revision_range option is not recognized

change some typos

Description:

		currently tracking changes in a branch or several revisions are supported. However since other SCM, clearcase proposes a system of label/tag ie lbtype
		It is very useful to compare a clearcase directory between two labels or between one label and current checked in code

		this has been implemented through --revision-range option. When it does not contains clearcase revision token "@@", we consider requesting a diff between labels
		following are supported:
~		--revision-range=label1 # compares recursively current directory ie .@@/label1 with current checked in files ie .@@.../LATEST (a temporary lbtype is created to perform this diff efficiently
~		--revision-range=label1:label2 # compares recursively current directory ie .@@/label1 with .@@/label2
	~	--revision-range=label1 # compares recursively current directory ie .@@/label1 with current checked in files ie .@@.../LATEST (a temporary lbtype is created to perform this diff efficiently
	~	--revision-range="label1;label2" # compares recursively current directory ie .@@/label1 with .@@/label2
-
-		there is a known issue when comparing rootdir of a vobtag, .@@/ is required for 2 parts of this option
-		--revision-range=.@@/label1:.@@/label2

		co-author:
		Christophe RABU

Testing Done:

~		cd /vobs/vobtag/subdir
~		post-review --revision-range=label1
~		post-review --revision-range=label1:label2
~
~		cd /vobs/vobtag
~		post-review --revision-range=.@@/label1
~		post-review --revision-range=.@@/label1:.@@/label2
	~	$ cd /vobs/vobtag/subdir or /vobs/vobtag
	~	post-review --revision-range=label1
	~	OK
	~	$ post-review --revision-range="label1;label2"
	~	OK
	~	$ post-review --revision-range="label1;label2;something"
	~	KO

Diff:

Revision 2 (+242 -19)

Show changes

rbtools/clients/clearcase.py

We've done a fair amount of work on the RBTools revision syntax (passing in revisions as arguments instead of with --revision-range). Could you update this patch to fit in with the current code on master?

You'll need to support the "lbtype:" prefix in parse_revision_spec, and then handle the diffing between labels in diff.

Status: Discarded

Change Summary:

Obsolete without updates.