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

CMAKE_INSTALL_PREFIX not used #38

Closed
scpeters opened this issue Sep 26, 2013 · 2 comments
Closed

CMAKE_INSTALL_PREFIX not used #38

scpeters opened this issue Sep 26, 2013 · 2 comments
Labels

Comments

@scpeters
Copy link
Member

In the root CMakeLists.txt, the CMAKE_INSTALL_PREFIX variable is replaced by the value of SimTK_INSTALL_PREFIX. In the comments (which were added in afc617e), it states that CMAKE_INSTALL_PREFIX is "always set to something incorrect by CMake". Is this just an issue with Windows? We use CMAKE_INSTALL_PREFIX regularly on Ubuntu and OSX for gazebo.

@sherm1
Copy link
Member

sherm1 commented Sep 27, 2013

I think it was true a few years ago that CMAKE_INSTALL_PREFIX didn't properly deal with Windows installation, I think that has probably been fixed for a long time and SimTK_INSTALL_PREFIX is now unnecessary baggage. I remember that it was unable to deal with the byzantine rules for where to install 32 vs. 64 bit binaries on 32 or 64 bit Windows but I think they fixed that.

@scpeters
Copy link
Member Author

Aside from the root CMakeLists.txt, it looks like the SimTK_INSTALL_PREFIX variable is referenced from the following:

./SimTKmath/sharedTarget/CMakeLists.txt
./SimTKmath/staticTarget/CMakeLists.txt
./Platform/CMakeLists.txt
./Simbody/Visualizer/VisualizerGUI/CMakeLists.txt
./Simbody/sharedTarget/CMakeLists.txt
./Simbody/staticTarget/CMakeLists.txt
./debian/rules
./SimTKcommon/sharedTarget/CMakeLists.txt
./SimTKcommon/staticTarget/CMakeLists.txt

scpeters added a commit to scpeters/simbody that referenced this issue Oct 2, 2013
Fix the use of SimTK_INSTALL_PREFIX for backward compatibility.
@sherm1 sherm1 closed this as completed in 7023a2b Oct 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants