Fix a few issues with connecting to the legacy ClearCase backend.

Review Request #12305 — Created May 29, 2022 and submitted — Latest diff uploaded

Information

RBTools
master

Reviewers

A few bugs slipped in at the last moment which broke posting changes in ClearCase
repositories when using the legacy community-driven ClearCase backend on the server.
This change fixes them up:

  • The return for the diff lines got indented one space too far, which meant we
    were never returning it for the legacy case. This resulted in always getting a
    "there don't seem to be any diffs" error.
  • The backend has diffs_use_absolute_paths=False (by virtue of not changing the
    default). This means that it's expected to send a basedir along with any diffs.
    The old implementation of the repository info structure set the base_path to
    be the vob tag, but this got lost when we redid everything. Since we only need
    this for the legacy implementation, I've made it so we set base_path after we
    negotiate with the server about which backend we're talking to.
  • The filenames listed in the diff header used to contain the vob tag. This of
    course duplicates information which is already present in both the server-side
    configuration of the repository AND the basedir, but not including it in the
    filenames causes the diff parser to hit an infinite loop.

Posted changes against a repository configured with the legacy backend (along
with some changes to that backend which will be coming in a separate review
request).

Commits

Files

    Loading...