Fix creating CC/VV diffs that have moved files.

Review Request #12893 — Created March 15, 2023 and submitted — Latest diff uploaded

Information

RBTools
release-4.x

Reviewers

Some code movement in the ClearCase/VersionVault client ended up breaking
the moved file case, where we tried to access a variable before defining it.
This wasn't immediately caught because my local test cases involved activities
that created a new file and then moved it, all in the same activity (which works
correctly).

Created an activity that moved an existing file. Saw the correct diff instead of
an RBTools crash.

Diff Revision 1

This is not the most recent revision of the diff. The latest diff is revision 2. See what's changed.

orig
1
2

Commits

First Last Summary ID Author
Fix creating CC/VV diffs that have moved files.
Some code movement in the ClearCase/VersionVault client ended up breaking the moved file case, where we tried to access a variable before defining it. This wasn't immediately caught because my local test cases involved activities that created a new file and then moved it, all in the same activity (which works correctly). Testing Done: Created an activity that moved an existing file. Saw the correct diff instead of an RBTools crash.
b8e38485e01e7f7941ee096bc1cbaadc92939b9d David Trowbridge
rbtools/clients/clearcase.py
Loading...