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

Enable USD version check for components distributed separately #846

Merged
merged 2 commits into from
Oct 20, 2020

Conversation

kxl-adsk
Copy link

USD plugins that are distributed in separate packages should perform runtime USD version check to avoid potential crashes caused by version mismatch. This PR adds:

  • MAYA_USD_VERSION and MAYAUSD_VERSION env variables via MayaUSD mod file
  • MAYA_PXR_PLUGINPATH_NAME env variable and a service that will append <USD_VERSION> to each path and call RegisterPlugins for them. This service runs only once per Maya session and is triggered by either MayaUSD plugin or mtoh (whichever is loaded first). For example, MayaUSD distribution with USD 20.11 will attempt registration of each path in MAYA_PXR_PLUGINPATH_NAME with the 2011 subdirectory.

@kxl-adsk kxl-adsk requested a review from seando-adsk October 18, 2020 01:01
@kxl-adsk
Copy link
Author

Still need to add a test, but wanted to open it to collect feedback.

Copy link
Collaborator

@seando-adsk seando-adsk left a comment

Choose a reason for hiding this comment

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

Never mind. I see you fixed it.

@kxl-adsk kxl-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Oct 20, 2020
@kxl-adsk kxl-adsk merged commit dbd9684 into dev Oct 20, 2020
@kxl-adsk kxl-adsk deleted the kxl-adsk/MAYA-107462/enable_version_check branch October 20, 2020 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants