All the libraries #1883
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,6 +6,6 @@ Name: libOSRM | |
| Description: Project OSRM library | ||
| Version: v@OSRM_VERSION_MAJOR@.@OSRM_VERSION_MINOR@.@OSRM_VERSION_PATCH@ | ||
| Requires: | ||
| Libs: -L${libdir} -losrm | ||
|
There was a problem hiding this comment. There are still libs missing, at least pthread and rt; take a look at my old Gist on how to compile the simpleclient, and why pkg-config was not sufficient: https://gist.github.com/daniel-j-h/53499ae4534920312992 There was a problem hiding this comment. Yes but the |
||
| Libs.private: @ENGINE_LIBRARY_LISTING@ | ||
| Cflags: -I${includedir} | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| cmake_minimum_required(VERSION 2.8.8) | ||
|
|
||
| if(CMAKE_CURRENT_SOURCE_DIR STREQUAL CMAKE_CURRENT_BINARY_DIR AND NOT MSVC_IDE) | ||
| message(FATAL_ERROR "In-source builds are not allowed. | ||
| Please create a directory and run cmake from there, passing the path to this source directory as the last argument. | ||
| This process created the file `CMakeCache.txt' and the directory `CMakeFiles'. Please delete them.") | ||
| endif() | ||
|
|
||
| project(osrm-example C CXX) | ||
|
There was a problem hiding this comment. I don't think we need a C compiler, CXX should suffice. And if we do, we should update Travis, as we only set CXX but not CC. There was a problem hiding this comment. I think we previously needed it for the PBF generation code that should be gone now with There was a problem hiding this comment. Actually turns out it was necessary for some CheckCXXCompile tests down the dependency chain. |
||
|
|
||
| list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/cmake") | ||
| set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 ") | ||
|
|
||
| set(bitness 32) | ||
| if(CMAKE_SIZEOF_VOID_P EQUAL 8) | ||
| set(bitness 64) | ||
| message(STATUS "Building on a 64 bit system") | ||
| else() | ||
| message(WARNING "Building on a 32 bit system is unsupported") | ||
|
There was a problem hiding this comment. While we're at it, this should really exit here i.e. |
||
| endif() | ||
|
|
||
| if(WIN32 AND MSVC_VERSION LESS 1800) | ||
| message(FATAL_ERROR "Building with Microsoft compiler needs Visual Studio 2013 or later (Express version works too)") | ||
| endif() | ||
|
|
||
| add_executable(osrm-example example.cpp) | ||
|
|
||
| find_package(LibOSRM REQUIRED) | ||
| find_package(Boost 1.49.0 COMPONENTS filesystem system thread REQUIRED) | ||
|
|
||
| target_link_libraries(osrm-example ${LibOSRM_LIBRARIES} ${Boost_LIBRARIES}) | ||
| include_directories(SYSTEM ${LibOSRM_INCLUDE_DIRS} ${Boost_INCLUDE_DIRS}) | ||
|
|
||
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.
libzip? Is this supposed to installzlibsincelibzip!=libz?Also, do we build the tools on Travis? If not drop the
GDALdependency.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.
Yes we build the tools on travis.