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

Sdf compilation error #125

Closed
krzykli opened this issue Dec 22, 2016 · 7 comments
Closed

Sdf compilation error #125

krzykli opened this issue Dec 22, 2016 · 7 comments

Comments

@krzykli
Copy link
Contributor

krzykli commented Dec 22, 2016

When compiling the current head revision of dev_win_ip branch (8cac49c) I'm getting the following error:

O:\dev\krzysztof.klimczyk\workspace\usdBuildFixed\Projects\USD\pxr\usd\lib\sdf\path.cpp(232): error C2039: 'ContainsTargetPath': is not a member of 'SdfPath' [O:\dev\krzysztof.klimczyk\workspace\usdBuildFixed\Projects\stage\prereq\build\USD\pxr\usd\lib\sdf\sdf.vcxproj]
O:\dev\krzysztof.klimczyk\workspace\usdBuildFixed\Projects\USD\pxr\usd\lib\sdf\path.cpp(233): error C2270: 'ContainsTargetPath': modifiers not allowed on nonmember functions [O:\dev\krzysztof.klimczy\workspace\usdBuildFixed\Projects\stage\prereq\build\USD\pxr\usd\lib\sdf\sdf.vcxproj]
O:\dev\krzysztof.klimczyk\workspace\usdBuildFixed\Projects\USD\pxr\usd\lib\sdf\path.cpp(234): error C2065: '_pathNode': undeclared identifier [O:\dev\krzysztof.klimczyk\workspace\usdBuildFixed\Projects\stage\prereq\build\USD\pxr\usd\lib\sdf\sdf.vcxproj]

System Information (OS, Hardware)

Windows 7

@jtran56
Copy link

jtran56 commented Dec 22, 2016

Filed as internal issue #141430.

@c64kernal
Copy link
Contributor

Hi! Do you mind checking to see if your sdf/path.h has ContainsTargetPath declared?
I'm not able to reproduce this problem locally unfortunately. Do you have any more information on how your project is configured? Were you able to successfully compile dev_win_ip before a certain commit? Thanks for any additional information!

@krzykli
Copy link
Contributor Author

krzykli commented Dec 22, 2016

Hey @c64kernal, just to give you a quick feedback - from memory, I've built USD successfully with 2bea7ca, but it failed already with 1549b88, with the error message above. I had a quick peek at the sdf/path.h and everything there seems to be in order. My config follows https://github.com/vfxpro99/usd-build-club.
If I have enough time today I will rebuild USD again with these two commits to double-check this for you.

@krzykli
Copy link
Contributor Author

krzykli commented Dec 22, 2016

I can confirm now what I wrote in the previous comment.
On my machine 1549b88 will fail to compile (also because of #116), but I have no problems with the commit before it. Would be great if somebody else could check this as well so we can rule out the possibility this issue affects just me :)

@c64kernal
Copy link
Contributor

Hi @krzykli -- thanks for confirming. I've tried again and I can't seem to get the same issue building from head. Apologies for a silly question, but are you building from a fresh branch completely from scratch? (Note: I don't use build-club)

@meshula
Copy link
Member

meshula commented Dec 22, 2016

Another related question, did you try deleting the published headers? Cmake doesn't detect stale headers very well, so my suspicion is that the new headers did not get published, and so you are compiling against a version of the header without the declspec in it.

@krzykli
Copy link
Contributor Author

krzykli commented Dec 25, 2016

Thanks for the tip @meshula. That was exactly it! I've deleted the published headers, and successfully recompiled USD. Now I know to keep an eye on it.
Apologies for the noise.
Closing!

@krzykli krzykli closed this as completed Dec 25, 2016
AdamFelt pushed a commit to autodesk-forks/USD that referenced this issue Oct 11, 2022
…re/TextCherryPick

[OGSMOD-2048]Cherry pick changes related with text feature
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

No branches or pull requests

4 participants