Add CLI11 to Tf

[dgovil]

Description of Change(s)

This PR adds CLI11 to Tf under the pxr/base/tf/pxrCLI11 in the PXR_NS namespace.

This splits out the addition of the library in #2090 per the notes there.

• I have verified that all unit tests pass with the proposed changes

• I have submitted a signed Contributor License Agreement

[sunyab]

Filed as internal issue #USD-7796

[sunyab]

Hey @dgovil, just a heads-up: we've merged this PR internally but during review wound up making some changes to more fully isolate the CLI11 in Tf by adding extra compile-time checks and a hardcoded pxr_CLI namespace along with the normal pxr namespace. These are to guard against the unlikely-but-possible cases where USD's embedded CLI11 might conflict with another version of CLI11 in user code.

Practically, this means the other PRs that build against Tf's CLI11 will need to have something like using namespace pxr_CLI added. I'm adding that in as I merge the other PRs, but wanted to let you know FYI.

[dgovil]

Thanks for the headsup, Sunya! It's much appreciated.

…

On Wed, 7 Dec 2022 at 15:44, Sunya Boonyatera ***@***.***> wrote: Hey @dgovil <https://github.com/dgovil>, just a heads-up: we've merged this PR internally but during review wound up making some changes to more fully isolate the CLI11 in Tf by adding extra compile-time checks and a hardcoded pxr_CLI namespace along with the normal pxr namespace. These are to guard against the unlikely-but-possible cases where USD's embedded CLI11 might conflict with another version of CLI11 in user code. Practically, this means the other PRs that build against Tf's CLI11 will need to have something like using namespace pxr_CLI added. I'm adding that in as I merge the other PRs, but wanted to let you know FYI. — Reply to this email directly, view it on GitHub <#2107 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABB4XUQAA3DEICZJSGCR5QTWMEOO7ANCNFSM6AAAAAASNU3DJU> . You are receiving this because you were mentioned.Message ID: ***@***.***>