• 
      

    Unconditionally try to load the SCM client when TREES is defined.

    Review Request #14345 — Created Feb. 13, 2025 and submitted — Latest diff uploaded

    Information

    RBTools
    release-5.x

    Reviewers

    When someone is using the TREES setting for RBTools, it's common that
    they'll be using repository paths as the keys to that. In this case, a
    command like rbt patch --print would fail because it was trying to
    avoid initializing the scmtool, and then we never got the correct server
    URL.

    This makes it so if TREES is present in the configuration (and now
    also actually populated with data, since the default value will be {}),
    and we otherwise did not need the scmtool, we'll still attempt to
    initialize it. If we get it, then great, we can use it for TREES. If we
    don't, we ignore that and proceed as previous.

    • Ran unit tests.
    • Verified that when TREES was defined, we now were initializing the
      scmtool when running rbt patch --print.

    Commits

    Files