Skip to content

Make path to cmake modules relative to CMakeLists.txt #504

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

Merged
merged 1 commit into from
Feb 12, 2014
Merged

Make path to cmake modules relative to CMakeLists.txt #504

merged 1 commit into from
Feb 12, 2014

Conversation

grtlr
Copy link
Contributor

@grtlr grtlr commented Feb 11, 2014

There was a problem when including the PCL as a dependency using the add_subdirectory(...) command. The cmake modules would not be found, since one would not invoke the CMakeLists.txt from the PCL root. Here is the cmake documentation (from http://www.cmake.org/Wiki/CMake_Useful_Variables):

CMAKE_SOURCE_DIR
this is the directory, from which cmake was started, i.e. the top level source directory

CMAKE_CURRENT_SOURCE_DIR
this is the directory where the currently processed CMakeLists.txt is located in

There was a problem when including the PCL as a dependency using the add_subdirectory(...) command. The cmake modules would not be found, since one would not invoke the CMakeLists.txt from the PCL root. Here is the cmake documentation (from http://www.cmake.org/Wiki/CMake_Useful_Variables):

*CMAKE_SOURCE_DIR*
    this is the directory, from which cmake was started, i.e. the top level source directory

*CMAKE_CURRENT_SOURCE_DIR*
    this is the directory where the currently processed CMakeLists.txt is located in
jspricke added a commit that referenced this pull request Feb 12, 2014
Make path to cmake modules relative to CMakeLists.txt
@jspricke jspricke merged commit e201e62 into PointCloudLibrary:master Feb 12, 2014
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

Successfully merging this pull request may close these issues.

2 participants