-
Notifications
You must be signed in to change notification settings - Fork 202
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
Boudrey/maya 114428/mayapy #1787
Conversation
Trying to be able to debug unit tests on Linux and OSX directly. OSX debugging support
This reverts commit a3f7800.
@@ -61,7 +61,7 @@ set(CMAKE_MODULE_PATH | |||
) | |||
|
|||
# Use RUNPATH instead of RPATH for all shared libs, executables and modules on Linux | |||
if(IS_LINUX) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure it ever worked, but it was a good intention and didn't seems to affect the build.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It probably didn't. The var "IS_LINUX" is set from utils.cmake which isn't included until line 99 below.
I guess that many previous hacks could be removed now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good to me. Thanks for catching that IS_LINUX problem.
@@ -61,7 +61,7 @@ set(CMAKE_MODULE_PATH | |||
) | |||
|
|||
# Use RUNPATH instead of RPATH for all shared libs, executables and modules on Linux | |||
if(IS_LINUX) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It probably didn't. The var "IS_LINUX" is set from utils.cmake which isn't included until line 99 below.
No description provided.