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

Add missing export attributes to SdfLayerDelegate and SdfData #623

Merged
merged 3 commits into from
Oct 19, 2018

Conversation

sopvop
Copy link
Contributor

@sopvop sopvop commented Sep 19, 2018

Description of Change(s)

Adds export attributes missing from SdlLayerStateDelegate and SdfData. These are necessary to make https://github.com/LumaPictures/usd-qt compile on windows with MSVC

Fixes Issue(s)

@jtran56
Copy link

jtran56 commented Sep 20, 2018

Filed as internal issue #USD-4768.

@sunyab sunyab changed the base branch from master to dev September 21, 2018 18:02
@sunyab sunyab changed the base branch from dev to master September 21, 2018 18:04
Copy link
Contributor

@sunyab sunyab left a comment

Choose a reason for hiding this comment

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

Hi @sopvop, just had one minor note below.

Also, could you make these changes in the 'dev' branch of the repository? All new development happens there. I think you may need to close this pull request and open a new one with those changes. Thanks!

pxr/usd/lib/sdf/layerStateDelegate.h Show resolved Hide resolved
@sunyab sunyab added the blocked Issue fix or pull request blocked until questions are answered or pending notes are addressed label Sep 21, 2018
@sopvop sopvop force-pushed the export-sdf-layer-delegate branch from 2e31134 to e27ff96 Compare September 24, 2018 08:18
@sopvop sopvop changed the base branch from master to dev September 24, 2018 08:19
@sopvop
Copy link
Contributor Author

sopvop commented Oct 4, 2018

BTW, recreating pull request was not necessary. You can change target branch if you click edit button to the right of request title.

@sopvop sopvop force-pushed the export-sdf-layer-delegate branch from 0491f32 to 4b6d8f9 Compare October 4, 2018 11:53
@sunyab sunyab added pending push and removed blocked Issue fix or pull request blocked until questions are answered or pending notes are addressed labels Oct 17, 2018
@pixar-oss pixar-oss merged commit 4b6d8f9 into PixarAnimationStudios:dev Oct 19, 2018
pixar-oss added a commit that referenced this pull request Oct 19, 2018
Add missing export attributes to SdfLayerDelegate and SdfData

(Internal change: 1903136)
AdamFelt pushed a commit to autodesk-forks/USD that referenced this pull request Apr 16, 2024
Merge pull request PixarAnimationStudios#623 from autodesk-forks/resolve/dev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants