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

Fixed a compile error when UFE_SCENE_SEGMENT_SUPPORT isn't available. #3205

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

jjzanin-SMS
Copy link

I'm not sure if this is the correct fix, but there is a compile error when the header file "ufe/sceneSegmentHandler.h" isn't available. I'm assuming it's because our studio is using Maya 2023 and this feature is available in the next version.

Fixed a compile error when UFE_SCENE_SEGMENT_SUPPORT isn't available.
@seando-adsk
Copy link
Collaborator

@jjzanin-SMS Thank you for submitting a pull-request to fix this. I believe the problem here is that that sceneSegmentHandler.h file was added to Ufe v3.2 which was integrated into Maya 2023.2. I guess you are building with an earlier version of Maya 2023. Sorry we should have caught this earlier.

However in order to accept this pull-request we will need to have a signed CLA. Please see https://github.com/Autodesk/maya-usd/blob/dev/doc/CONTRIBUTING.md.

Sean

@seando-adsk seando-adsk added the do-not-merge-yet Development is not finished, PR not ready for merge label Jul 7, 2023
@jjzanin-SMS
Copy link
Author

Interesting, we are on 2023.3.0-14.0.PFIX. I assume that should have the new header file. I'll ask around to see why our include files aren't being updated.

I'll also talk to my lead about the CLA. Thanks!

@seando-adsk
Copy link
Collaborator

Ufe is part of the Maya devkit. So you probably just need to download the devkit for Maya 2023.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge-yet Development is not finished, PR not ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants