Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MAYA-129258 - MayaUsd: remove support for USD < v21.11 #3128

Merged

Conversation

seando-adsk
Copy link
Collaborator

MAYA-129258 - MayaUsd: remove support for USD < v21.11

  • Commit 1: cmake changes to remove old USD versions
  • Commit 2: Code changes to remove USD versions < 21.11
  • Commit 3: USD version is always at least 21.11 now.

Some notes about the changes:

  • There are puposely 3 commits here. You might want to review each one separately as they each deal with a specific set of changes (see the 3 comments above for what each commit is).
  • We now only support USD version >= 21.11.
  • Because of if PXR_VERSION removal, sometimes an include might have moved - this is because of clang-format rules.
  • ENABLE_RENDERTAG_VISIBILITY_WORKAROUND was removed since it was only set for < 2008 which we don't support now.

* cmake changes to remove old USD versions
* Code changes to remove USD versions < 21.11
* USD version is always at least 21.11 now.
@seando-adsk seando-adsk added the build Related to building maya-usd repository label May 30, 2023
@@ -66,6 +66,7 @@ if(CMAKE_UFE_V2_FEATURES_AVAILABLE)
UsdUndoInsertChildCommand.cpp
UsdUndoReorderCommand.cpp
UsdUndoSetKindCommand.cpp
#UsdUndoMaterialCommands.cpp
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick question. Seems like L202 to L208 are already commented out. Do we still need this file?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes we still need this file. In another set of changes (different pull-request) I have removed all the commented lines.

@seando-adsk seando-adsk merged commit c72d87b into dev Jun 6, 2023
@seando-adsk seando-adsk deleted the donnels/MAYA-129258/remove_support_for_usd_less_than_21_11 branch June 6, 2023 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to building maya-usd repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants