Skip to content

Commit

Permalink
Printing: Change "NEW_PRINTING" to "NO_PRINTING"
Browse files Browse the repository at this point in the history
If NO_PRINTING is not set, subsurface will require grantlee during
building. As Grantlee will be a hard dependency for Subsurface, so no
Grantlee means no printing.

Signed-off-by: Gehad elrobey <gehadelrobey@gmail.com>
Signed-off-by: Lubomir I. Ivanov <neolit123@gmail.com>
  • Loading branch information
Gehadelrobey authored and neolit123 committed Jun 4, 2015
1 parent 3f3937f commit 75b5d7e
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions CMakeLists.txt
Expand Up @@ -13,6 +13,7 @@ option(LIBGRANTLEE_FROM_PKGCONFIG "use pkg-config to retrieve grantlee" OFF)
option(NO_MARBLE "disable the marble widget" OFF)
option(NO_TESTS "disable the tests" OFF)
option(NO_DOCS "disable the docs" OFF)
option(NO_PRINTING "disable the printing support" OFF)
option(USE_LIBGIT23_API "allow building with libgit2 master" OFF)

set(CMAKE_MODULE_PATH
Expand Down Expand Up @@ -94,15 +95,24 @@ endif()

# setup Grantlee

if(NEW_PRINTING)
if(NOT NO_PRINTING)
if(LIBGRANTLEE_FROM_PKGCONFIG)
pkg_config_library(GRANTLEE libgrantlee REQUIRED)
set(GRANTLEE_LIBRARIES "")
else()
find_package(Grantlee5)
set(GRANTLEE_LIBRARIES Grantlee5::Templates)
endif()
add_definitions(-DNEW_PRINTING)
set(SUBSURFACE_PRINTING_SRCS
printer.cpp
templatelayout.cpp
)
endif()

if(NO_PRINTING)
message(STATUS "building without printing support")
add_definitions(-DNO_PRINTING)
set(GRANTLEE_LIBRARIES "")
endif()

set(SUBSURFACE_LINK_LIBRARIES ${SUBSURFACE_LINK_LIBRARIES} ${LIBDIVECOMPUTER_LIBRARIES} ${LIBGIT2_LIBRARIES} ${GRANTLEE_LIBRARIES} -lusb-1.0)
Expand Down Expand Up @@ -260,8 +270,7 @@ set(SUBSURFACE_CORE_LIB_SRCS
configuredivecomputer.cpp
configuredivecomputerthreads.cpp
divesitehelpers.cpp
templatelayout.cpp
printer.cpp
${SUBSURFACE_PRINTING_SRCS}
${PLATFORM_SRC}
)
source_group("Subsurface Core" FILES ${SUBSURFACE_CORE_LIB_SRCS})
Expand Down

0 comments on commit 75b5d7e

Please sign in to comment.