-
-
Notifications
You must be signed in to change notification settings - Fork 227
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 broken? #125
Comments
It seemed to install all the header files without subdirectories. The install looked like this for me, when I had the same error:
While the build looked like:
I don't have a solution, but I am having the same problem. |
Hello. Thanks everyone for reporting this issue. Yes, the issue is that cmake will copy the headers in the ${CMAKE_INSTALL_INCLUDEDIR}/reactphysics3d/ in a flat way (without the subfolders) which is not working. I have just pushed a fix in the "master" branch that will copy the headers in the ${CMAKE_INSTALL_INCLUDEDIR}/reactphysics3d/ with the correct hierarchy of subdirectories. Note that you might need to add the ${CMAKE_INSTALL_INCLUDEDIR}/reactphysics3d/ directory as an "include directory" for your project so that the compiler correctly finds the ReactPhysics3D headers. This is a temporary fix. For the next release of the library, I will change a little the structure of headers in the project so that it will be installed correctly with cmake. |
Duplicate of #106 |
It is a known problem of CMake when using: The workaround I found was to use INSTALL:
|
The version v0.8.0 of ReactPhysics3D has been released. In this version, this issue is now fixed. Note that this version contains some public API changes. Please read carefully the change log before upgrading to this new version and do not hesitate to take a look at the user manual. Thanks for reporting this issue. I am now closing this issue. |
When I cmake install reactphysics3d I get the directory:
but the following test program:
Doesn't compile:
error:
This is because:
includes:
which has:
But no containers directory is in the
<installdir>
Am I missing something?
The text was updated successfully, but these errors were encountered: