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

Have Qt4, but don't have PysideTools, build fails #18

Closed
meshula opened this issue Aug 11, 2016 · 1 comment
Closed

Have Qt4, but don't have PysideTools, build fails #18

meshula opened this issue Aug 11, 2016 · 1 comment
Assignees

Comments

@meshula
Copy link
Member

meshula commented Aug 11, 2016

If one has Qt4 but not PysideTools (a likely scenario), the build will fail on usdviewq because it calls the PysideTools to wrap usdviewq. Could we add something like this to the usdviewq/CMakeLists.txt file? (Not sure if PYSIDEUICTOOLS_FOUND the right variable to key from, but you get the gist)

 if (NOT QT4_FOUND)
     message(WARNING "Not building ${PXR_PACKAGE} because of missing dependency: Qt4")
     return()
 endif()

+if (NOT PYSIDEUICTOOLS_FOUND)
+    message(WARNING "Not building ${PXR_PACKAGE} because of missing dependency: PysideTools")
+    return()
+endif()
+

@amooty
Copy link

amooty commented Aug 16, 2016

Filed as internal issue #135882.

@sunyab sunyab self-assigned this Aug 19, 2016
asluk pushed a commit to asluk/USD that referenced this issue Aug 31, 2016
This allows the usdImaging component to build on systems that
have Qt4 but not PySide. In such cases, usdview will just not be
available.

Fixes PixarAnimationStudios#18

(Internal change: 1646492)
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

3 participants