diff --git a/.github/actions/run_tests/action.yml b/.github/actions/run_tests/action.yml index 44db0d99b..14b7fe91e 100644 --- a/.github/actions/run_tests/action.yml +++ b/.github/actions/run_tests/action.yml @@ -27,14 +27,14 @@ runs: - name: Test visuals (windows) continue-on-error: true - working-directory: build/${{ inputs.cmake-preset }}/out + working-directory: build/out shell: "pwsh" if: runner.os == 'Windows' run: OpenCppCoverage --modules *libmapnik* --modules mapnik*.exe --modules *.input --sources ${{ github.workspace }} --export_type binary --input_coverage=${{ github.workspace }}/ctest.cov --cover_children -- .\mapnik-test-visual.exe -j (Get-CimInstance -ClassName Win32_ComputerSystem).NumberOfLogicalProcessors --output-dir ./visual-test-result - name: Test visuals (linux & mac) continue-on-error: true - working-directory: build/${{ inputs.cmake-preset }}/out + working-directory: build/out shell: "bash" if: runner.os != 'Windows' run: | @@ -45,7 +45,7 @@ runs: fi - name: Pack visual test results - working-directory: build/${{ inputs.cmake-preset }}/out + working-directory: build/out shell: "pwsh" run: tar cfvz visual-test-results.tar.gz ./visual-test-result @@ -60,16 +60,16 @@ runs: uses: actions/upload-artifact@v3 with: name: ${{ inputs.cmake-preset }}-visual-tests-${{ steps.run-guid.outputs.GUID }} - path: build/${{ inputs.cmake-preset }}/out/visual-test-results.tar.gz + path: build/out/visual-test-results.tar.gz - name: Run Benchmarks - working-directory: build/${{ inputs.cmake-preset }}/out + working-directory: build/out if: runner.os != 'Windows' shell: "pwsh" run: ./run_benchmarks - name: Collect coverage (linux & macos) - working-directory: build/${{ inputs.cmake-preset }} + working-directory: build if: runner.os != 'Windows' shell: "bash" run: | @@ -81,10 +81,10 @@ runs: if: runner.os != 'Windows' uses: codecov/codecov-action@v3 with: - files: build/${{ inputs.cmake-preset }}/coverage.info + files: build/coverage.info - name: Upload coverage to Codecov (windows) if: runner.os == 'Windows' uses: codecov/codecov-action@v3 with: - files: build/${{ inputs.cmake-preset }}/out/mapnik-test-visual.cov + files: build/out/mapnik-test-visual.cov diff --git a/.github/workflows/build_and_test.yml b/.github/workflows/build_and_test.yml index 8fda668c2..8186fbfbb 100644 --- a/.github/workflows/build_and_test.yml +++ b/.github/workflows/build_and_test.yml @@ -32,6 +32,7 @@ jobs: matrix: os: [macos-latest, windows-latest, ubuntu-latest] memory-mapped: ["OFF", "ON"] + static-build: ["OFF", "ON"] include: - os: ubuntu-latest mono: mono @@ -82,7 +83,7 @@ jobs: echo "PRESET=${{ steps.lc_platform.outputs.lowercase }}-ci" >>${GITHUB_ENV} - name: Configure CMake - run: cmake -DUSE_MEMORY_MAPPED_FILE=${{ matrix.memory-mapped }} --preset ${{ env.PRESET }} + run: cmake -DUSE_MEMORY_MAPPED_FILE=${{ matrix.memory-mapped }} -DBUILD_SHARED_LIBS=${{ matrix.static-build }} --preset ${{ env.PRESET }} - name: Build run: cmake --build --preset ${{ env.PRESET }} diff --git a/CHANGELOG.md b/CHANGELOG.md index a5c176354..5e3ec9b22 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -12,6 +12,12 @@ For a complete change history, see the git log. #### Notice - Mapnik now requires C++14 compliant compiler (`-std=c++14`) +- Mapnik now supports CMake as a build system. See [#4191](https://github.com/mapnik/mapnik/pull/4191) and the [docs](https://github.com/mapnik/mapnik/blob/master/docs/cmake-usage.md) for more info. + +#### Breaking Changes + +- Reworked datasource plugin system. Plugins now need to implement a class with the macros in `datasource_plugin.hpp` ([#4291](https://github.com/mapnik/mapnik/pull/4291)) +- mapnik now has a global `mapnik::setup()` method which have to be called before any other functions of mapnik. Defined in `mapnik.hpp`. Currently there is a auto setup functionality. It can be disabled using the cmake option `DISABLE_MAPNIK_AUTOSETUP=ON`. Note: In order not to insert this change into every code base, it is currently still called during the dynamic initialisation time. However, if mapnik is compiled statically, this function must be called. ([#4291](https://github.com/mapnik/mapnik/pull/4291)) #### Core @@ -27,6 +33,7 @@ For a complete change history, see the git log. - Slightly improved `sql_utils::table_from_sql` ([2587bb3](https://github.com/mapnik/mapnik/commit/2587bb3a1d8db397acfa8dcc2d332da3a8a9399f)) - Added wrappers for proper quoting in SQL query construction: `sql_utils::identifier`, `sql_utils::literal` ([7b21713](https://github.com/mapnik/mapnik/commit/7b217133e2749b82c2638551045c4edbece15086)) - Added two-argument `sql_utils::unquote`, `sql_utils::unquote_copy` that also collapse inner quotes ([a4e8ea2](https://github.com/mapnik/mapnik/commit/a4e8ea21be297d89bbf36ba594d6c661a7a9ac81)) +- Fixed mapnik static build with static plugins ([#4291](https://github.com/mapnik/mapnik/pull/4291)) - Reworked mapnik::enumeration<...> ([#4372](https://github.com/mapnik/mapnik/pull/4372)) #### Plugins diff --git a/CMakeLists.txt b/CMakeLists.txt index d87a7052d..31510c913 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -16,6 +16,7 @@ message(STATUS "mapnik version: ${PROJECT_VERSION}") # https://cliutils.gitlab.io/modern-cmake/chapters/features/ides.html set_property(GLOBAL PROPERTY USE_FOLDERS ON) +set(CMAKE_FIND_PACKAGE_TARGETS_GLOBAL ON) # with newer cmake versions put all find_package in global scope set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_CURRENT_SOURCE_DIR}/cmake") include(FeatureSummary) include(MapnikOption) @@ -27,8 +28,13 @@ include(CTest) add_feature_info(BUILD_TESTING BUILD_TESTING "Adds tests") mapnik_option(INSTALL_DEPENDENCIES "if ON, all dependencies (eg. required dlls) will be copied into CMAKE_INSTALL_PREFIX/MAPNIK_BIN_DIR." ON) - mapnik_option(BUILD_SHARED_LIBS "build mapnik dynamic(ON) or static(OFF)" ON) +mapnik_option(BUILD_SHARED_PLUGINS "build dynamic plugins" ${BUILD_SHARED_LIBS}) # use BUILD_SHARED_LIBS as default option +mapnik_option(BUILD_SHARED_CRT "(only windows with msvc) use msvc shared crt" ON) +if(WIN32 AND BUILD_SHARED_PLUGINS AND NOT BUILD_SHARED_LIBS) + message(FATAL_ERROR "static libmapnik and dynamic plugins won't work correctly") +endif() + mapnik_option(USE_EXTERNAL_MAPBOX_GEOMETRY "Use a external mapnik/geometry.hpp. If off, use the submodule" OFF) mapnik_option(USE_EXTERNAL_MAPBOX_POLYLABEL "Use a external mapnik/polylabel. If off, use the submodule" OFF) mapnik_option(USE_EXTERNAL_MAPBOX_PROTOZERO "Use a external mapnik/protozero. If off, use the submodule" OFF) @@ -56,6 +62,8 @@ mapnik_option(USE_LOG "enables logging output. See log severity level." OFF) set(USE_LOG_SEVERITY "1" CACHE STRING "sets the logging severity (only applies when USE_LOG is ON") mapnik_option(USE_STATS "Enable statistics reporting" OFF) +mapnik_option(DISABLE_MAPNIK_AUTOSETUP "disables the autosetup. Need to call mapnik::setup() then" OFF) + mapnik_option(USE_PLUGIN_INPUT_CSV "adds plugin input csv" ON) mapnik_option(USE_PLUGIN_INPUT_GDAL "adds plugin input gdal" ON) mapnik_option(USE_PLUGIN_INPUT_GEOBUF "adds plugin input geobuf" ON) @@ -87,18 +95,6 @@ mapnik_option(USE_GLIBC_WORKAROUND "see https://github.com/mapnik/mapnik/pull/37 feature_summary(WHAT ENABLED_FEATURES DISABLED_FEATURES) feature_summary(FILENAME "${CMAKE_CURRENT_BINARY_DIR}/features.log" WHAT ENABLED_FEATURES DISABLED_FEATURES) -set(CMAKE_CXX_STANDARD 14 CACHE STRING "Sets the c++ standard. c++14 is minimum.") -message(STATUS "Using c++${CMAKE_CXX_STANDARD}") -# https://cmake.org/cmake/help/latest/prop_tgt/CXX_EXTENSIONS.html -set(CXX_EXTENSIONS OFF CACHE STRING "Enables the compiler specific extensions.") # Fallsback to -std=c++ if off -message(STATUS "Using c++ extensions: ${CXX_EXTENSIONS}") -# https://cmake.org/cmake/help/latest/prop_tgt/CXX_STANDARD_REQUIRED.html#prop_tgt:CXX_STANDARD_REQUIRED -set(CXX_STANDARD_REQUIRED ON) # require the specified CMAKE_CXX_STANDARD - -# add debug postfix to the libraries -set(MAPNIK_DEBUG_POSTFIX "d" CACHE STRING "sets the debug library postfix on mapnik, wkt and json") -message(STATUS "postfix for debug libraries: ${MAPNIK_DEBUG_POSTFIX}") - include(GNUInstallDirs) # See for more details: https://cmake.org/cmake/help/latest/manual/cmake-buildsystem.7.html set(MAPNIK_BIN_DIR ${CMAKE_INSTALL_BINDIR} CACHE STRING "Install directory for binaries") @@ -123,9 +119,29 @@ set(MAPNIK_COMPILE_DEFS "") set(MAPNIK_OPTIONAL_LIBS "") set(MAPNIK_OPTIONAL_LIBS_INCLUDE "") +############################# +############################# # Begin project configuration +############################# +############################# + +set(CMAKE_CXX_STANDARD 14 CACHE STRING "Sets the c++ standard. c++14 is minimum.") +set(CMAKE_CXX_STANDARD_REQUIRED ON) # require the specified CMAKE_CXX_STANDARD +set(CMAKE_CXX_EXTENSIONS OFF CACHE STRING "Enables the compiler specific extensions.") # Fallsback to -std=c++ if off +message(STATUS "Using c++${CMAKE_CXX_STANDARD}") +message(STATUS "Using c++ extensions: ${CXX_EXTENSIONS}") + +# add debug postfix to the libraries +set(CMAKE_DEBUG_POSTFIX "d" CACHE STRING "sets the debug library postfix on mapnik, wkt and json") +message(STATUS "postfix for debug libraries: ${CMAKE_DEBUG_POSTFIX}") + +set(CMAKE_MSVC_RUNTIME_LIBRARY "MultiThreaded$<$:Debug>$<$:DLL>") +set(CMAKE_RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}") +set(CMAKE_LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}") +set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib") # needs to be before the first call of find_boost. +list(APPEND MAPNIK_COMPILE_DEFS BOOST_SPIRIT_X3_HIDE_CXX17_WARNING) if(USE_MULTITHREADED) set(Boost_USE_MULTITHREADED ON) list(APPEND MAPNIK_COMPILE_DEFS MAPNIK_THREADSAFE) @@ -134,7 +150,7 @@ else() endif() -find_package(PkgConfig) +mapnik_find_package(PkgConfig REQUIRED) mapnik_find_threads() mapnik_find_package(ICU REQUIRED COMPONENTS uc i18n data) @@ -163,7 +179,7 @@ else() # It might be possible that in future version harfbuzz could only be found via pkg-config. # harfbuzz related discussion: https://github.com/harfbuzz/harfbuzz/issues/2653 message(STATUS "harfbuzz not found via cmake. Searching via pkg-config...") - pkg_check_modules(harfbuzz REQUIRED IMPORTED_TARGET harfbuzz>=${HARFBUZZ_MIN_VERSION}) + mapnik_pkg_check_modules(harfbuzz REQUIRED IMPORTED_TARGET harfbuzz>=${HARFBUZZ_MIN_VERSION}) list(APPEND MAPNIK_OPTIONAL_LIBS PkgConfig::harfbuzz) endif() @@ -290,20 +306,15 @@ if(USE_TIFF) endif() if(USE_WEBP) - mapnik_find_package(WebP REQUIRED) + mapnik_pkg_check_modules(WebP REQUIRED IMPORTED_TARGET libwebp) list(APPEND MAPNIK_COMPILE_DEFS HAVE_WEBP) - list(APPEND MAPNIK_OPTIONAL_LIBS WebP::WebP) + list(APPEND MAPNIK_OPTIONAL_LIBS PkgConfig::WebP) endif() if(USE_CAIRO) - if(WIN32) - mapnik_find_package(Cairo REQUIRED) - list(APPEND MAPNIK_OPTIONAL_LIBS Cairo::Cairo) - else() - pkg_check_modules(Cairo REQUIRED IMPORTED_TARGET cairo) - list(APPEND MAPNIK_OPTIONAL_LIBS PkgConfig::Cairo) - endif() + mapnik_pkg_check_modules(Cairo REQUIRED IMPORTED_TARGET cairo) list(APPEND MAPNIK_COMPILE_DEFS HAVE_CAIRO) + list(APPEND MAPNIK_OPTIONAL_LIBS PkgConfig::Cairo) endif() if(USE_PROJ) @@ -312,7 +323,7 @@ if(USE_PROJ) # currently the cmake files are not installed, when installing proj via apt-get. So search via pkg-config if(NOT PROJ_FOUND) message(STATUS "PROJ not found via FindPROJ. Searching via pkg-config...") - pkg_check_modules(PROJ REQUIRED IMPORTED_TARGET proj>=${PROJ_MIN_VERSION}) + mapnik_pkg_check_modules(PROJ REQUIRED IMPORTED_TARGET proj>=${PROJ_MIN_VERSION}) string(REGEX MATCH "([0-9]+)\.([0-9]+)\.([0-9]+)" _dummy "${PROJ_VERSION}") set(PROJ_VERSION_MAJOR "${CMAKE_MATCH_1}") set(PROJ_VERSION_MINOR "${CMAKE_MATCH_2}") @@ -344,10 +355,24 @@ if(NOT WIN32) list(APPEND MAPNIK_OPTIONAL_LIBS ${CMAKE_DL_LIBS}) endif() +if(NOT BUILD_SHARED_PLUGINS) + list(APPEND MAPNIK_COMPILE_DEFS MAPNIK_STATIC_PLUGINS) +endif() +# when building static, this have to be public so that all depending libs know about +if(NOT BUILD_SHARED_LIBS) + list(APPEND MAPNIK_COMPILE_DEFS MAPNIK_STATIC_DEFINE) +endif() + +if(DISABLE_MAPNIK_AUTOSETUP) + list(APPEND MAPNIK_COMPILE_DEFS MAPNIK_DISABLE_AUTOSETUP) +endif() + # force utf-8 source code processing # see https://docs.microsoft.com/de-de/cpp/build/reference/utf-8-set-source-and-executable-character-sets-to-utf-8?view=msvc-170 -add_compile_options("$<$:/utf-8>") -add_compile_options("$<$:/utf-8>") +add_compile_options( + "$<$:/utf-8>" + "$<$:/EHsc>" +) add_library(core INTERFACE) add_library(mapnik::core ALIAS core) @@ -377,9 +402,20 @@ target_compile_definitions(core INTERFACE ${MAPNIK_COMPILE_DEFS}) mapnik_install(core) +### +# forward declaring libraries to consume them when building static plugins (circle deps between mapnik <-> plugin_target) +add_library(mapnik "") +add_library(mapnik::mapnik ALIAS mapnik) +add_library(wkt STATIC "") +add_library(mapnik::wkt ALIAS wkt) +add_library(json STATIC "") +add_library(mapnik::json ALIAS json) +# end forward declaration +### + add_subdirectory(deps) -add_subdirectory(src) add_subdirectory(plugins) +add_subdirectory(src) add_subdirectory(utils) add_subdirectory(demo) if(BUILD_BENCHMARK) @@ -396,23 +432,23 @@ feature_summary(FILENAME "${CMAKE_CURRENT_BINARY_DIR}/packages.log" WHAT PACKAGE include(MapnikExport) include(MapnikExportPkgConfig) -install(DIRECTORY include/ DESTINATION ${MAPNIK_INCLUDE_DIR}) -install(DIRECTORY deps/agg/include/ DESTINATION ${MAPNIK_INCLUDE_DIR}) -install(DIRECTORY deps/mapnik DESTINATION ${MAPNIK_INCLUDE_DIR}) +install(DIRECTORY include/ DESTINATION "${MAPNIK_INCLUDE_DIR}") +install(DIRECTORY deps/agg/include/ DESTINATION "${MAPNIK_INCLUDE_DIR}") +install(DIRECTORY deps/mapnik DESTINATION "${MAPNIK_INCLUDE_DIR}") file(GLOB TTF_FONT_FILES "fonts/*/*/*.ttf") -install(FILES ${TTF_FONT_FILES} DESTINATION ${FONTS_INSTALL_DIR}) +install(FILES ${TTF_FONT_FILES} DESTINATION "${FONTS_INSTALL_DIR}") if(NOT USE_EXTERNAL_MAPBOX_GEOMETRY) - install(DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/deps/mapbox/geometry/include/ DESTINATION ${MAPNIK_INCLUDE_DIR}) + install(DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/deps/mapbox/geometry/include/" DESTINATION "${MAPNIK_INCLUDE_DIR}") endif() if(NOT USE_EXTERNAL_MAPBOX_POLYLABEL) - install(DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/deps/mapbox/polylabel/include/ DESTINATION ${MAPNIK_INCLUDE_DIR}) + install(DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/deps/mapbox/polylabel/include/" DESTINATION "${MAPNIK_INCLUDE_DIR}") endif() if(NOT USE_EXTERNAL_MAPBOX_PROTOZERO) - install(DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/deps/mapbox/protozero/include/ DESTINATION ${MAPNIK_INCLUDE_DIR}) + install(DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/deps/mapbox/protozero/include/" DESTINATION "${MAPNIK_INCLUDE_DIR}") endif() if(NOT USE_EXTERNAL_MAPBOX_VARIANT) - install(DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/deps/mapbox/variant/include/ DESTINATION ${MAPNIK_INCLUDE_DIR}) + install(DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/deps/mapbox/variant/include/" DESTINATION "${MAPNIK_INCLUDE_DIR}") endif() mapnik_install_targets() diff --git a/CMakePresets.json b/CMakePresets.json index f0bbb21a6..bc5a7c995 100644 --- a/CMakePresets.json +++ b/CMakePresets.json @@ -17,7 +17,7 @@ { "name": "default-build-dir", "hidden": true, - "binaryDir": "${sourceDir}/build/${presetName}" + "binaryDir": "${sourceDir}/build" }, { "name": "debug-build", @@ -180,8 +180,9 @@ "cacheVariables": { "BUILD_TESTING": "ON", "BUILD_DEMO_VIEWER": "OFF", - "CMAKE_TOOLCHAIN_FILE": "vcpkg/scripts/buildsystems/vcpkg.cmake" - } + "DISABLE_MAPNIK_AUTOSETUP": "ON" + }, + "toolchainFile": "vcpkg/scripts/buildsystems/vcpkg.cmake" }, { "name": "windows-ci", @@ -192,10 +193,10 @@ ], "cacheVariables": { "INSTALL_DEPENDENCIES": "ON", - "ADDITIONAL_LIBARIES_PATHS": "${sourceDir}/build/${presetName}/vcpkg_installed/x64-windows/bin" + "ADDITIONAL_LIBARIES_PATHS": "${sourceDir}/build/vcpkg_installed/x64-windows/bin" }, "environment": { - "PROJ_LIB": "${sourceDir}/build/${presetName}/vcpkg_installed/x64-windows/share/proj" + "PROJ_LIB": "${sourceDir}/build/vcpkg_installed/x64-windows/share/proj" } }, { @@ -210,7 +211,7 @@ "CMAKE_CXX_FLAGS": "--coverage" }, "environment": { - "PROJ_LIB": "${sourceDir}/build/${presetName}/vcpkg_installed/x64-linux/share/proj" + "PROJ_LIB": "${sourceDir}/build/vcpkg_installed/x64-linux/share/proj" } }, { @@ -239,7 +240,7 @@ "CMAKE_CXX_FLAGS": "-fprofile-arcs -ftest-coverage" }, "environment": { - "PROJ_LIB": "${sourceDir}/build/${presetName}/vcpkg_installed/x64-osx/share/proj" + "PROJ_LIB": "${sourceDir}/build/vcpkg_installed/x64-osx/share/proj" } } ], diff --git a/SConstruct b/SConstruct index 9df2c5f9f..99577df30 100644 --- a/SConstruct +++ b/SConstruct @@ -1879,6 +1879,7 @@ if not preconfigured: env.Prepend(CPPPATH = '#deps/mapbox/geometry/include') env.Prepend(CPPPATH = '#deps/mapbox/protozero/include') env.Prepend(CPPPATH = '#deps/mapbox/polylabel/include') + env.Prepend(CPPPATH = '#plugins/input/base/include') # prepend deps dir for auxillary headers env.Prepend(CPPPATH = '#deps') diff --git a/benchmark/CMakeLists.txt b/benchmark/CMakeLists.txt index 8c328c9fe..822fe56a7 100644 --- a/benchmark/CMakeLists.txt +++ b/benchmark/CMakeLists.txt @@ -31,11 +31,12 @@ function(mapnik_create_benchmark) set(TARGET_NAME "mapnik-benchmark-${BENCHNAME}") add_executable(${TARGET_NAME} ${ARGV0}) target_include_directories(${TARGET_NAME} PRIVATE include) - target_link_libraries(${TARGET_NAME} PRIVATE mapnik::agg mapnik::mapnik) + target_link_libraries(${TARGET_NAME} PRIVATE + mapnik::agg + mapnik::mapnik + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) + ) set_target_properties(${TARGET_NAME} PROPERTIES - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" OUTPUT_NAME "${BENCHNAME}" ) endfunction() diff --git a/benchmark/include/bench_framework.hpp b/benchmark/include/bench_framework.hpp index 94c785ac5..0acfe234e 100644 --- a/benchmark/include/bench_framework.hpp +++ b/benchmark/include/bench_framework.hpp @@ -2,6 +2,7 @@ #define MAPNIK_BENCH_FRAMEWORK_HPP // mapnik +#include #include #include #include @@ -124,6 +125,7 @@ inline int handle_args(int argc, char** argv, mapnik::parameters& params) #define BENCHMARK(test_class, name) \ int main(int argc, char** argv) \ { \ + mapnik::setup(); \ try \ { \ mapnik::parameters params; \ diff --git a/benchmark/src/normalize_angle.cpp b/benchmark/src/normalize_angle.cpp index fcfc7a5c2..97b6cdad1 100644 --- a/benchmark/src/normalize_angle.cpp +++ b/benchmark/src/normalize_angle.cpp @@ -30,6 +30,7 @@ struct bench_func : benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); return benchmark::sequencer(argc, argv) .BENCH_FUNC1(mapnik::util::normalize_angle, +3) .BENCH_FUNC1(mapnik::util::normalize_angle, +6) diff --git a/benchmark/src/test_array_allocation.cpp b/benchmark/src/test_array_allocation.cpp index 592a93d07..b80e9271f 100644 --- a/benchmark/src/test_array_allocation.cpp +++ b/benchmark/src/test_array_allocation.cpp @@ -349,6 +349,7 @@ class test7 : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); return benchmark::sequencer(argc, argv) .run("calloc") .run("malloc/memcpy") diff --git a/benchmark/src/test_expression_parse.cpp b/benchmark/src/test_expression_parse.cpp index d35a57ab2..956dcd39e 100644 --- a/benchmark/src/test_expression_parse.cpp +++ b/benchmark/src/test_expression_parse.cpp @@ -36,6 +36,7 @@ class test : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); mapnik::parameters params; benchmark::handle_args(argc, argv, params); test test_runner(params); diff --git a/benchmark/src/test_face_ptr_creation.cpp b/benchmark/src/test_face_ptr_creation.cpp index 4bbdf0301..64071c32f 100644 --- a/benchmark/src/test_face_ptr_creation.cpp +++ b/benchmark/src/test_face_ptr_creation.cpp @@ -59,6 +59,7 @@ class test : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); mapnik::parameters params; benchmark::handle_args(argc, argv, params); bool success = mapnik::freetype_engine::register_fonts("./fonts", true); diff --git a/benchmark/src/test_getline.cpp b/benchmark/src/test_getline.cpp index 7320d9271..d6b6996fa 100644 --- a/benchmark/src/test_getline.cpp +++ b/benchmark/src/test_getline.cpp @@ -101,6 +101,7 @@ class test2 : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); int return_value = 0; try { diff --git a/benchmark/src/test_numeric_cast_vs_static_cast.cpp b/benchmark/src/test_numeric_cast_vs_static_cast.cpp index 1b28513f2..90ea86c8b 100644 --- a/benchmark/src/test_numeric_cast_vs_static_cast.cpp +++ b/benchmark/src/test_numeric_cast_vs_static_cast.cpp @@ -76,5 +76,6 @@ class test_numeric : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); return benchmark::sequencer(argc, argv).run("static_cast").run("numeric_cast").done(); } diff --git a/benchmark/src/test_offset_converter.cpp b/benchmark/src/test_offset_converter.cpp index 93988ce4c..e591d552f 100644 --- a/benchmark/src/test_offset_converter.cpp +++ b/benchmark/src/test_offset_converter.cpp @@ -92,6 +92,7 @@ class test_offset : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); mapnik::parameters params; benchmark::handle_args(argc, argv, params); int return_value = 0; diff --git a/benchmark/src/test_polygon_clipping.cpp b/benchmark/src/test_polygon_clipping.cpp index 9ec54f251..d8c3fce6d 100644 --- a/benchmark/src/test_polygon_clipping.cpp +++ b/benchmark/src/test_polygon_clipping.cpp @@ -510,6 +510,7 @@ expected_count << "\n"; valid = false; int main(int argc, char** argv) { + mapnik::setup(); mapnik::parameters params; benchmark::handle_args(argc, argv, params); diff --git a/benchmark/src/test_polygon_clipping_rendering.cpp b/benchmark/src/test_polygon_clipping_rendering.cpp index b0daba0c9..9c3e99b33 100644 --- a/benchmark/src/test_polygon_clipping_rendering.cpp +++ b/benchmark/src/test_polygon_clipping_rendering.cpp @@ -43,6 +43,7 @@ class test : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); mapnik::parameters params; benchmark::handle_args(argc, argv, params); mapnik::datasource_cache::instance().register_datasources("./plugins/input/"); diff --git a/benchmark/src/test_proj_transform1.cpp b/benchmark/src/test_proj_transform1.cpp index 96e338ceb..e179a671b 100644 --- a/benchmark/src/test_proj_transform1.cpp +++ b/benchmark/src/test_proj_transform1.cpp @@ -60,6 +60,7 @@ class test : public benchmark::test_case // echo -180 -60 | cs2cs -f "%.10f" epsg:4326 +to epsg:3857 int main(int argc, char** argv) { + mapnik::setup(); mapnik::box2d from(-180, -80, 180, 80); mapnik::box2d to(-20037508.3427892476, -15538711.0963092316, 20037508.3427892476, 15538711.0963092316); std::string from_str("epsg:4326"); diff --git a/benchmark/src/test_rendering.cpp b/benchmark/src/test_rendering.cpp index 3764f946c..3905af922 100644 --- a/benchmark/src/test_rendering.cpp +++ b/benchmark/src/test_rendering.cpp @@ -95,6 +95,7 @@ class test : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); int return_value = 0; try { diff --git a/benchmark/src/test_rendering_shared_map.cpp b/benchmark/src/test_rendering_shared_map.cpp index 43ae727b9..bf5078bbf 100644 --- a/benchmark/src/test_rendering_shared_map.cpp +++ b/benchmark/src/test_rendering_shared_map.cpp @@ -150,6 +150,7 @@ class test : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); int return_value = 0; try { diff --git a/benchmark/src/test_utf_encoding.cpp b/benchmark/src/test_utf_encoding.cpp index badb0a661..c677541db 100644 --- a/benchmark/src/test_utf_encoding.cpp +++ b/benchmark/src/test_utf_encoding.cpp @@ -106,6 +106,7 @@ class test3 : public benchmark::test_case int main(int argc, char** argv) { + mapnik::setup(); mapnik::parameters params; benchmark::handle_args(argc, argv, params); int return_value = 0; diff --git a/cmake/FindCairo.cmake b/cmake/FindCairo.cmake deleted file mode 100644 index 30b175010..000000000 --- a/cmake/FindCairo.cmake +++ /dev/null @@ -1,98 +0,0 @@ -# Distributed under the OSI-approved BSD 3-Clause License. See accompanying -# file Copyright.txt or https://cmake.org/licensing for details. - -#[=======================================================================[.rst: -FindCairo ------------ - -Find Cairo 2D graphics library. - - -Imported Targets -^^^^^^^^^^^^^^^^ -This module defines :prop_tgt:`IMPORTED` target ``Cairo::Cairo``, if -cairo has been found. - -Result variables -^^^^^^^^^^^^^^^^ - -This module will set the following variables in your project: - -``CAIRO_FOUND`` - True if cairo headers and library were found. -``CAIRO_INCLUDE_DIRS`` - Directory where cairo headers are located. -``CAIRO_LIBRARIES`` - cairo libraries to link against. -``CAIRO_VERSION_MAJOR`` - The major version of cairo -``CAIRO_VERSION_MINOR`` - The minor version of cairo -``CAIRO_VERSION_PATCH`` - The patch version of cairo -``CAIRO_VERSION_STRING`` - version number as a string (ex: "1.16.0") -#]=======================================================================] - -if(NOT CAIRO_LIBRARY) - find_path(CAIRO_INCLUDE_DIR NAMES cairo.h HINTS ${PC_CAIRO_INCLUDEDIR} ${PC_CAIRO_INCLUDE_DIR} PATH_SUFFIXES cairo) - find_library(CAIRO_LIBRARY_RELEASE NAMES ${Cairo_NAMES} cairo HINTS ${PC_CAIRO_LIBDIR} ${PC_CAIRO_LIBRARY_DIRS}) - find_library(CAIRO_LIBRARY_DEBUG NAMES ${Cairo_NAMES} cairod HINTS ${PC_CAIRO_LIBDIR} ${PC_CAIRO_LIBRARY_DIRS}) - include(SelectLibraryConfigurations) - select_library_configurations(CAIRO) -else() - file(TO_CMAKE_PATH "${CAIRO_LIBRARY}" CAIRO_LIBRARY) -endif() - -if(CAIRO_INCLUDE_DIR AND NOT CAIRO_VERSION) - if(EXISTS "${CAIRO_INCLUDE_DIR}/cairo-version.h") - file(READ "${CAIRO_INCLUDE_DIR}/cairo-version.h" CAIRO_VERSION_CONTENT) - - string(REGEX MATCH "#define +CAIRO_VERSION_MAJOR +([0-9]+)" _dummy "${CAIRO_VERSION_CONTENT}") - set(CAIRO_VERSION_MAJOR "${CMAKE_MATCH_1}") - - string(REGEX MATCH "#define +CAIRO_VERSION_MINOR +([0-9]+)" _dummy "${CAIRO_VERSION_CONTENT}") - set(CAIRO_VERSION_MINOR "${CMAKE_MATCH_1}") - - string(REGEX MATCH "#define +CAIRO_VERSION_MICRO +([0-9]+)" _dummy "${CAIRO_VERSION_CONTENT}") - set(CAIRO_VERSION_PATCH "${CMAKE_MATCH_1}") - - set(CAIRO_VERSION "${CAIRO_VERSION_MAJOR}.${CAIRO_VERSION_MINOR}.${CAIRO_VERSION_PATCH}") - set(CAIRO_VERSION_STRING ${CAIRO_VERSION}) - endif() -endif() - -include(FindPackageHandleStandardArgs) -find_package_handle_standard_args(Cairo - REQUIRED_VARS - CAIRO_LIBRARY - CAIRO_INCLUDE_DIR - VERSION_VAR - CAIRO_VERSION_STRING -) -mark_as_advanced(CAIRO_INCLUDE_DIR CAIRO_LIBRARY) - -if (CAIRO_FOUND) - set(CAIRO_LIBRARIES ${CAIRO_LIBRARY}) - set(CAIRO_INCLUDE_DIRS ${CAIRO_INCLUDE_DIR}) - if(NOT TARGET Cairo::Cairo) - add_library(Cairo::Cairo UNKNOWN IMPORTED) - set_target_properties(Cairo::Cairo PROPERTIES - INTERFACE_INCLUDE_DIRECTORIES ${CAIRO_INCLUDE_DIR} - IMPORTED_LINK_INTERFACE_LANGUAGES C) - - if(CAIRO_LIBRARY_RELEASE) - set_property(TARGET Cairo::Cairo APPEND PROPERTY IMPORTED_CONFIGURATIONS RELEASE) - set_target_properties(Cairo::Cairo PROPERTIES IMPORTED_LOCATION_RELEASE "${CAIRO_LIBRARY_RELEASE}") - endif() - - if(CAIRO_LIBRARY_DEBUG) - set_property(TARGET Cairo::Cairo APPEND PROPERTY IMPORTED_CONFIGURATIONS DEBUG) - set_target_properties(Cairo::Cairo PROPERTIES IMPORTED_LOCATION_DEBUG "${CAIRO_LIBRARY_DEBUG}") - endif() - - if(NOT CAIRO_LIBRARY_RELEASE AND NOT CAIRO_LIBRARY_DEBUG) - set_target_properties(Cairo::Cairo PROPERTIES IMPORTED_LOCATION "${CAIRO_LIBRARY}") - endif() - endif() -endif () diff --git a/cmake/FindWebP.cmake b/cmake/FindWebP.cmake deleted file mode 100644 index 9fc3ac841..000000000 --- a/cmake/FindWebP.cmake +++ /dev/null @@ -1,97 +0,0 @@ -# Distributed under the OSI-approved BSD 3-Clause License. See accompanying -# file Copyright.txt or https://cmake.org/licensing for details. - -#[=======================================================================[.rst: -FindWebP -------- - -Finds the WebP library. - -Imported Targets -^^^^^^^^^^^^^^^^ - -This module provides the following imported targets, if found: - -``WebP::WebP`` - The WebP library - -Result Variables -^^^^^^^^^^^^^^^^ - -This will define the following variables: - -``WebP_FOUND`` - True if the system has the WebP library. -``WebP_VERSION`` - The version of the WebP library which was found. -``WebP_INCLUDE_DIRS`` - Include directories needed to use WebP. -``WebP_LIBRARIES`` - Libraries needed to link to WebP. - -Cache Variables -^^^^^^^^^^^^^^^ - -The following cache variables may also be set: - -``WebP_INCLUDE_DIR`` - The directory containing ``decode.h``. -``WebP_LIBRARY`` - The path to the Foo library. - -#]=======================================================================] - -if(NOT WebP_LIBRARY) - find_package(PkgConfig QUIET) - pkg_check_modules(PC_WebP QUIET libwebp) - set(WebP_VERSION ${PC_WebP_VERSION}) - find_path(WebP_INCLUDE_DIR NAMES decode.h HINTS ${PC_WebP_INCLUDEDIR} ${PC_WebP_INCLUDE_DIR} PATH_SUFFIXES webp) - find_library(WebP_LIBRARY_RELEASE NAMES ${WebP_NAMES} webp HINTS ${PC_WebP_LIBDIR} ${PC_WebP_LIBRARY_DIRS}) - find_library(WebP_LIBRARY_DEBUG NAMES ${WebP_NAMES} webpd HINTS ${PC_WebP_LIBDIR} ${PC_WebP_LIBRARY_DIRS}) - include(SelectLibraryConfigurations) - select_library_configurations(WebP) -else() - file(TO_CMAKE_PATH "${WebP_LIBRARY}" WebP_LIBRARY) -endif() - -if ("${WebP_FIND_VERSION}" VERSION_GREATER "${WebP_VERSION}") - if (WebP_VERSION) - message(FATAL_ERROR "Required version (" ${WebP_FIND_VERSION} ") is higher than found version (" ${PC_WebP_VERSION} ")") - else () - message(WARNING "Cannot determine WebP version without pkg-config") - endif () -endif () - -include(FindPackageHandleStandardArgs) -find_package_handle_standard_args(WebP - REQUIRED_VARS - WebP_LIBRARY - WebP_INCLUDE_DIR - VERSION_VAR WebP_VERSION -) -mark_as_advanced(WebP_INCLUDE_DIR WebP_LIBRARY) - -if (WebP_FOUND) - set(WebP_LIBRARIES ${WebP_LIBRARY}) - set(WebP_INCLUDE_DIRS ${WebP_INCLUDE_DIR}) - if(NOT TARGET WebP::WebP) - add_library(WebP::WebP UNKNOWN IMPORTED) - set_target_properties(WebP::WebP PROPERTIES - INTERFACE_INCLUDE_DIRECTORIES ${WebP_INCLUDE_DIR} - IMPORTED_LINK_INTERFACE_LANGUAGES C) - - if(WebP_LIBRARY_RELEASE) - set_property(TARGET WebP::WebP APPEND PROPERTY IMPORTED_CONFIGURATIONS RELEASE) - set_target_properties(WebP::WebP PROPERTIES IMPORTED_LOCATION_RELEASE "${WebP_LIBRARY_RELEASE}") - endif() - - if(WebP_LIBRARY_DEBUG) - set_property(TARGET WebP::WebP APPEND PROPERTY IMPORTED_CONFIGURATIONS DEBUG) - set_target_properties(WebP::WebP PROPERTIES IMPORTED_LOCATION_DEBUG "${WebP_LIBRARY_DEBUG}") - endif() - - if(NOT WebP_LIBRARY_RELEASE AND NOT WebP_LIBRARY_DEBUG) - set_target_properties(WebP::WebP PROPERTIES IMPORTED_LOCATION "${WebP_LIBRARY}") - endif() - endif() -endif () diff --git a/cmake/MapnikExport.cmake b/cmake/MapnikExport.cmake index 5c083bba9..34c5573b7 100644 --- a/cmake/MapnikExport.cmake +++ b/cmake/MapnikExport.cmake @@ -1,34 +1,46 @@ include(CMakePackageConfigHelpers) -# export mapnik configuration -write_basic_package_version_file( - "${CMAKE_CURRENT_BINARY_DIR}/mapnikConfigVersion.cmake" - VERSION ${MAPNIK_VERSION} - COMPATIBILITY ExactVersion -) -get_property(MAPNIK_UTILITIES GLOBAL PROPERTY MAPNIK_UTILITIES) -list(JOIN MAPNIK_DEPENDENCIES "\n" MAPNIK_DEPENDENCIES) -configure_package_config_file(${CMAKE_CURRENT_SOURCE_DIR}/cmake/mapnikConfig.cmake.in - "${CMAKE_CURRENT_BINARY_DIR}/mapnikConfig.cmake" - INSTALL_DESTINATION ${MAPNIK_CMAKE_DIR} - PATH_VARS MAPNIK_INCLUDE_DIR PLUGINS_INSTALL_DIR FONTS_INSTALL_DIR MAPNIK_DEPENDENCIES MAPNIK_UTILITIES - NO_CHECK_REQUIRED_COMPONENTS_MACRO -) - -install( - FILES - "${CMAKE_CURRENT_BINARY_DIR}/mapnikConfig.cmake" +### exports mapnik cmake config files (mapnikConfigVersion and mapnikConfig) +function(mapnik_export_cmake_config) + # export mapnik configuration + write_basic_package_version_file( "${CMAKE_CURRENT_BINARY_DIR}/mapnikConfigVersion.cmake" - DESTINATION ${MAPNIK_CMAKE_DIR} -) + VERSION ${MAPNIK_VERSION} + COMPATIBILITY ExactVersion + ) + get_property(MAPNIK_UTILITIES GLOBAL PROPERTY MAPNIK_UTILITIES) -# install our modules, so that the expected target names are found. -install( - FILES - "${CMAKE_CURRENT_SOURCE_DIR}/cmake/FindCairo.cmake" - "${CMAKE_CURRENT_SOURCE_DIR}/cmake/FindWebP.cmake" - DESTINATION ${MAPNIK_CMAKE_DIR}/Modules -) + # generate all find_dependency and pkg_config calls + set(mapnik_find_deps) + foreach(dep IN LISTS mapnik_deps) + set(ver_comment "# ${dep} used with version ${mapnik_${dep}_version}") + set(mapnik_find_deps "${mapnik_find_deps}\n${ver_comment}\n") + if(mapnik_${dep}_find_args) + list(REMOVE_DUPLICATES mapnik_${dep}_find_args) + list(JOIN mapnik_${dep}_find_args " " m_args_joined) + set(mapnik_find_deps "${mapnik_find_deps}find_dependency(${dep} ${m_args_joined})") + else() + list(JOIN mapnik_${dep}_pkg_args " " m_args_joined) + set(mapnik_find_deps "${mapnik_find_deps}pkg_check_modules(${dep} ${m_args_joined})") + endif() + endforeach() + + configure_package_config_file("${CMAKE_CURRENT_SOURCE_DIR}/cmake/mapnikConfig.cmake.in" + "${CMAKE_CURRENT_BINARY_DIR}/mapnikConfig.cmake" + INSTALL_DESTINATION ${MAPNIK_CMAKE_DIR} + PATH_VARS MAPNIK_INCLUDE_DIR PLUGINS_INSTALL_DIR FONTS_INSTALL_DIR mapnik_find_deps MAPNIK_UTILITIES + NO_CHECK_REQUIRED_COMPONENTS_MACRO + ) + install( + FILES + "${CMAKE_CURRENT_BINARY_DIR}/mapnikConfig.cmake" + "${CMAKE_CURRENT_BINARY_DIR}/mapnikConfigVersion.cmake" + DESTINATION ${MAPNIK_CMAKE_DIR} + ) +endfunction() + + +mapnik_export_cmake_config() install(EXPORT MapnikTargets DESTINATION ${MAPNIK_CMAKE_DIR} @@ -36,21 +48,21 @@ install(EXPORT MapnikTargets NAMESPACE mapnik:: ) - +### install plugin cmake config files ### # Create configuration dependend files for the plugin install dirs. # some package managers are using different paths per configuration. string(TOLOWER "${CMAKE_BUILD_TYPE}" _build_type) string(TOUPPER "${CMAKE_BUILD_TYPE}" _build_type_l) -set(_mapnik_plugin_file_name "mapnikPlugins-${_build_type}") -file(WRITE ${CMAKE_CURRENT_BINARY_DIR}/${_mapnik_plugin_file_name}.cmake.in "set(MAPNIK_PLUGINS_DIR_${_build_type_l} \"@PACKAGE_PLUGINS_INSTALL_DIR@\" CACHE STRING \"\")\n") +set(m_mapnik_plugin_file_name mapnikPlugins-${_build_type}) +file(WRITE "${CMAKE_CURRENT_BINARY_DIR}/${m_mapnik_plugin_file_name}.cmake.in" "set(MAPNIK_PLUGINS_DIR_${_build_type_l} \"@PACKAGE_PLUGINS_INSTALL_DIR@\" CACHE STRING \"\")\n") include(CMakePackageConfigHelpers) configure_package_config_file( - ${CMAKE_CURRENT_BINARY_DIR}/${_mapnik_plugin_file_name}.cmake.in - ${CMAKE_CURRENT_BINARY_DIR}/${_mapnik_plugin_file_name}.cmake + "${CMAKE_CURRENT_BINARY_DIR}/${m_mapnik_plugin_file_name}.cmake.in" + "${CMAKE_CURRENT_BINARY_DIR}/${m_mapnik_plugin_file_name}.cmake" PATH_VARS PLUGINS_INSTALL_DIR INSTALL_DESTINATION ${MAPNIK_CMAKE_DIR} ) install( - FILES ${CMAKE_CURRENT_BINARY_DIR}/${_mapnik_plugin_file_name}.cmake + FILES "${CMAKE_CURRENT_BINARY_DIR}/${m_mapnik_plugin_file_name}.cmake" DESTINATION ${MAPNIK_CMAKE_DIR} ) diff --git a/cmake/MapnikFindPackage.cmake b/cmake/MapnikFindPackage.cmake index 8bcd96216..e80ad44c1 100644 --- a/cmake/MapnikFindPackage.cmake +++ b/cmake/MapnikFindPackage.cmake @@ -1,28 +1,50 @@ -macro(mapnik_print_version) - string(TOUPPER ${ARGV0} TLNUP) - set(TLN ${ARGV0}) - if(${TLN}_VERSION_STRING) - message(STATUS "Using ${ARGV0} version: ${${TLN}_VERSION_STRING}") - elseif(${TLN}_VERSION) - message(STATUS "Using ${ARGV0} version: ${${TLN}_VERSION}") - elseif(${TLNUP}_VERSION_STRING) - message(STATUS "Using ${ARGV0} version: ${${TLNUP}_VERSION_STRING}") - elseif(${TLNUP}_VERSION) - message(STATUS "Using ${ARGV0} version: ${${TLNUP}_VERSION}") +function(mapnik_set_dep_version dep var) + string(TOUPPER ${dep} m_package_name_upc) + set(m_package_name ${dep}) + if(${m_package_name}_VERSION_STRING) + set(${var} ${${m_package_name}_VERSION_STRING} PARENT_SCOPE) + elseif(${m_package_name}_VERSION) + set(${var} ${${m_package_name}_VERSION} PARENT_SCOPE) + elseif(${m_package_name_upc}_VERSION_STRING) + set(${var} ${${m_package_name_upc}_VERSION_STRING} PARENT_SCOPE) + elseif(${m_package_name_upc}_VERSION) + set(${var} ${${m_package_name_upc}_VERSION} PARENT_SCOPE) + endif() +endfunction() + +function(mapnik_print_package_info dep) + message(STATUS "Using ${dep} version: ${mapnik_${dep}_version}") +endfunction() + +macro(mapnik_find_package dep) + find_package(${dep} ${ARGN}) + if(${dep}_FOUND) + list(APPEND mapnik_deps ${dep}) + + if(mapnik_${dep}_find_args) + list(APPEND mapnik_${dep}_find_args ${ARGN}) + else() + set(mapnik_${dep}_find_args ${ARGN}) + endif() + mapnik_set_dep_version(${dep} mapnik_${dep}_version) + + mapnik_print_package_info(${dep}) else() - message(STATUS "Using ${ARGV0}") + message(STATUS "not found: ${dep}") endif() endmacro() -macro(mapnik_find_package) - find_package(${ARGN}) - if(${ARGV0}_FOUND) - set(MAPNIK_TMP_DEP ${ARGN}) - list(JOIN MAPNIK_TMP_DEP " " MAPNIK_TMP_DEP) - list(APPEND MAPNIK_DEPENDENCIES "find_dependency(${MAPNIK_TMP_DEP})") - mapnik_print_version(${ARGV0}) +macro(mapnik_pkg_check_modules dep) + pkg_check_modules(${dep} ${ARGN}) + if(${dep}_FOUND) + list(APPEND mapnik_deps ${dep}) + + set(mapnik_${dep}_pkg_args ${ARGN}) + mapnik_set_dep_version(${dep} mapnik_${dep}_version) + + mapnik_print_package_info(${dep}) else() - message(STATUS "not found: ${ARGV0}") + message(STATUS "not found: ${dep}") endif() endmacro() diff --git a/cmake/MapnikInstall.cmake b/cmake/MapnikInstall.cmake index b0a781116..43e01b8dd 100644 --- a/cmake/MapnikInstall.cmake +++ b/cmake/MapnikInstall.cmake @@ -23,6 +23,9 @@ endfunction() # Install plugins # function(mapnik_install_plugin _target) + if(NOT BUILD_SHARED_PLUGINS) + return() + endif() install(TARGETS ${_target} RUNTIME DESTINATION ${PLUGINS_INSTALL_DIR} COMPONENT MapnikPluginRuntime diff --git a/cmake/mapnikConfig.cmake.in b/cmake/mapnikConfig.cmake.in index e5553f964..5ec8e38a3 100644 --- a/cmake/mapnikConfig.cmake.in +++ b/cmake/mapnikConfig.cmake.in @@ -8,7 +8,7 @@ set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_CURRENT_LIST_DIR}/Modules/") include(CMakeFindDependencyMacro) find_dependency(Threads REQUIRED) -@MAPNIK_DEPENDENCIES@ +@mapnik_find_deps@ include("${CMAKE_CURRENT_LIST_DIR}/mapnikTargets.cmake") diff --git a/demo/c++/CMakeLists.txt b/demo/c++/CMakeLists.txt index 7bd9397c9..25bfd356d 100644 --- a/demo/c++/CMakeLists.txt +++ b/demo/c++/CMakeLists.txt @@ -1,8 +1,6 @@ add_executable(mapnik-demo rundemo.cpp) -set_target_properties(mapnik-demo PROPERTIES - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" +target_link_libraries(mapnik-demo PRIVATE + mapnik::agg + mapnik::mapnik + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) - -target_link_libraries(mapnik-demo PRIVATE mapnik::agg mapnik::mapnik) diff --git a/demo/c++/rundemo.cpp b/demo/c++/rundemo.cpp index 022442dec..ab4413161 100644 --- a/demo/c++/rundemo.cpp +++ b/demo/c++/rundemo.cpp @@ -20,6 +20,7 @@ * *****************************************************************************/ +#include #include #include #include @@ -52,7 +53,7 @@ int main(int, char**) "+proj=lcc +ellps=GRS80 +lat_0=49 +lon_0=-95 +lat+1=49 +lat_2=77 +datum=NAD83 +units=m +no_defs"; const std::string srs_merc = "+proj=merc +a=6378137 +b=6378137 +lat_ts=0.0 +lon_0=0.0 +x_0=0.0 +y_0=0.0 +k=1.0 " "+units=m +nadgrids=@null +wktext +no_defs +over"; - + mapnik::setup(); try { std::cout << " running demo ... \n"; diff --git a/demo/viewer/CMakeLists.txt b/demo/viewer/CMakeLists.txt index 33d33d9ec..89c0865cb 100644 --- a/demo/viewer/CMakeLists.txt +++ b/demo/viewer/CMakeLists.txt @@ -40,15 +40,13 @@ set_target_properties(mapnik-viewer PROPERTIES AUTORCC ON AUTOUIC ON AUTOMOC ON - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" ) target_link_libraries(mapnik-viewer PRIVATE Qt${QT_VERSION_MAJOR}::Widgets mapnik::agg mapnik::mapnik + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) file(WRITE ${CMAKE_CURRENT_BINARY_DIR}/viewer.ini diff --git a/demo/viewer/main.cpp b/demo/viewer/main.cpp index aa73b9ce3..114655d79 100644 --- a/demo/viewer/main.cpp +++ b/demo/viewer/main.cpp @@ -23,12 +23,14 @@ #include #include #include +#include #include "mainwindow.hpp" int main(int argc, char** argv) { using mapnik::datasource_cache; using mapnik::freetype_engine; + mapnik::setup(); try { #if QT_VERSION >= QT_VERSION_CHECK(5, 6, 0) diff --git a/include/mapnik/config.hpp b/include/mapnik/config.hpp index 0370de1f8..312d72ffb 100644 --- a/include/mapnik/config.hpp +++ b/include/mapnik/config.hpp @@ -24,32 +24,40 @@ #define MAPNIK_CONFIG_HPP // Windows DLL support - +// clang-format off #ifdef _WIN32 -#define MAPNIK_EXP __declspec(dllexport) -#define MAPNIK_IMP __declspec(dllimport) -#ifdef MAPNIK_EXPORTS -#define MAPNIK_DECL __declspec(dllexport) +#ifdef MAPNIK_STATIC_DEFINE +# define MAPNIK_DECL +# define MAPNIK_EXP #else -#define MAPNIK_DECL __declspec(dllimport) -#endif -#pragma warning(disable: 4251) -#pragma warning(disable: 4275) -#if (_MSC_VER >= 1400) // vc8 -#pragma warning(disable: 4996) //_CRT_SECURE_NO_DEPRECATE +# define MAPNIK_EXP __declspec(dllexport) +# ifndef MAPNIK_DECL +# ifdef MAPNIK_EXPORTS + /* We are building this library */ +# define MAPNIK_DECL __declspec(dllexport) +# else + /* We are using this library */ +# define MAPNIK_DECL __declspec(dllimport) +# endif +# endif #endif +# pragma warning( disable: 4251 ) +# pragma warning( disable: 4275 ) +# if (_MSC_VER >= 1400) // vc8 +# pragma warning(disable : 4996) //_CRT_SECURE_NO_DEPRECATE +# endif #else -#if __GNUC__ >= 4 -#define MAPNIK_EXP __attribute__((visibility("default"))) -#define MAPNIK_DECL __attribute__((visibility("default"))) -#define MAPNIK_IMP __attribute__((visibility("default"))) -#else -#define MAPNIK_EXP -#define MAPNIK_DECL -#define MAPNIK_IMP +# if __GNUC__ >= 4 +# define MAPNIK_EXP __attribute__ ((visibility ("default"))) +# define MAPNIK_DECL __attribute__ ((visibility ("default"))) +# define MAPNIK_IMP __attribute__ ((visibility ("default"))) +# else +# define MAPNIK_EXP +# define MAPNIK_DECL +# define MAPNIK_IMP +# endif #endif -#endif - +// clang-format on #define PROJ_ENVELOPE_POINTS 20 #endif // MAPNIK_CONFIG_HPP diff --git a/include/mapnik/css/css_color_grammar_x3_def.hpp b/include/mapnik/css/css_color_grammar_x3_def.hpp index 2b07f0dea..eec9abae3 100644 --- a/include/mapnik/css/css_color_grammar_x3_def.hpp +++ b/include/mapnik/css/css_color_grammar_x3_def.hpp @@ -48,11 +48,9 @@ namespace css_color_grammar { using x3::attr; using x3::double_; -using x3::hex; using x3::lit; using x3::no_case; using x3::no_skip; -using x3::omit; using x3::symbols; using x3::uint_parser; @@ -214,11 +212,11 @@ struct named_colors_ : x3::symbols ; // clang-format on } -} named_colors; +} const named_colors; -x3::uint_parser hex2; -x3::uint_parser hex1; -x3::uint_parser dec3; +const x3::uint_parser hex2; +const x3::uint_parser hex1; +const x3::uint_parser dec3; // rules x3::rule const hex2_color("hex2_color"); @@ -245,67 +243,67 @@ struct percent_converter static std::uint8_t call(double val) { return safe_cast(std::lround((255.0 * val) / 100.0)); } }; -auto dec_red = [](auto& ctx) { +const auto dec_red = [](auto& ctx) { _val(ctx).red_ = _attr(ctx); }; -auto dec_green = [](auto& ctx) { +const auto dec_green = [](auto& ctx) { _val(ctx).green_ = _attr(ctx); }; -auto dec_blue = [](auto& ctx) { +const auto dec_blue = [](auto& ctx) { _val(ctx).blue_ = _attr(ctx); }; -auto opacity = [](auto& ctx) { +const auto opacity = [](auto& ctx) { _val(ctx).alpha_ = uint8_t((255.0 * clip_opacity::call(_attr(ctx))) + 0.5); }; -auto percent_red = [](auto& ctx) { +const auto percent_red = [](auto& ctx) { _val(ctx).red_ = percent_converter::call(_attr(ctx)); }; -auto percent_green = [](auto& ctx) { +const auto percent_green = [](auto& ctx) { _val(ctx).green_ = percent_converter::call(_attr(ctx)); }; -auto percent_blue = [](auto& ctx) { +const auto percent_blue = [](auto& ctx) { _val(ctx).blue_ = percent_converter::call(_attr(ctx)); }; -auto hex1_red = [](auto& ctx) { +const auto hex1_red = [](auto& ctx) { _val(ctx).red_ = _attr(ctx) | _attr(ctx) << 4; }; -auto hex1_green = [](auto& ctx) { +const auto hex1_green = [](auto& ctx) { _val(ctx).green_ = _attr(ctx) | _attr(ctx) << 4; }; -auto hex1_blue = [](auto& ctx) { +const auto hex1_blue = [](auto& ctx) { _val(ctx).blue_ = _attr(ctx) | _attr(ctx) << 4; }; -auto hex1_opacity = [](auto& ctx) { +const auto hex1_opacity = [](auto& ctx) { _val(ctx).alpha_ = _attr(ctx) | _attr(ctx) << 4; }; -auto hex2_red = [](auto& ctx) { +const auto hex2_red = [](auto& ctx) { _val(ctx).red_ = _attr(ctx); }; -auto hex2_green = [](auto& ctx) { +const auto hex2_green = [](auto& ctx) { _val(ctx).green_ = _attr(ctx); }; -auto hex2_blue = [](auto& ctx) { +const auto hex2_blue = [](auto& ctx) { _val(ctx).blue_ = _attr(ctx); }; -auto hex2_opacity = [](auto& ctx) { +const auto hex2_opacity = [](auto& ctx) { _val(ctx).alpha_ = _attr(ctx); }; -auto hsl_to_rgba = [](auto& ctx) { +const auto hsl_to_rgba = [](auto& ctx) { double h = std::get<0>(_attr(ctx)); double s = std::get<1>(_attr(ctx)); double l = std::get<2>(_attr(ctx)); @@ -325,10 +323,10 @@ auto hsl_to_rgba = [](auto& ctx) { } m1 = l * 2 - m2; - double r = hue_to_rgb(m1, m2, h + 1.0 / 3.0); - double g = hue_to_rgb(m1, m2, h); - double b = hue_to_rgb(m1, m2, h - 1.0 / 3.0); - uint8_t alpha = uint8_t((255.0 * clip_opacity::call(std::get<3>(_attr(ctx)))) + 0.5); + const double r = hue_to_rgb(m1, m2, h + 1.0 / 3.0); + const double g = hue_to_rgb(m1, m2, h); + const double b = hue_to_rgb(m1, m2, h - 1.0 / 3.0); + const uint8_t alpha = uint8_t((255.0 * clip_opacity::call(std::get<3>(_attr(ctx)))) + 0.5); _val(ctx) = color(safe_cast(std::lround(255.0 * r)), safe_cast(std::lround(255.0 * g)), safe_cast(std::lround(255.0 * b)), diff --git a/include/mapnik/css/css_grammar_x3_def.hpp b/include/mapnik/css/css_grammar_x3_def.hpp index a4f533c47..2e18c9fb7 100644 --- a/include/mapnik/css/css_grammar_x3_def.hpp +++ b/include/mapnik/css/css_grammar_x3_def.hpp @@ -150,32 +150,27 @@ namespace css_grammar { using x3::alnum; using x3::alpha; -using x3::attr; using x3::char_; using x3::lexeme; using x3::lit; -using x3::no_case; -using x3::no_skip; using x3::raw; using x3::standard::space; // import unicode string rule -namespace { -auto const& css_string = mapnik::json::grammar::unicode_string; -} +const auto css_string = mapnik::json::grammar::unicode_string; -auto assign_def = [](auto const& ctx) { +const auto assign_def = [](auto const& ctx) { for (auto const& k : std::get<0>(_attr(ctx))) { _val(ctx).emplace(k, std::get<1>(_attr(ctx))); } }; -auto assign_key = [](auto const& ctx) { +const auto assign_key = [](auto const& ctx) { _val(ctx).first = std::move(_attr(ctx)); }; -auto assign_value = [](auto const& ctx) { +const auto assign_value = [](auto const& ctx) { _val(ctx).second = std::move(_attr(ctx)); }; diff --git a/include/mapnik/csv/csv_grammar_x3_def.hpp b/include/mapnik/csv/csv_grammar_x3_def.hpp index 2fc1f021e..1d9a20135 100644 --- a/include/mapnik/csv/csv_grammar_x3_def.hpp +++ b/include/mapnik/csv/csv_grammar_x3_def.hpp @@ -39,9 +39,9 @@ using standard::char_; using x3::lexeme; using x3::lit; -struct unesc_char_ : x3::symbols +struct unesc_char_csv_ : x3::symbols { - unesc_char_() + unesc_char_csv_() { add("\\a", '\a') // ("\\b", '\b') // @@ -56,7 +56,7 @@ struct unesc_char_ : x3::symbols ("\"\"", '\"') // double quote ; } -} unesc_char; +} const unesc_char_csv; template struct literal : x3::parser> @@ -93,7 +93,7 @@ auto const column_def = quoted_text | *(char_ - separator); auto const quoted_text_def = quote > text > quote // support unmatched quotes or not (??) ; -auto const text_def = *(unesc_char | (char_ - quote)); +auto const text_def = *(unesc_char_csv | (char_ - quote)); BOOST_SPIRIT_DEFINE(line, column, quoted_text, text); diff --git a/include/mapnik/datasource.hpp b/include/mapnik/datasource.hpp index 6dd82e088..b08a5f993 100644 --- a/include/mapnik/datasource.hpp +++ b/include/mapnik/datasource.hpp @@ -113,25 +113,6 @@ class datasource_deleter }; using datasource_ptr = std::shared_ptr; - -#ifdef MAPNIK_STATIC_PLUGINS -#define DATASOURCE_PLUGIN(classname) -#else -#define DATASOURCE_PLUGIN(classname) \ - extern "C" MAPNIK_EXP const char* datasource_name() \ - { \ - return classname::name(); \ - } \ - extern "C" MAPNIK_EXP datasource* create(parameters const& params) \ - { \ - return new classname(params); \ - } \ - extern "C" MAPNIK_EXP void destroy(datasource* ds) \ - { \ - delete ds; \ - } -#endif - } // namespace mapnik #endif // MAPNIK_DATASOURCE_HPP diff --git a/include/mapnik/datasource_cache.hpp b/include/mapnik/datasource_cache.hpp index 8504b05c2..4e321c9ad 100644 --- a/include/mapnik/datasource_cache.hpp +++ b/include/mapnik/datasource_cache.hpp @@ -47,7 +47,8 @@ class MAPNIK_DECL datasource_cache : public singleton; public: - std::vector plugin_names(); + bool plugin_registered(const std::string& plugin_name) const; + std::vector plugin_names() const; std::string plugin_directories(); bool register_datasources(std::string const& path, bool recurse = false); bool register_datasource(std::string const& path); @@ -62,7 +63,7 @@ class MAPNIK_DECL datasource_cache : public singleton; diff --git a/include/mapnik/expression_grammar_x3_def.hpp b/include/mapnik/expression_grammar_x3_def.hpp index 9b6d28f74..56ab3e348 100644 --- a/include/mapnik/expression_grammar_x3_def.hpp +++ b/include/mapnik/expression_grammar_x3_def.hpp @@ -51,23 +51,17 @@ namespace grammar { namespace x3 = boost::spirit::x3; namespace ascii = boost::spirit::x3::ascii; using ascii::char_; -using ascii::string; using x3::_attr; using x3::_val; using x3::alnum; using x3::alpha; -using x3::bool_; -using x3::double_; -using x3::int_; using x3::lexeme; using x3::lit; using x3::no_case; using x3::no_skip; x3::uint_parser const hex2{}; -namespace { -auto const& escaped_unicode = json::grammar::escaped_unicode; -} +const auto escaped_unicode = json::grammar::escaped_unicode; template inline mapnik::transcoder const& extract_transcoder(Context const& ctx) @@ -75,19 +69,19 @@ inline mapnik::transcoder const& extract_transcoder(Context const& ctx) return x3::get(ctx); } -auto append = [](auto const& ctx) { +const auto append = [](auto const& ctx) { _val(ctx) += _attr(ctx); }; -auto do_assign = [](auto const& ctx) { +const auto do_assign = [](auto const& ctx) { _val(ctx) = std::move(_attr(ctx)); }; -auto do_negate = [](auto const& ctx) { +const auto do_negate = [](auto const& ctx) { _val(ctx) = std::move(unary_node(_attr(ctx))); }; -auto do_attribute = [](auto const& ctx) { +const auto do_attribute = [](auto const& ctx) { auto const& attr = _attr(ctx); if (attr == "mapnik::geometry_type") { @@ -99,84 +93,84 @@ auto do_attribute = [](auto const& ctx) { } }; -auto do_global_attribute = [](auto const& ctx) { +const auto do_global_attribute = [](auto const& ctx) { _val(ctx) = std::move(global_attribute(_attr(ctx))); }; -auto do_add = [](auto const& ctx) { +const auto do_add = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_subt = [](auto const& ctx) { +const auto do_subt = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_mult = [](auto const& ctx) { +const auto do_mult = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_div = [](auto const& ctx) { +const auto do_div = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_mod = [](auto const& ctx) { +const auto do_mod = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_unicode = [](auto const& ctx) { +const auto do_unicode = [](auto const& ctx) { auto const& tr = extract_transcoder(ctx); _val(ctx) = std::move(tr.transcode(_attr(ctx).c_str())); }; -auto do_null = [](auto const& ctx) { +const auto do_null = [](auto const& ctx) { _val(ctx) = std::move(mapnik::value_null()); }; -auto do_not = [](auto const& ctx) { +const auto do_not = [](auto const& ctx) { mapnik::unary_node node(_attr(ctx)); _val(ctx) = std::move(node); }; -auto do_and = [](auto const& ctx) { +const auto do_and = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_or = [](auto const& ctx) { +const auto do_or = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_equal = [](auto const& ctx) { +const auto do_equal = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_not_equal = [](auto const& ctx) { +const auto do_not_equal = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_less = [](auto const& ctx) { +const auto do_less = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_less_equal = [](auto const& ctx) { +const auto do_less_equal = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_greater = [](auto const& ctx) { +const auto do_greater = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_greater_equal = [](auto const& ctx) { +const auto do_greater_equal = [](auto const& ctx) { _val(ctx) = std::move(mapnik::binary_node(std::move(_val(ctx)), std::move(_attr(ctx)))); }; // regex -auto do_regex_match = [](auto const& ctx) { +const auto do_regex_match = [](auto const& ctx) { auto const& tr = extract_transcoder(ctx); _val(ctx) = std::move(mapnik::regex_match_node(tr, std::move(_val(ctx)), std::move(_attr(ctx)))); }; -auto do_regex_replace = [](auto const& ctx) { +const auto do_regex_replace = [](auto const& ctx) { auto const& tr = extract_transcoder(ctx); auto const& pair = _attr(ctx); auto const& pattern = std::get<0>(pair); @@ -197,7 +191,7 @@ struct boolean_ : x3::symbols ("false", false) // ; } -} boolean; +} const boolean; struct floating_point_constants : x3::symbols { @@ -208,7 +202,7 @@ struct floating_point_constants : x3::symbols ("rad_to_deg", 57.295779513082320876798154814105) // ; } -} float_const; +} const float_const; // unary functions struct unary_function_types_ : x3::symbols @@ -225,7 +219,7 @@ struct unary_function_types_ : x3::symbols ("length", length_impl()) // ; } -} unary_func_types; +} const unary_func_types; // binary functions @@ -238,7 +232,7 @@ struct binary_function_types_ : x3::symbols ("pow", binary_function_impl(pow_impl)) // ; } -} binary_func_types; +} const binary_func_types; // geometry types struct geometry_types_ : x3::symbols @@ -251,7 +245,7 @@ struct geometry_types_ : x3::symbols ("collection", 4) // ; } -} geometry_type; +} const geometry_type; struct unesc_chars_ : x3::symbols { @@ -269,7 +263,7 @@ struct unesc_chars_ : x3::symbols ("\\\"", '\"') // ; } -} unesc_char; +} const unesc_char; // rules x3::rule const logical_expression("logical expression"); diff --git a/include/mapnik/image_filter_grammar_x3_def.hpp b/include/mapnik/image_filter_grammar_x3_def.hpp index b51a40801..ecbf9826f 100644 --- a/include/mapnik/image_filter_grammar_x3_def.hpp +++ b/include/mapnik/image_filter_grammar_x3_def.hpp @@ -57,41 +57,35 @@ namespace x3 = boost::spirit::x3; namespace image_filter { -using x3::attr; -using x3::char_; using x3::double_; -using x3::hex; using x3::lit; -using x3::no_case; -using x3::no_skip; -using x3::omit; using x3::symbols; using x3::uint_parser; -auto push_back = [](auto& ctx) { +const auto push_back = [](auto& ctx) { _val(ctx).push_back(_attr(ctx)); }; -auto set_rx_ry = [](auto& ctx) { +const auto set_rx_ry = [](auto& ctx) { _val(ctx).rx = _val(ctx).ry = _attr(ctx); }; -auto set_ry = [](auto& ctx) { +const auto set_ry = [](auto& ctx) { _val(ctx).ry = _attr(ctx); }; -auto offset_value = [](auto& ctx) { +const auto offset_value = [](auto& ctx) { _val(ctx) = _attr(ctx); }; -auto percent = [](auto& ctx) { +const auto percent = [](auto& ctx) { double val = std::abs(_val(ctx) / 100.0); if (val > 1.0) val = 1.0; _val(ctx) = val; }; -x3::uint_parser radius; +const x3::uint_parser radius; // Import the expression rule namespace { diff --git a/include/mapnik/json/feature_grammar_x3_def.hpp b/include/mapnik/json/feature_grammar_x3_def.hpp index ac5ad68fe..a3b518928 100644 --- a/include/mapnik/json/feature_grammar_x3_def.hpp +++ b/include/mapnik/json/feature_grammar_x3_def.hpp @@ -154,14 +154,14 @@ struct geometry_type_ : x3::symbols ("\"GeometryCollection\"", mapnik::geometry::geometry_types::GeometryCollection) // ; } -} geometry_type_symbols; +} const geometry_type_symbols; namespace { -auto assign_name = [](auto const& ctx) { +const auto assign_name = [](auto const& ctx) { std::get<0>(_val(ctx)) = std::move(_attr(ctx)); }; -auto assign_value = [](auto const& ctx) { +const auto assign_value = [](auto const& ctx) { std::get<1>(_val(ctx)) = std::move(_attr(ctx)); }; @@ -207,7 +207,7 @@ auto const assign_collection = [](auto const& ctx) { std::get<2>(_val(ctx)) = std::move(_attr(ctx)); }; -auto assign_property = [](auto const& ctx) { +const auto assign_property = [](auto const& ctx) { mapnik::feature_impl& feature = x3::get(ctx); mapnik::transcoder const& tr = x3::get(ctx); feature.put_new(std::get<0>(_attr(ctx)), diff --git a/include/mapnik/json/generic_json_grammar_x3_def.hpp b/include/mapnik/json/generic_json_grammar_x3_def.hpp index dcd1cd9be..ca7801b86 100644 --- a/include/mapnik/json/generic_json_grammar_x3_def.hpp +++ b/include/mapnik/json/generic_json_grammar_x3_def.hpp @@ -35,39 +35,36 @@ namespace x3 = boost::spirit::x3; namespace { -auto make_null = [](auto const& ctx) { +const auto make_null = [](auto const& ctx) { _val(ctx) = mapnik::value_null{}; }; -auto make_true = [](auto const& ctx) { +const auto make_true = [](auto const& ctx) { _val(ctx) = true; }; -auto make_false = [](auto const& ctx) { +const auto make_false = [](auto const& ctx) { _val(ctx) = false; }; -auto assign = [](auto const& ctx) { +const auto assign = [](auto const& ctx) { _val(ctx) = std::move(_attr(ctx)); }; -auto assign_key = [](auto const& ctx) { +const auto assign_key = [](auto const& ctx) { std::get<0>(_val(ctx)) = std::move(_attr(ctx)); }; -auto assign_value = [](auto const& ctx) { +const auto assign_value = [](auto const& ctx) { std::get<1>(_val(ctx)) = std::move(_attr(ctx)); }; } // namespace using x3::lit; -using x3::string; // import unicode string rule -namespace { -auto const& json_string = mapnik::json::grammar::unicode_string; -} +const auto json_string = mapnik::json::grammar::unicode_string; // rules x3::rule const object("JSON Object"); diff --git a/include/mapnik/json/geojson_grammar_x3_def.hpp b/include/mapnik/json/geojson_grammar_x3_def.hpp index 2fab6b902..db3257a48 100644 --- a/include/mapnik/json/geojson_grammar_x3_def.hpp +++ b/include/mapnik/json/geojson_grammar_x3_def.hpp @@ -35,30 +35,30 @@ namespace grammar { namespace x3 = boost::spirit::x3; -auto make_null = [](auto const& ctx) { +const auto make_null = [](auto const& ctx) { _val(ctx) = mapnik::value_null{}; }; -auto make_true = [](auto const& ctx) { +const auto make_true = [](auto const& ctx) { _val(ctx) = true; }; -auto make_false = [](auto const& ctx) { +const auto make_false = [](auto const& ctx) { _val(ctx) = false; }; -auto assign = [](auto const& ctx) { +const auto assign = [](auto const& ctx) { _val(ctx) = std::move(_attr(ctx)); }; -auto assign_key = [](auto const& ctx) { +const auto assign_key = [](auto const& ctx) { std::string const& name = _attr(ctx); keys_map& keys = x3::get(ctx); auto result = keys.insert(keys_map::value_type(name, keys.size() + 1)); std::get<0>(_val(ctx)) = result.first->right; }; -auto assign_value = [](auto const& ctx) { +const auto assign_value = [](auto const& ctx) { std::get<1>(_val(ctx)) = std::move(_attr(ctx)); }; @@ -81,7 +81,7 @@ struct geometry_type_ : x3::symbols ("\"GeometryCollection\"", mapnik::geometry::geometry_types::GeometryCollection) // ; } -} geometry_type_sym; +} const geometry_type_sym; // rules x3::rule const object("JSON Object"); @@ -95,13 +95,9 @@ auto const geojson_double = x3::real_parser(); // import unicode string rule -namespace { -auto const& geojson_string = unicode_string; -} +const auto geojson_string = unicode_string; // import positions rule -namespace { -auto const& positions_rule = positions; -} +const auto positions_rule = positions; // GeoJSON types // clang-format off diff --git a/include/mapnik/json/positions_grammar_x3_def.hpp b/include/mapnik/json/positions_grammar_x3_def.hpp index da9ab75d5..e4399d759 100644 --- a/include/mapnik/json/positions_grammar_x3_def.hpp +++ b/include/mapnik/json/positions_grammar_x3_def.hpp @@ -33,14 +33,11 @@ namespace grammar { namespace x3 = boost::spirit::x3; using x3::double_; using x3::lit; -using x3::no_case; using x3::omit; -namespace { -auto assign_helper = [](auto const& ctx) { +const auto assign_helper = [](auto const& ctx) { _val(ctx) = std::move(_attr(ctx)); }; -} // namespace // rules x3::rule const point("Position"); diff --git a/include/mapnik/json/topojson_grammar_x3_def.hpp b/include/mapnik/json/topojson_grammar_x3_def.hpp index 6a4d02026..40b083915 100644 --- a/include/mapnik/json/topojson_grammar_x3_def.hpp +++ b/include/mapnik/json/topojson_grammar_x3_def.hpp @@ -169,7 +169,7 @@ struct create_multi_polygon } }; -auto create_geometry = [](auto const& ctx) { +const auto create_geometry = [](auto const& ctx) { auto const geom_type = std::get<0>(_attr(ctx)); auto const& coord = std::get<1>(_attr(ctx)); auto const& arcs = std::get<2>(_attr(ctx)); @@ -199,27 +199,27 @@ auto create_geometry = [](auto const& ctx) { _val(ctx) = std::move(geom); }; -auto assign_bbox = [](auto const& ctx) { +const auto assign_bbox = [](auto const& ctx) { _val(ctx).bbox = std::move(_attr(ctx)); }; -auto assign_transform = [](auto const& ctx) { +const auto assign_transform = [](auto const& ctx) { _val(ctx).tr = std::move(_attr(ctx)); }; -auto assign_arcs = [](auto const& ctx) { +const auto assign_arcs = [](auto const& ctx) { _val(ctx).arcs = std::move(_attr(ctx)); }; -auto assign_objects = [](auto const& ctx) { +const auto assign_objects = [](auto const& ctx) { _val(ctx).geometries = std::move(_attr(ctx)); }; -auto push_geometry = [](auto const& ctx) { +const auto push_geometry = [](auto const& ctx) { _val(ctx).push_back(std::move(_attr(ctx))); }; -auto push_collection = [](auto const& ctx) { +const auto push_collection = [](auto const& ctx) { auto& dest = _val(ctx); auto& src = _attr(ctx); if (dest.empty()) @@ -231,27 +231,27 @@ auto push_collection = [](auto const& ctx) { } }; -auto assign_geometry_type = [](auto const& ctx) { +const auto assign_geometry_type = [](auto const& ctx) { std::get<0>(_val(ctx)) = _attr(ctx); }; -auto assign_coordinates = [](auto const& ctx) { +const auto assign_coordinates = [](auto const& ctx) { std::get<1>(_val(ctx)) = std::move(_attr(ctx)); }; -auto assign_rings = [](auto const& ctx) { +const auto assign_rings = [](auto const& ctx) { std::get<2>(_val(ctx)) = std::move(_attr(ctx)); }; -auto assign_properties = [](auto const& ctx) { +const auto assign_properties = [](auto const& ctx) { std::get<3>(_val(ctx)) = std::move(_attr(ctx)); }; -auto assign_prop_name = [](auto const& ctx) { +const auto assign_prop_name = [](auto const& ctx) { std::get<0>(_val(ctx)) = std::move(_attr(ctx)); }; -auto assign_prop_value = [](auto const& ctx) { +const auto assign_prop_value = [](auto const& ctx) { std::get<1>(_val(ctx)) = std::move(_attr(ctx)); }; @@ -262,12 +262,10 @@ using x3::int_; using x3::lit; using x3::omit; -namespace { // import unicode string rule -auto const& json_string = json::grammar::unicode_string; +const auto json_string = json::grammar::unicode_string; // json value -auto const& json_value = json::grammar::value; -} // namespace +const auto json_value = json::grammar::value; using coordinates_type = util::variant>; using arcs_type = util::variant, @@ -287,7 +285,7 @@ struct topojson_geometry_type_ : x3::symbols ("\"MultiPolygon\"", 6) // ; } -} topojson_geometry_type; +} const topojson_geometry_type; // rules x3::rule const transform = "Transform"; diff --git a/include/mapnik/json/unicode_string_grammar_x3_def.hpp b/include/mapnik/json/unicode_string_grammar_x3_def.hpp index 6a8c77eb1..cbd854532 100644 --- a/include/mapnik/json/unicode_string_grammar_x3_def.hpp +++ b/include/mapnik/json/unicode_string_grammar_x3_def.hpp @@ -35,7 +35,7 @@ namespace x3 = boost::spirit::x3; using uchar = std::uint32_t; // a unicode code point -auto append = [](auto const& ctx) { +const auto append = [](auto const& ctx) { _val(ctx) += _attr(ctx); }; @@ -50,15 +50,15 @@ static inline void push_utf8_impl(std::string& str, uchar code_point) } } // namespace detail -auto push_char = [](auto const& ctx) { +const auto push_char = [](auto const& ctx) { _val(ctx).push_back(_attr(ctx)); }; -auto push_utf8 = [](auto const& ctx) { +const auto push_utf8 = [](auto const& ctx) { detail::push_utf8_impl(_val(ctx), _attr(ctx)); }; -auto push_utf16 = [](auto const& ctx) { +const auto push_utf16 = [](auto const& ctx) { using iterator_type = std::vector::const_iterator; auto const& utf16 = _attr(ctx); try @@ -76,7 +76,7 @@ auto push_utf16 = [](auto const& ctx) { } }; -auto push_esc = [](auto const& ctx) { +const auto push_esc = [](auto const& ctx) { std::string& utf8 = _val(ctx); char c = _attr(ctx); switch (c) diff --git a/include/mapnik/mapnik.hpp b/include/mapnik/mapnik.hpp new file mode 100644 index 000000000..e1545a48d --- /dev/null +++ b/include/mapnik/mapnik.hpp @@ -0,0 +1,9 @@ +#ifndef MAPNIK_MAPNIK_HPP +#define MAPNIK_MAPNIK_HPP +#include "config.hpp" + +namespace mapnik { +MAPNIK_DECL void setup(); +} + +#endif diff --git a/include/mapnik/memory_datasource.hpp b/include/mapnik/memory_datasource.hpp index 833b3c09e..9591c8181 100644 --- a/include/mapnik/memory_datasource.hpp +++ b/include/mapnik/memory_datasource.hpp @@ -26,12 +26,14 @@ // mapnik #include #include +#include // stl #include namespace mapnik { +DATASOURCE_PLUGIN_DEF(memory_datasource_plugin, memory) class MAPNIK_DECL memory_datasource : public datasource { friend class memory_featureset; diff --git a/include/mapnik/path_expression_grammar_x3_def.hpp b/include/mapnik/path_expression_grammar_x3_def.hpp index f137868f3..3600fa986 100644 --- a/include/mapnik/path_expression_grammar_x3_def.hpp +++ b/include/mapnik/path_expression_grammar_x3_def.hpp @@ -32,10 +32,10 @@ namespace grammar { namespace x3 = boost::spirit::x3; using x3::lexeme; using x3::standard_wide::char_; -auto create_string = [](auto& ctx) { +const auto create_string = [](auto& ctx) { _val(ctx).push_back(_attr(ctx)); }; -auto create_attribute = [](auto& ctx) { +const auto create_attribute = [](auto& ctx) { _val(ctx).push_back(mapnik::attribute(_attr(ctx))); }; // rules diff --git a/include/mapnik/plugin.hpp b/include/mapnik/plugin.hpp index 46c06c421..d1907411d 100644 --- a/include/mapnik/plugin.hpp +++ b/include/mapnik/plugin.hpp @@ -25,9 +25,9 @@ // mapnik #include - // stl #include +#include namespace mapnik { @@ -45,13 +45,10 @@ class PluginInfo : util::noncopyable bool valid() const; std::string get_error() const; void* get_symbol(std::string const& sym_name) const; - static void init(); - static void exit(); private: std::string filename_; - std::string name_; - mapnik_lib_t* module_; + std::unique_ptr module_; }; } // namespace mapnik diff --git a/include/mapnik/webp_io.hpp b/include/mapnik/webp_io.hpp index 02a468ba0..404bcbaf3 100644 --- a/include/mapnik/webp_io.hpp +++ b/include/mapnik/webp_io.hpp @@ -24,6 +24,7 @@ #define MAPNIK_WEBP_IO_HPP // mapnik +#include #include #include @@ -50,45 +51,7 @@ int webp_stream_write(const uint8_t* data, size_t data_size, const WebPPicture* return true; } -std::string webp_encoding_error(WebPEncodingError error) -{ - std::string os; - switch (error) - { - case VP8_ENC_ERROR_OUT_OF_MEMORY: - os = "memory error allocating objects"; - break; - case VP8_ENC_ERROR_BITSTREAM_OUT_OF_MEMORY: - os = "memory error while flushing bits"; - break; - case VP8_ENC_ERROR_NULL_PARAMETER: - os = "a pointer parameter is NULL"; - break; - case VP8_ENC_ERROR_INVALID_CONFIGURATION: - os = "configuration is invalid"; - break; - case VP8_ENC_ERROR_BAD_DIMENSION: - os = "picture has invalid width/height"; - break; - case VP8_ENC_ERROR_PARTITION0_OVERFLOW: - os = "partition is bigger than 512k"; - break; - case VP8_ENC_ERROR_PARTITION_OVERFLOW: - os = "partition is bigger than 16M"; - break; - case VP8_ENC_ERROR_BAD_WRITE: - os = "error while flushing bytes"; - break; - case VP8_ENC_ERROR_FILE_TOO_BIG: - os = "file is bigger than 4G"; - break; - default: - mapnik::util::to_string(os, error); - os = "unknown error (" + os + ")"; - break; - } - return os; -} +std::string MAPNIK_DECL webp_encoding_error(WebPEncodingError error); template inline int import_image(T2 const& im_in, WebPPicture& pic, bool alpha) diff --git a/include/mapnik/wkt/wkt_grammar_x3_def.hpp b/include/mapnik/wkt/wkt_grammar_x3_def.hpp index 40302fb7f..3af4dcbd4 100644 --- a/include/mapnik/wkt/wkt_grammar_x3_def.hpp +++ b/include/mapnik/wkt/wkt_grammar_x3_def.hpp @@ -47,17 +47,16 @@ namespace mapnik { namespace grammar { namespace x3 = boost::spirit::x3; -namespace ascii = boost::spirit::x3::ascii; using x3::double_; using x3::lit; using x3::no_case; // functors -auto make_empty = [](auto const& ctx) { +const auto make_empty = [](auto const& ctx) { _val(ctx) = geometry::geometry_empty(); }; -auto add_ring = [](auto const& ctx) { +const auto add_ring = [](auto const& ctx) { auto& ring = reinterpret_cast&>(_attr(ctx)); _val(ctx).push_back(std::move(ring)); }; diff --git a/plugins/input/CMakeLists.txt b/plugins/input/CMakeLists.txt index 4bf03e016..8a1bcd5fd 100644 --- a/plugins/input/CMakeLists.txt +++ b/plugins/input/CMakeLists.txt @@ -1,5 +1,28 @@ set(_plugin_prefix "") set(_plugin_suffix ".input") +set(_plugin_library_output "${MAPNIK_OUTPUT_DIR}/plugins/input") +set(_plugin_fpic ON) +set(_plugin_linkage MODULE) +set(_plugin_visibility PRIVATE) +if(NOT BUILD_SHARED_PLUGINS) + set(_plugin_linkage INTERFACE) + set(_plugin_visibility INTERFACE) +endif() + +macro(add_plugin_target plugin_target output_name) + add_library(${plugin_target} ${_plugin_linkage}) + set_target_properties(${plugin_target} PROPERTIES + OUTPUT_NAME "${output_name}" + POSITION_INDEPENDENT_CODE ${_plugin_fpic} + PREFIX "${_plugin_prefix}" + SUFFIX "${_plugin_suffix}" + LIBRARY_OUTPUT_DIRECTORY "${_plugin_library_output}" + ) + mapnik_install_plugin(${plugin_target}) +endmacro() + + +add_subdirectory(base) # add a list with all build plugins so the copy dependencies command can wait for all build events set(m_build_plugins "") @@ -51,7 +74,7 @@ endif() # # Copy all plugin dlls, so that these are in the main output dir, since cmake copies those into ${MAPNIK_OUTPUT_DIR}/plugins/input, too. # -if(WIN32) +if(BUILD_SHARED_PLUGINS AND WIN32) list(LENGTH m_build_plugins m_number_plugins) if(m_number_plugins GREATER 0) string(CONFIGURE diff --git a/plugins/input/base/CMakeLists.txt b/plugins/input/base/CMakeLists.txt new file mode 100644 index 000000000..bf76706b8 --- /dev/null +++ b/plugins/input/base/CMakeLists.txt @@ -0,0 +1,12 @@ +add_library(datasource-base INTERFACE) +add_library(mapnik::datasource-base ALIAS datasource-base) + + +target_include_directories(datasource-base INTERFACE + $ + $ +) +target_link_libraries(datasource-base INTERFACE mapnik::core) + +mapnik_install(datasource-base) +install(DIRECTORY include/ DESTINATION "${MAPNIK_INCLUDE_DIR}") diff --git a/plugins/input/base/include/mapnik/datasource_plugin.hpp b/plugins/input/base/include/mapnik/datasource_plugin.hpp new file mode 100644 index 000000000..ee6f7a854 --- /dev/null +++ b/plugins/input/base/include/mapnik/datasource_plugin.hpp @@ -0,0 +1,54 @@ +#ifndef DATASOURCE_PLUGIN_HPP +#define DATASOURCE_PLUGIN_HPP +#include +#include +#include + +namespace mapnik { +class MAPNIK_DECL datasource_plugin +{ + public: + datasource_plugin() = default; + virtual ~datasource_plugin() = default; + virtual void after_load() const = 0; + virtual void before_unload() const = 0; + virtual const char* name() const = 0; + virtual datasource_ptr create(parameters const& params) const = 0; +}; +} // namespace mapnik + +#define DATASOURCE_PLUGIN_DEF(classname, pluginname) \ + class classname : public mapnik::datasource_plugin \ + { \ + public: \ + static constexpr const char* kName = #pluginname; \ + void after_load() const override; \ + void before_unload() const override; \ + const char* name() const override; \ + mapnik::datasource_ptr create(mapnik::parameters const& params) const override; \ + }; + +#define DATASOURCE_PLUGIN_IMPL(classname, pluginclassname) \ + const char* classname::name() const \ + { \ + return kName; \ + } \ + mapnik::datasource_ptr classname::create(mapnik::parameters const& params) const \ + { \ + return std::make_shared(params); \ + } + +#define DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(classname) \ + void classname::after_load() const {} +#define DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(classname) \ + void classname::before_unload() const {} + +#ifndef MAPNIK_STATIC_PLUGINS +#define DATASOURCE_PLUGIN_EXPORT(classname) \ + extern "C" MAPNIK_EXP classname plugin; \ + classname plugin; +#else +#define DATASOURCE_PLUGIN_EXPORT(classname) // export classname +#endif + +#endif diff --git a/plugins/input/csv/CMakeLists.txt b/plugins/input/csv/CMakeLists.txt index 5fc60254c..bb2739dd1 100644 --- a/plugins/input/csv/CMakeLists.txt +++ b/plugins/input/csv/CMakeLists.txt @@ -1,21 +1,15 @@ -add_library(input-csv MODULE +add_plugin_target(input-csv "csv") + +target_sources(input-csv ${_plugin_visibility} csv_datasource.cpp csv_featureset.cpp csv_index_featureset.cpp csv_inline_featureset.cpp csv_utils.cpp ) -target_link_libraries(input-csv PRIVATE +target_link_libraries(input-csv ${_plugin_visibility} mapnik::mapnik mapnik::wkt mapnik::json + mapnik::datasource-base ) -set_target_properties(input-csv PROPERTIES - OUTPUT_NAME "csv" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) -mapnik_install_plugin(input-csv) diff --git a/plugins/input/csv/csv_datasource.cpp b/plugins/input/csv/csv_datasource.cpp index 1a0d36461..7068882db 100644 --- a/plugins/input/csv/csv_datasource.cpp +++ b/plugins/input/csv/csv_datasource.cpp @@ -65,7 +65,10 @@ MAPNIK_DISABLE_WARNING_POP using mapnik::datasource; using mapnik::parameters; -DATASOURCE_PLUGIN(csv_datasource) +DATASOURCE_PLUGIN_IMPL(csv_datasource_plugin, csv_datasource); +DATASOURCE_PLUGIN_EXPORT(csv_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(csv_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(csv_datasource_plugin); csv_datasource::csv_datasource(parameters const& params) : datasource(params) diff --git a/plugins/input/csv/csv_datasource.hpp b/plugins/input/csv/csv_datasource.hpp index 8dcd5e6e5..9a710f25f 100644 --- a/plugins/input/csv/csv_datasource.hpp +++ b/plugins/input/csv/csv_datasource.hpp @@ -48,6 +48,8 @@ MAPNIK_DISABLE_WARNING_POP #include #include +#include + template struct csv_linear : boost::geometry::index::linear {}; @@ -79,6 +81,8 @@ struct options_type> } // namespace geometry } // namespace boost +DATASOURCE_PLUGIN_DEF(csv_datasource_plugin, csv); + class csv_datasource : public mapnik::datasource, private csv_utils::csv_file_parser { diff --git a/plugins/input/gdal/CMakeLists.txt b/plugins/input/gdal/CMakeLists.txt index cf6f272c5..440c545b3 100644 --- a/plugins/input/gdal/CMakeLists.txt +++ b/plugins/input/gdal/CMakeLists.txt @@ -1,20 +1,13 @@ find_package(GDAL REQUIRED) -add_library(input-gdal MODULE +add_plugin_target(input-gdal "gdal") +target_sources(input-gdal ${_plugin_visibility} gdal_datasource.cpp gdal_featureset.cpp ) -target_include_directories(input-gdal PRIVATE ${GDAL_INCLUDE_DIRS}) -target_link_libraries(input-gdal PRIVATE +target_include_directories(input-gdal ${_plugin_visibility} ${GDAL_INCLUDE_DIRS}) +target_link_libraries(input-gdal ${_plugin_visibility} mapnik::mapnik + mapnik::datasource-base ${GDAL_LIBRARIES} ) -set_target_properties(input-gdal PROPERTIES - OUTPUT_NAME "gdal" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) -mapnik_install_plugin(input-gdal) diff --git a/plugins/input/gdal/gdal_datasource.cpp b/plugins/input/gdal/gdal_datasource.cpp index 54b5c20ed..f86e17361 100644 --- a/plugins/input/gdal/gdal_datasource.cpp +++ b/plugins/input/gdal/gdal_datasource.cpp @@ -37,8 +37,6 @@ using mapnik::datasource; using mapnik::parameters; -DATASOURCE_PLUGIN(gdal_datasource) - using mapnik::box2d; using mapnik::coord2d; using mapnik::datasource_exception; @@ -46,12 +44,16 @@ using mapnik::featureset_ptr; using mapnik::layer_descriptor; using mapnik::query; -static std::once_flag once_flag; - -extern "C" MAPNIK_EXP void on_plugin_load() +DATASOURCE_PLUGIN_IMPL(gdal_datasource_plugin, gdal_datasource); +DATASOURCE_PLUGIN_EXPORT(gdal_datasource_plugin); +void gdal_datasource_plugin::after_load() const { - // initialize gdal formats - std::call_once(once_flag, []() { GDALAllRegister(); }); + GDALAllRegister(); +} + +void gdal_datasource_plugin::before_unload() const +{ + GDALDestroyDriverManager(); } gdal_datasource::gdal_datasource(parameters const& params) diff --git a/plugins/input/gdal/gdal_datasource.hpp b/plugins/input/gdal/gdal_datasource.hpp index 5ff2a72b0..2dad23023 100644 --- a/plugins/input/gdal/gdal_datasource.hpp +++ b/plugins/input/gdal/gdal_datasource.hpp @@ -31,7 +31,7 @@ #include #include #include - +#include // boost #include @@ -42,6 +42,8 @@ // gdal #include +DATASOURCE_PLUGIN_DEF(gdal_datasource_plugin, gdal); + class gdal_datasource : public mapnik::datasource { public: diff --git a/plugins/input/geobuf/CMakeLists.txt b/plugins/input/geobuf/CMakeLists.txt index feadf82d9..8bcf7b132 100644 --- a/plugins/input/geobuf/CMakeLists.txt +++ b/plugins/input/geobuf/CMakeLists.txt @@ -1,14 +1,10 @@ -add_library(input-geobuf MODULE +add_plugin_target(input-geobuf "geobuf") + +target_sources(input-geobuf ${_plugin_visibility} geobuf_datasource.cpp geobuf_featureset.cpp ) -target_link_libraries(input-geobuf PRIVATE mapnik::mapnik) -set_target_properties(input-geobuf PROPERTIES - OUTPUT_NAME "geobuf" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" +target_link_libraries(input-geobuf ${_plugin_visibility} + mapnik::mapnik + mapnik::datasource-base ) -mapnik_install_plugin(input-geobuf) diff --git a/plugins/input/geobuf/geobuf_datasource.cpp b/plugins/input/geobuf/geobuf_datasource.cpp index 974225153..48eafa299 100644 --- a/plugins/input/geobuf/geobuf_datasource.cpp +++ b/plugins/input/geobuf/geobuf_datasource.cpp @@ -48,7 +48,10 @@ using mapnik::datasource; using mapnik::parameters; -DATASOURCE_PLUGIN(geobuf_datasource) +DATASOURCE_PLUGIN_IMPL(geobuf_datasource_plugin, geobuf_datasource); +DATASOURCE_PLUGIN_EXPORT(geobuf_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(geobuf_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(geobuf_datasource_plugin); struct attr_value_converter { diff --git a/plugins/input/geobuf/geobuf_datasource.hpp b/plugins/input/geobuf/geobuf_datasource.hpp index f6da2ebb4..48088ec9b 100644 --- a/plugins/input/geobuf/geobuf_datasource.hpp +++ b/plugins/input/geobuf/geobuf_datasource.hpp @@ -32,6 +32,7 @@ #include #include #include +#include // boost #include #include @@ -84,6 +85,7 @@ struct options_type> } // namespace geometry } // namespace boost +DATASOURCE_PLUGIN_DEF(geobuf_datasource_plugin, geobuf); class geobuf_datasource : public mapnik::datasource { public: diff --git a/plugins/input/geojson/CMakeLists.txt b/plugins/input/geojson/CMakeLists.txt index b973f3f68..d6919a405 100644 --- a/plugins/input/geojson/CMakeLists.txt +++ b/plugins/input/geojson/CMakeLists.txt @@ -1,19 +1,12 @@ -add_library(input-geojson MODULE +add_plugin_target(input-geojson "geojson") +target_sources(input-geojson ${_plugin_visibility} geojson_datasource.cpp geojson_featureset.cpp geojson_index_featureset.cpp geojson_memory_index_featureset.cpp ) -target_link_libraries(input-geojson PRIVATE +target_link_libraries(input-geojson ${_plugin_visibility} mapnik::mapnik mapnik::json + mapnik::datasource-base ) -set_target_properties(input-geojson PROPERTIES - OUTPUT_NAME "geojson" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) -mapnik_install_plugin(input-geojson) diff --git a/plugins/input/geojson/geojson_datasource.cpp b/plugins/input/geojson/geojson_datasource.cpp index b811e9d8d..f4ecf82ed 100644 --- a/plugins/input/geojson/geojson_datasource.cpp +++ b/plugins/input/geojson/geojson_datasource.cpp @@ -67,7 +67,10 @@ MAPNIK_DISABLE_WARNING_POP using mapnik::datasource; using mapnik::parameters; -DATASOURCE_PLUGIN(geojson_datasource) +DATASOURCE_PLUGIN_IMPL(geojson_datasource_plugin, geojson_datasource); +DATASOURCE_PLUGIN_EXPORT(geojson_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(geojson_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(geojson_datasource_plugin); struct attr_value_converter { diff --git a/plugins/input/geojson/geojson_datasource.hpp b/plugins/input/geojson/geojson_datasource.hpp index 216c656a4..57584717e 100644 --- a/plugins/input/geojson/geojson_datasource.hpp +++ b/plugins/input/geojson/geojson_datasource.hpp @@ -32,6 +32,7 @@ #include #include #include +#include #include MAPNIK_DISABLE_WARNING_PUSH @@ -82,6 +83,8 @@ struct options_type> } // namespace geometry } // namespace boost +DATASOURCE_PLUGIN_DEF(geojson_datasource_plugin, geojson); + class geojson_datasource : public mapnik::datasource { public: diff --git a/plugins/input/ogr/CMakeLists.txt b/plugins/input/ogr/CMakeLists.txt index d030eb618..0034d4684 100644 --- a/plugins/input/ogr/CMakeLists.txt +++ b/plugins/input/ogr/CMakeLists.txt @@ -1,22 +1,15 @@ find_package(GDAL REQUIRED) -add_library(input-ogr MODULE +add_plugin_target(input-ogr "ogr") +target_sources(input-ogr ${_plugin_visibility} ogr_converter.cpp ogr_datasource.cpp ogr_featureset.cpp ogr_index_featureset.cpp ) -target_include_directories(input-ogr PRIVATE ${GDAL_INCLUDE_DIRS}) -target_link_libraries(input-ogr PRIVATE +target_include_directories(input-ogr ${_plugin_visibility} ${GDAL_INCLUDE_DIRS}) +target_link_libraries(input-ogr ${_plugin_visibility} mapnik::mapnik + mapnik::datasource-base ${GDAL_LIBRARIES} ) -set_target_properties(input-ogr PROPERTIES - OUTPUT_NAME "ogr" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) -mapnik_install_plugin(input-ogr) diff --git a/plugins/input/ogr/ogr_datasource.cpp b/plugins/input/ogr/ogr_datasource.cpp index 6bc00a23e..2e4cad22e 100644 --- a/plugins/input/ogr/ogr_datasource.cpp +++ b/plugins/input/ogr/ogr_datasource.cpp @@ -33,6 +33,7 @@ #include #include #include +#include #include MAPNIK_DISABLE_WARNING_PUSH @@ -49,8 +50,6 @@ MAPNIK_DISABLE_WARNING_POP using mapnik::datasource; using mapnik::parameters; -DATASOURCE_PLUGIN(ogr_datasource) - using mapnik::attribute_descriptor; using mapnik::box2d; using mapnik::coord2d; @@ -63,11 +62,21 @@ using mapnik::query; static std::once_flag once_flag; -extern "C" MAPNIK_EXP void on_plugin_load() +DATASOURCE_PLUGIN_IMPL(ogr_datasource_plugin, ogr_datasource); +DATASOURCE_PLUGIN_EXPORT(ogr_datasource_plugin); + +void ogr_datasource_plugin::after_load() const { // initialize ogr formats // NOTE: in GDAL >= 2.0 this is the same as GDALAllRegister() - std::call_once(once_flag, []() { OGRRegisterAll(); }); + OGRRegisterAll(); +} + +void ogr_datasource_plugin::before_unload() const +{ + // initialize ogr formats + // NOTE: in GDAL >= 2.0 this is the same as GDALDestroyDriverManager() + OGRCleanupAll(); } ogr_datasource::ogr_datasource(parameters const& params) diff --git a/plugins/input/ogr/ogr_datasource.hpp b/plugins/input/ogr/ogr_datasource.hpp index 26f74d221..bde5829b0 100644 --- a/plugins/input/ogr/ogr_datasource.hpp +++ b/plugins/input/ogr/ogr_datasource.hpp @@ -31,6 +31,7 @@ #include #include #include +#include // boost #include @@ -47,6 +48,8 @@ MAPNIK_DISABLE_WARNING_PUSH MAPNIK_DISABLE_WARNING_POP #include "ogr_layer_ptr.hpp" +DATASOURCE_PLUGIN_DEF(ogr_datasource_plugin, ogr); + class ogr_datasource : public mapnik::datasource { public: diff --git a/plugins/input/pgraster/CMakeLists.txt b/plugins/input/pgraster/CMakeLists.txt index 387f168ca..21a58e733 100644 --- a/plugins/input/pgraster/CMakeLists.txt +++ b/plugins/input/pgraster/CMakeLists.txt @@ -1,20 +1,13 @@ -find_package(PostgreSQL REQUIRED) +mapnik_find_package(PostgreSQL REQUIRED) -add_library(input-pgraster MODULE +add_plugin_target(input-pgraster "pgraster") +target_sources(input-pgraster ${_plugin_visibility} pgraster_datasource.cpp pgraster_featureset.cpp pgraster_wkb_reader.cpp ) -target_link_libraries(input-pgraster PRIVATE +target_link_libraries(input-pgraster ${_plugin_visibility} mapnik::mapnik + mapnik::datasource-base PostgreSQL::PostgreSQL ) -set_target_properties(input-pgraster PROPERTIES - OUTPUT_NAME "pgraster" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) -mapnik_install_plugin(input-pgraster) diff --git a/plugins/input/pgraster/pgraster_datasource.cpp b/plugins/input/pgraster/pgraster_datasource.cpp index 4428cfc06..fdba3c335 100644 --- a/plugins/input/pgraster/pgraster_datasource.cpp +++ b/plugins/input/pgraster/pgraster_datasource.cpp @@ -51,7 +51,10 @@ MAPNIK_DISABLE_WARNING_POP #include #include -DATASOURCE_PLUGIN(pgraster_datasource) +DATASOURCE_PLUGIN_IMPL(pgraster_datasource_plugin, pgraster_datasource); +DATASOURCE_PLUGIN_EXPORT(pgraster_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(pgraster_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(pgraster_datasource_plugin); const std::string pgraster_datasource::RASTER_COLUMNS = "raster_columns"; const std::string pgraster_datasource::RASTER_OVERVIEWS = "raster_overviews"; diff --git a/plugins/input/pgraster/pgraster_datasource.hpp b/plugins/input/pgraster/pgraster_datasource.hpp index 3d3a0b4c8..136bd50e0 100644 --- a/plugins/input/pgraster/pgraster_datasource.hpp +++ b/plugins/input/pgraster/pgraster_datasource.hpp @@ -37,6 +37,7 @@ #include #include #include +#include // boost #include @@ -73,6 +74,7 @@ struct pgraster_overview float scale; // max absolute scale between x and y }; +DATASOURCE_PLUGIN_DEF(pgraster_datasource_plugin, pgraster); class pgraster_datasource : public datasource { public: diff --git a/plugins/input/postgis/CMakeLists.txt b/plugins/input/postgis/CMakeLists.txt index 3fde486aa..e26e9b8c9 100644 --- a/plugins/input/postgis/CMakeLists.txt +++ b/plugins/input/postgis/CMakeLists.txt @@ -1,19 +1,12 @@ -find_package(PostgreSQL REQUIRED) +mapnik_find_package(PostgreSQL REQUIRED) -add_library(input-postgis MODULE +add_plugin_target(input-postgis "postgis") +target_sources(input-postgis ${_plugin_visibility} postgis_datasource.cpp postgis_featureset.cpp ) -target_link_libraries(input-postgis PRIVATE +target_link_libraries(input-postgis ${_plugin_visibility} mapnik::mapnik + mapnik::datasource-base PostgreSQL::PostgreSQL ) -set_target_properties(input-postgis PROPERTIES - OUTPUT_NAME "postgis" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) -mapnik_install_plugin(input-postgis) diff --git a/plugins/input/postgis/postgis_datasource.cpp b/plugins/input/postgis/postgis_datasource.cpp index 9b236e149..88a03a680 100644 --- a/plugins/input/postgis/postgis_datasource.cpp +++ b/plugins/input/postgis/postgis_datasource.cpp @@ -48,7 +48,10 @@ MAPNIK_DISABLE_WARNING_POP #include #include -DATASOURCE_PLUGIN(postgis_datasource) +DATASOURCE_PLUGIN_IMPL(postgis_datasource_plugin, postgis_datasource); +DATASOURCE_PLUGIN_EXPORT(postgis_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(postgis_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(postgis_datasource_plugin); const std::string postgis_datasource::GEOMETRY_COLUMNS = "geometry_columns"; const std::string postgis_datasource::SPATIAL_REF_SYS = "spatial_ref_system"; diff --git a/plugins/input/postgis/postgis_datasource.hpp b/plugins/input/postgis/postgis_datasource.hpp index ce02454ee..377da93b3 100644 --- a/plugins/input/postgis/postgis_datasource.hpp +++ b/plugins/input/postgis/postgis_datasource.hpp @@ -34,6 +34,7 @@ #include #include #include +#include // boost #include @@ -62,6 +63,7 @@ using mapnik::transcoder; using CnxPool_ptr = std::shared_ptr; +DATASOURCE_PLUGIN_DEF(postgis_datasource_plugin, postgis); class postgis_datasource : public datasource { public: diff --git a/plugins/input/raster/CMakeLists.txt b/plugins/input/raster/CMakeLists.txt index 265d600b0..ca3826a70 100644 --- a/plugins/input/raster/CMakeLists.txt +++ b/plugins/input/raster/CMakeLists.txt @@ -1,15 +1,10 @@ -add_library(input-raster MODULE +add_plugin_target(input-raster "raster") +target_sources(input-raster ${_plugin_visibility} raster_datasource.cpp raster_featureset.cpp raster_info.cpp ) -target_link_libraries(input-raster PRIVATE mapnik::mapnik) -set_target_properties(input-raster PROPERTIES - OUTPUT_NAME "raster" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" +target_link_libraries(input-raster ${_plugin_visibility} + mapnik::mapnik + mapnik::datasource-base ) -mapnik_install_plugin(input-raster) diff --git a/plugins/input/raster/raster_datasource.cpp b/plugins/input/raster/raster_datasource.cpp index 3c136cb88..5e04ebf4f 100644 --- a/plugins/input/raster/raster_datasource.cpp +++ b/plugins/input/raster/raster_datasource.cpp @@ -43,7 +43,10 @@ using mapnik::layer_descriptor; using mapnik::parameters; using mapnik::query; -DATASOURCE_PLUGIN(raster_datasource) +DATASOURCE_PLUGIN_IMPL(raster_datasource_plugin, raster_datasource); +DATASOURCE_PLUGIN_EXPORT(raster_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(raster_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(raster_datasource_plugin); raster_datasource::raster_datasource(parameters const& params) : datasource(params) diff --git a/plugins/input/raster/raster_datasource.hpp b/plugins/input/raster/raster_datasource.hpp index 6f834a4b4..f1b78dfe9 100644 --- a/plugins/input/raster/raster_datasource.hpp +++ b/plugins/input/raster/raster_datasource.hpp @@ -31,6 +31,7 @@ #include #include #include +#include // boost #include @@ -40,6 +41,8 @@ #include #include +DATASOURCE_PLUGIN_DEF(raster_datasource_plugin, raster); + class raster_datasource : public mapnik::datasource { public: diff --git a/plugins/input/shape/CMakeLists.txt b/plugins/input/shape/CMakeLists.txt index 8e9c0832f..930d85f11 100644 --- a/plugins/input/shape/CMakeLists.txt +++ b/plugins/input/shape/CMakeLists.txt @@ -1,4 +1,5 @@ -add_library(input-shape MODULE +add_plugin_target(input-shape "shape") +target_sources(input-shape ${_plugin_visibility} dbfile.cpp dbf_test.cpp shape_datasource.cpp @@ -6,13 +7,7 @@ add_library(input-shape MODULE shape_index_featureset.cpp shape_io.cpp shape_utils.cpp ) -target_link_libraries(input-shape PRIVATE mapnik::mapnik) -set_target_properties(input-shape PROPERTIES - OUTPUT_NAME "shape" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" +target_link_libraries(input-shape ${_plugin_visibility} + mapnik::mapnik + mapnik::datasource-base ) -mapnik_install_plugin(input-shape) diff --git a/plugins/input/shape/shape_datasource.cpp b/plugins/input/shape/shape_datasource.cpp index df96d6684..38f3dbf08 100644 --- a/plugins/input/shape/shape_datasource.cpp +++ b/plugins/input/shape/shape_datasource.cpp @@ -47,7 +47,10 @@ MAPNIK_DISABLE_WARNING_POP #include #include -DATASOURCE_PLUGIN(shape_datasource) +DATASOURCE_PLUGIN_IMPL(shape_datasource_plugin, shape_datasource); +DATASOURCE_PLUGIN_EXPORT(shape_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(shape_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(shape_datasource_plugin); using mapnik::attribute_descriptor; using mapnik::Boolean; diff --git a/plugins/input/shape/shape_datasource.hpp b/plugins/input/shape/shape_datasource.hpp index 7329b9345..bc98b9c3b 100644 --- a/plugins/input/shape/shape_datasource.hpp +++ b/plugins/input/shape/shape_datasource.hpp @@ -32,6 +32,7 @@ #include #include #include +#include // boost #include @@ -50,6 +51,8 @@ using mapnik::layer_descriptor; using mapnik::parameters; using mapnik::query; +DATASOURCE_PLUGIN_DEF(shape_datasource_plugin, shape); + class shape_datasource : public datasource { public: diff --git a/plugins/input/sqlite/CMakeLists.txt b/plugins/input/sqlite/CMakeLists.txt index 26e452de3..5d93c3756 100644 --- a/plugins/input/sqlite/CMakeLists.txt +++ b/plugins/input/sqlite/CMakeLists.txt @@ -1,20 +1,12 @@ find_package(SQLite3 REQUIRED) -add_library(input-sqlite MODULE +add_plugin_target(input-sqlite "sqlite") +target_sources(input-sqlite ${_plugin_visibility} sqlite_datasource.cpp sqlite_featureset.cpp ) -add_library(mapnik::plugin::input::sqlite ALIAS input-sqlite) -target_link_libraries(input-sqlite PRIVATE +target_link_libraries(input-sqlite ${_plugin_visibility} mapnik::mapnik + mapnik::datasource-base SQLite::SQLite3 ) -set_target_properties(input-sqlite PROPERTIES - OUTPUT_NAME "sqlite" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) -mapnik_install_plugin(input-sqlite) diff --git a/plugins/input/sqlite/sqlite_datasource.cpp b/plugins/input/sqlite/sqlite_datasource.cpp index c3522742d..71a55a2eb 100644 --- a/plugins/input/sqlite/sqlite_datasource.cpp +++ b/plugins/input/sqlite/sqlite_datasource.cpp @@ -50,7 +50,10 @@ using mapnik::layer_descriptor; using mapnik::parameters; using mapnik::query; -DATASOURCE_PLUGIN(sqlite_datasource) +DATASOURCE_PLUGIN_IMPL(sqlite_datasource_plugin, sqlite_datasource); +DATASOURCE_PLUGIN_EXPORT(sqlite_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(sqlite_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(sqlite_datasource_plugin); sqlite_datasource::sqlite_datasource(parameters const& params) : datasource(params) diff --git a/plugins/input/sqlite/sqlite_datasource.hpp b/plugins/input/sqlite/sqlite_datasource.hpp index 40a989026..49d588f23 100644 --- a/plugins/input/sqlite/sqlite_datasource.hpp +++ b/plugins/input/sqlite/sqlite_datasource.hpp @@ -33,6 +33,7 @@ #include #include #include +#include // boost #include @@ -45,6 +46,8 @@ // sqlite #include "sqlite_connection.hpp" +DATASOURCE_PLUGIN_DEF(sqlite_datasource_plugin, sqlite); + class sqlite_datasource : public mapnik::datasource { public: diff --git a/plugins/input/topojson/CMakeLists.txt b/plugins/input/topojson/CMakeLists.txt index 865adf685..03f885e44 100644 --- a/plugins/input/topojson/CMakeLists.txt +++ b/plugins/input/topojson/CMakeLists.txt @@ -1,18 +1,10 @@ -add_library(input-topojson MODULE +add_plugin_target(input-topojson "topojson") +target_sources(input-topojson ${_plugin_visibility} topojson_datasource.cpp topojson_featureset.cpp ) -add_library(mapnik::plugin::input::topojson ALIAS input-topojson) -target_link_libraries(input-topojson PRIVATE +target_link_libraries(input-topojson ${_plugin_visibility} mapnik::mapnik mapnik::json + mapnik::datasource-base ) -set_target_properties(input-topojson PROPERTIES - OUTPUT_NAME "topojson" - PREFIX "${_plugin_prefix}" - SUFFIX "${_plugin_suffix}" - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/plugins/input" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) -mapnik_install_plugin(input-topojson) diff --git a/plugins/input/topojson/topojson_datasource.cpp b/plugins/input/topojson/topojson_datasource.cpp index b704c4eee..2981ed65d 100644 --- a/plugins/input/topojson/topojson_datasource.cpp +++ b/plugins/input/topojson/topojson_datasource.cpp @@ -42,7 +42,10 @@ using mapnik::datasource; using mapnik::parameters; -DATASOURCE_PLUGIN(topojson_datasource) +DATASOURCE_PLUGIN_IMPL(topojson_datasource_plugin, topojson_datasource); +DATASOURCE_PLUGIN_EXPORT(topojson_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(topojson_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(topojson_datasource_plugin); struct attr_value_converter { diff --git a/plugins/input/topojson/topojson_datasource.hpp b/plugins/input/topojson/topojson_datasource.hpp index f1f082961..88aff1c0f 100644 --- a/plugins/input/topojson/topojson_datasource.hpp +++ b/plugins/input/topojson/topojson_datasource.hpp @@ -33,6 +33,7 @@ #include #include #include +#include #include MAPNIK_DISABLE_WARNING_PUSH @@ -52,6 +53,8 @@ MAPNIK_DISABLE_WARNING_POP #include #include +DATASOURCE_PLUGIN_DEF(topojson_datasource_plugin, topojson); + class topojson_datasource : public mapnik::datasource { public: diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index beb303d4d..65e649228 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -1,27 +1,55 @@ add_subdirectory(json) add_subdirectory(wkt) -add_library(mapnik "") -add_library(mapnik::mapnik ALIAS mapnik) - if(BUILD_SHARED_LIBS) + # private as this should only be visibile when building, to dllexport/dllimport correctly target_compile_definitions(mapnik PRIVATE MAPNIK_EXPORTS) endif() -target_link_libraries(mapnik PUBLIC mapnik::core) -target_link_libraries(mapnik PRIVATE mapnik::agg) +target_link_libraries(mapnik PUBLIC mapnik::core mapnik::datasource-base) +target_link_libraries(mapnik PRIVATE + mapnik::agg + # expr: if(BUILD_SHARED_PLUGINS == OFF && is build) then add link target + $<$>,$>:input-csv> + $<$>,$>:input-gdal> + $<$>,$>:input-geobuf> + $<$>,$>:input-geojson> + $<$>,$>:input-ogr> + $<$>,$>:input-pgraster> + $<$>,$>:input-postgis> + $<$>,$>:input-raster> + $<$>,$>:input-shape> + $<$>,$>:input-sqlite> + $<$>,$>:input-topojson> +) +target_compile_definitions(mapnik PRIVATE + # expr: if(BUILD_SHARED_PLUGINS == OFF && is build) then add build definition + $<$>,$>:MAPNIK_STATIC_PLUGIN_CSV> + $<$>,$>:MAPNIK_STATIC_PLUGIN_GDAL> + $<$>,$>:MAPNIK_STATIC_PLUGIN_GEOBUF> + $<$>,$>:MAPNIK_STATIC_PLUGIN_GEOJSON> + $<$>,$>:MAPNIK_STATIC_PLUGIN_OGR> + $<$>,$>:MAPNIK_STATIC_PLUGIN_PGRASTER> + $<$>,$>:MAPNIK_STATIC_PLUGIN_POSTGIS> + $<$>,$>:MAPNIK_STATIC_PLUGIN_RASTER> + $<$>,$>:MAPNIK_STATIC_PLUGIN_SHAPE> + $<$>,$>:MAPNIK_STATIC_PLUGIN_SQLITE> + $<$>,$>:MAPNIK_STATIC_PLUGIN_TOPOJSON> + +) + +if(NOT BUILD_SHARED_PLUGINS) + target_include_directories(mapnik PRIVATE "${PROJECT_SOURCE_DIR}/plugins") +endif() set_target_properties(mapnik PROPERTIES - DEBUG_POSTFIX "${MAPNIK_DEBUG_POSTFIX}" + POSITION_INDEPENDENT_CODE ON OUTPUT_NAME "mapnik" PREFIX "lib" IMPORT_PREFIX "lib" # for the archive on dll platforms VERSION ${MAPNIK_VERSION} # see https://github.com/mapnik/mapnik/pull/4248#issuecomment-925596509 => ABI compability only with the full version SOVERSION ${MAPNIK_VERSION} - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" ) if(MSVC) @@ -74,6 +102,7 @@ target_sources(mapnik PRIVATE layer.cpp load_map.cpp map.cpp + mapnik.cpp mapped_memory_cache.cpp marker_cache.cpp marker_helpers.cpp @@ -258,7 +287,10 @@ if(USE_TIFF) endif() if(USE_WEBP) - target_sources(mapnik PRIVATE webp_reader.cpp) + target_sources(mapnik PRIVATE + webp_reader.cpp + webp_io.cpp + ) endif() if(USE_GRID_RENDERER) diff --git a/src/build.py b/src/build.py index ac35d44f0..b68d33ef1 100644 --- a/src/build.py +++ b/src/build.py @@ -87,6 +87,7 @@ if '-DHAVE_TIFF' in env['CPPDEFINES']: if '-DHAVE_WEBP' in env['CPPDEFINES']: lib_env['LIBS'].append('webp') enabled_imaging_libraries.append('webp_reader.cpp') + enabled_imaging_libraries.append('webp_io.cpp') if env['XMLPARSER'] == 'libxml2' and env['HAS_LIBXML2']: lib_env['LIBS'].append('xml2') @@ -152,6 +153,7 @@ else: # unix, non-macos source = Split( """ + mapnik.cpp expression_grammar_x3.cpp fs.cpp request.cpp diff --git a/src/create_image_reader.hpp b/src/create_image_reader.hpp new file mode 100644 index 000000000..184b4eb13 --- /dev/null +++ b/src/create_image_reader.hpp @@ -0,0 +1,19 @@ +#ifndef MAPNIK_CREATE_IMAGE_READER_HPP +#define MAPNIK_CREATE_IMAGE_READER_HPP + +namespace mapnik { +#ifdef HAVE_JPEG +void register_jpeg_reader(); +#endif +#ifdef HAVE_PNG +void register_png_reader(); +#endif +#ifdef HAVE_TIFF +void register_tiff_reader(); +#endif +#ifdef HAVE_WEBP +void register_webp_reader(); +#endif +} // namespace mapnik + +#endif diff --git a/src/datasource_cache.cpp b/src/datasource_cache.cpp index 54871f7f3..5539e3fc6 100644 --- a/src/datasource_cache.cpp +++ b/src/datasource_cache.cpp @@ -24,6 +24,7 @@ #include #include #include +#include #include #include #include @@ -53,15 +54,8 @@ bool is_input_plugin(std::string const& filename) return boost::algorithm::ends_with(filename, std::string(".input")); } -datasource_cache::datasource_cache() -{ - PluginInfo::init(); -} - -datasource_cache::~datasource_cache() -{ - PluginInfo::exit(); -} +datasource_cache::datasource_cache() {} +datasource_cache::~datasource_cache() {} datasource_ptr datasource_cache::create(parameters const& params) { @@ -71,11 +65,9 @@ datasource_ptr datasource_cache::create(parameters const& params) throw config_error(std::string("Could not create datasource. Required ") + "parameter 'type' is missing"); } - datasource_ptr ds; - #ifdef MAPNIK_STATIC_PLUGINS // return if it's created, raise otherwise - ds = create_static_datasource(params); + datasource_ptr ds = create_static_datasource(params); if (ds) { return ds; @@ -114,16 +106,14 @@ datasource_ptr datasource_cache::create(parameters const& params) #ifdef __GNUC__ __extension__ #endif - create_ds create_datasource = reinterpret_cast(itr->second->get_symbol("create")); + datasource_plugin* create_datasource = reinterpret_cast(itr->second->get_symbol("plugin")); if (!create_datasource) { throw std::runtime_error(std::string("Cannot load symbols: ") + itr->second->get_error()); } - ds = datasource_ptr(create_datasource(params), datasource_deleter()); - - return ds; + return create_datasource->create(params); } std::string datasource_cache::plugin_directories() @@ -134,7 +124,29 @@ std::string datasource_cache::plugin_directories() return boost::algorithm::join(plugin_directories_, ", "); } -std::vector datasource_cache::plugin_names() +bool datasource_cache::plugin_registered(const std::string& plugin_name) const +{ +#ifdef MAPNIK_STATIC_PLUGINS + const auto static_names = get_static_datasource_names(); + const auto static_it = std::find(static_names.begin(), static_names.end(), plugin_name); + if (static_it != static_names.end()) + return true; +#endif + +#ifdef MAPNIK_THREADSAFE + std::lock_guard lock(instance_mutex_); +#endif + + std::map>::const_iterator itr; + for (itr = plugins_.begin(); itr != plugins_.end(); ++itr) + { + if (itr->second->name() == plugin_name) + return true; + } + return false; +} + +std::vector datasource_cache::plugin_names() const { std::vector names; diff --git a/src/datasource_cache_static.cpp b/src/datasource_cache_static.cpp index 4da5d3480..97e67aad4 100644 --- a/src/datasource_cache_static.cpp +++ b/src/datasource_cache_static.cpp @@ -26,9 +26,7 @@ #ifdef MAPNIK_STATIC_PLUGINS #include - -// boost -#include +#include #endif // stl @@ -43,6 +41,9 @@ #if defined(MAPNIK_STATIC_PLUGIN_GDAL) #include "input/gdal/gdal_datasource.hpp" #endif +#if defined(MAPNIK_STATIC_PLUGIN_GEOBUF) +#include "input/geobuf/geobuf_datasource.hpp" +#endif #if defined(MAPNIK_STATIC_PLUGIN_GEOJSON) #include "input/geojson/geojson_datasource.hpp" #endif @@ -58,6 +59,9 @@ #if defined(MAPNIK_STATIC_PLUGIN_OGR) #include "input/ogr/ogr_datasource.hpp" #endif +#if defined(MAPNIK_STATIC_PLUGIN_PGRASTER) +#include "input/pgraster/pgraster_datasource.hpp" +#endif #if defined(MAPNIK_STATIC_PLUGIN_OSM) #include "input/osm/osm_datasource.hpp" #endif @@ -76,11 +80,21 @@ #if defined(MAPNIK_STATIC_PLUGIN_SQLITE) #include "input/sqlite/sqlite_datasource.hpp" #endif +#if defined(MAPNIK_STATIC_PLUGIN_TOPOJSON) +#include "input/topojson/topojson_datasource.hpp" +#endif #endif +#define REGISTER_STATIC_DATASOURCE_PLUGIN(classname) \ + { \ + auto plugin = std::make_shared(); \ + plugin->after_load(); \ + ds_map.emplace(std::string{classname::kName}, std::move(plugin)); \ + } namespace mapnik { #ifdef MAPNIK_STATIC_PLUGINS + template datasource_ptr ds_generator(parameters const& params) { @@ -88,56 +102,65 @@ datasource_ptr ds_generator(parameters const& params) } typedef datasource_ptr (*ds_generator_ptr)(parameters const& params); -using datasource_map = std::unordered_map; +using datasource_map = std::unordered_map>; -static datasource_map ds_map = boost::assign::map_list_of +static datasource_map ds_map{}; + +void init_datasource_cache_static() +{ #if defined(MAPNIK_STATIC_PLUGIN_CSV) - (std::string("csv"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(csv_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_GDAL) - (std::string("gdal"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(gdal_datasource_plugin); +#endif +#if defined(MAPNIK_STATIC_PLUGIN_GEOBUF) + REGISTER_STATIC_DATASOURCE_PLUGIN(geobuf_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_GEOJSON) - (std::string("geojson"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(geojson_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_OCCI) - (std::string("occi"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(occi_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_OGR) - (std::string("ogr"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(ogr_datasource_plugin); +#endif +#if defined(MAPNIK_STATIC_PLUGIN_PGRASTER) + REGISTER_STATIC_DATASOURCE_PLUGIN(pgraster_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_OSM) - (std::string("osm"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(osm_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_POSTGIS) - (std::string("postgis"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(postgis_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_RASTER) - (std::string("raster"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(raster_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_RASTERLITE) - (std::string("rasterlite"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(rasterlite_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_SHAPE) - (std::string("shape"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(shape_datasource_plugin); #endif #if defined(MAPNIK_STATIC_PLUGIN_SQLITE) - (std::string("sqlite"), &ds_generator) + REGISTER_STATIC_DATASOURCE_PLUGIN(sqlite_datasource_plugin); #endif - ; +#if defined(MAPNIK_STATIC_PLUGIN_TOPOJSON) + REGISTER_STATIC_DATASOURCE_PLUGIN(topojson_datasource_plugin); #endif +}; -#ifdef MAPNIK_STATIC_PLUGINS datasource_ptr create_static_datasource(parameters const& params) { - datasource_ptr ds; boost::optional type = params.get("type"); datasource_map::iterator it = ds_map.find(*type); if (it != ds_map.end()) { - ds = it->second(params); + return it->second->create(params); } - return ds; + return datasource_ptr{}; } #else datasource_ptr create_static_datasource(parameters const& /*params*/) diff --git a/src/jpeg_reader.cpp b/src/jpeg_reader.cpp index ff704f691..1e14acb59 100644 --- a/src/jpeg_reader.cpp +++ b/src/jpeg_reader.cpp @@ -91,7 +91,6 @@ class jpeg_reader : public image_reader static void attach_stream(j_decompress_ptr cinfo, input_stream* in); }; -namespace { image_reader* create_jpeg_reader(std::string const& filename) { return new jpeg_reader(filename); @@ -101,10 +100,11 @@ image_reader* create_jpeg_reader2(char const* data, size_t size) { return new jpeg_reader(data, size); } - -const bool registered = register_image_reader("jpeg", create_jpeg_reader); -const bool registered2 = register_image_reader("jpeg", create_jpeg_reader2); -} // namespace +void register_jpeg_reader() +{ + const bool registered = register_image_reader("jpeg", create_jpeg_reader); + const bool registered2 = register_image_reader("jpeg", create_jpeg_reader2); +} // ctors template diff --git a/src/json/CMakeLists.txt b/src/json/CMakeLists.txt index 5e57a1f20..4f64355fd 100644 --- a/src/json/CMakeLists.txt +++ b/src/json/CMakeLists.txt @@ -1,4 +1,4 @@ -add_library(json STATIC +target_sources(json PRIVATE extract_bounding_boxes_x3.cpp feature_from_geojson.cpp feature_grammar_x3.cpp @@ -13,7 +13,6 @@ add_library(json STATIC topojson_grammar_x3.cpp unicode_string_grammar_x3.cpp ) -add_library(mapnik::json ALIAS json) target_include_directories(json PRIVATE ${JPEG_INCLUDE_DIR} @@ -26,16 +25,12 @@ target_link_libraries(json PRIVATE mapnik::core ${ICUU_LIB}) set_target_properties(json PROPERTIES POSITION_INDEPENDENT_CODE ON - DEBUG_POSTFIX "${MAPNIK_DEBUG_POSTFIX}" OUTPUT_NAME "mapnikjson" PREFIX "lib" IMPORT_PREFIX "lib" VERSION ${MAPNIK_VERSION} # see mapnik target for explanation SOVERSION ${MAPNIK_VERSION} - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" ) mapnik_install(json) diff --git a/src/mapnik.cpp b/src/mapnik.cpp new file mode 100644 index 000000000..11406cd69 --- /dev/null +++ b/src/mapnik.cpp @@ -0,0 +1,53 @@ +#include +#include // call once +#include "create_image_reader.hpp" +namespace mapnik { + +static void setup_once(); + +void setup() +{ + static std::once_flag init_flag; + std::call_once(init_flag, setup_once); +} +#ifdef MAPNIK_STATIC_PLUGINS +extern void init_datasource_cache_static(); +#endif +static void register_image_readers(); + +void setup_once() +{ +#ifdef MAPNIK_STATIC_PLUGINS + init_datasource_cache_static(); // defined in datasource_cache_static.cpp +#endif + register_image_readers(); +} + +void register_image_readers() +{ +#ifdef HAVE_JPEG + register_jpeg_reader(); +#endif +#ifdef HAVE_PNG + register_png_reader(); +#endif +#ifdef HAVE_TIFF + register_tiff_reader(); +#endif +#ifdef HAVE_WEBP + register_webp_reader(); +#endif +} +} // namespace mapnik + +// only on other systems then windows and in shared builds +#if !defined(MAPNIK_DISABLE_AUTOSETUP) && MAPNIK_STATIC_PLUGINS == 0 && !defined(_WIN32) +namespace { +class AutoSetup final +{ + public: + AutoSetup() { mapnik::setup(); }; +}; +AutoSetup auto_setup{}; +} // namespace +#endif diff --git a/src/memory_datasource.cpp b/src/memory_datasource.cpp index eee058be6..60019011a 100644 --- a/src/memory_datasource.cpp +++ b/src/memory_datasource.cpp @@ -35,10 +35,14 @@ using mapnik::datasource; using mapnik::parameters; -DATASOURCE_PLUGIN(mapnik::memory_datasource) +DATASOURCE_PLUGIN_EXPORT(mapnik::memory_datasource_plugin) namespace mapnik { +DATASOURCE_PLUGIN_IMPL(memory_datasource_plugin, memory_datasource); +DATASOURCE_PLUGIN_EMPTY_AFTER_LOAD(memory_datasource_plugin); +DATASOURCE_PLUGIN_EMPTY_BEFORE_UNLOAD(memory_datasource_plugin); + struct accumulate_extent { accumulate_extent(box2d& ext) @@ -67,7 +71,7 @@ struct accumulate_extent const char* memory_datasource::name() { - return "memory"; + return mapnik::memory_datasource_plugin::kName; } memory_datasource::memory_datasource(parameters const& _params) diff --git a/src/plugin.cpp b/src/plugin.cpp index 6b6c6e8b7..fb37f9a03 100644 --- a/src/plugin.cpp +++ b/src/plugin.cpp @@ -23,6 +23,8 @@ #include #include +#include + #ifdef _WIN32 #define NOMINMAX #include @@ -45,56 +47,15 @@ namespace mapnik { struct _mapnik_lib_t { + std::string name; + std::string error_str; handle dl; -}; - -PluginInfo::PluginInfo(std::string const& filename, std::string const& library_name) - : filename_(filename) - , name_() - , module_(new mapnik_lib_t) -{ -#ifdef _WIN32 - if (module_) - module_->dl = LoadLibraryA(filename.c_str()); - if (module_ && module_->dl) - { - callable_returning_string name_call = - reinterpret_cast(dlsym(module_->dl, library_name.c_str())); - if (name_call) - name_ = name_call(); - callable_returning_void init_once = - reinterpret_cast(dlsym(module_->dl, "on_plugin_load")); - if (init_once) - { - init_once(); - } - } -#else -#ifdef MAPNIK_HAS_DLCFN - if (module_) - module_->dl = dlopen(filename.c_str(), RTLD_LAZY); - if (module_ && module_->dl) - { - callable_returning_string name_call = - reinterpret_cast(dlsym(module_->dl, library_name.c_str())); - if (name_call) - name_ = name_call(); - callable_returning_void init_once = - reinterpret_cast(dlsym(module_->dl, "on_plugin_load")); - if (init_once) - { - init_once(); - } - } -#else - throw std::runtime_error("no support for loading dynamic objects (Mapnik not compiled with -DMAPNIK_HAS_DLCFN)"); -#endif -#endif -} - -PluginInfo::~PluginInfo() -{ - if (module_) + _mapnik_lib_t() + : name{"unknown"} + , error_str{""} + , dl{nullptr} + {} + ~_mapnik_lib_t() { #ifdef MAPNIK_SUPPORTS_DLOPEN /* @@ -108,17 +69,55 @@ PluginInfo::~PluginInfo() in the case that gdal is linked as a shared library. This workaround therefore prevents crashes with gdal 1.11.x and gdal 2.x when using a static libgdal. */ - if (module_->dl && name_ != "gdal" && name_ != "ogr") + if (dl /*&& name_ != "gdal" && name_ != "ogr"*/) // is the gdal issue sill present? We are now + // unregister all drivers for ogal and gdal (todo: + // before_unload call) { #ifndef MAPNIK_NO_DLCLOSE - dlclose(module_->dl), module_->dl = 0; + dlclose(dl); + dl = nullptr; #endif } #endif - delete module_; } +}; + +PluginInfo::PluginInfo(std::string const& filename, std::string const& library_name) + : filename_(filename) + , module_{std::make_unique()} +{ + assert(module_ != nullptr); +#if defined(_WIN32) + module_->dl = LoadLibraryA(filename.c_str()); +#elif defined(MAPNIK_HAS_DLCFN) + module_->dl = dlopen(filename.c_str(), RTLD_LAZY); +#else + throw std::runtime_error("no support for loading dynamic objects (Mapnik not compiled with -DMAPNIK_HAS_DLCFN)"); +#endif +#if defined(MAPNIK_HAS_DLCFN) || defined(_WIN32) + if (module_->dl) + { + datasource_plugin* plugin{reinterpret_cast(get_symbol("plugin"))}; + if (!plugin) + throw std::runtime_error("plugin has a false interface"); //! todo: better error text + module_->name = plugin->name(); + module_->error_str = ""; + plugin->after_load(); + } + else + { + const auto errcode = dlerror(); +#ifdef _WIN32 + module_->error_str = std::system_category().message(errcode); +#else + module_->error_str = errcode ? errcode : ""; +#endif + } +#endif // defined(MAPNIK_HAS_DLCFN) || defined(_WIN32) } +PluginInfo::~PluginInfo() {} + void* PluginInfo::get_symbol(std::string const& sym_name) const { #ifdef MAPNIK_SUPPORTS_DLOPEN @@ -130,13 +129,13 @@ void* PluginInfo::get_symbol(std::string const& sym_name) const std::string const& PluginInfo::name() const { - return name_; + return module_->name; } bool PluginInfo::valid() const { #ifdef MAPNIK_SUPPORTS_DLOPEN - if (module_ && module_->dl && !name_.empty()) + if (module_->dl && !module_->name.empty()) return true; #endif return false; @@ -144,17 +143,7 @@ bool PluginInfo::valid() const std::string PluginInfo::get_error() const { - return std::string("could not open: '") + name_ + "'"; -} - -void PluginInfo::init() -{ - // do any initialization needed -} - -void PluginInfo::exit() -{ - // do any shutdown needed + return std::string{"could not open: '"} + module_->name + "'. Error: " + module_->error_str; } } // namespace mapnik diff --git a/src/png_reader.cpp b/src/png_reader.cpp index 746705b21..799cdeda5 100644 --- a/src/png_reader.cpp +++ b/src/png_reader.cpp @@ -80,8 +80,6 @@ class png_reader : public image_reader static void png_read_data(png_structp png_ptr, png_bytep data, png_size_t length); }; -namespace { - image_reader* create_png_reader(std::string const& filename) { return new png_reader(filename); @@ -92,9 +90,11 @@ image_reader* create_png_reader2(char const* data, std::size_t size) return new png_reader(data, size); } -const bool registered = register_image_reader("png", create_png_reader); -const bool registered2 = register_image_reader("png", create_png_reader2); -} // namespace +void register_png_reader() +{ + const bool registered = register_image_reader("png", create_png_reader); + const bool registered2 = register_image_reader("png", create_png_reader2); +} void user_error_fn(png_structp /*png_ptr*/, png_const_charp error_msg) { diff --git a/src/tiff_reader.cpp b/src/tiff_reader.cpp index fb8b243a8..4446add71 100644 --- a/src/tiff_reader.cpp +++ b/src/tiff_reader.cpp @@ -104,8 +104,6 @@ int tiff_map_proc(thandle_t, tdata_t*, toff_t*) } } // namespace detail -namespace { - image_reader* create_tiff_reader(std::string const& filename) { #if defined(MAPNIK_MEMORY_MAPPED_FILE) @@ -120,9 +118,10 @@ image_reader* create_tiff_reader2(char const* data, std::size_t size) return new tiff_reader(data, size); } -const bool registered = register_image_reader("tiff", create_tiff_reader); -const bool registered2 = register_image_reader("tiff", create_tiff_reader2); - -} // namespace +void register_tiff_reader() +{ + const bool registered = register_image_reader("tiff", create_tiff_reader); + const bool registered2 = register_image_reader("tiff", create_tiff_reader2); +} } // namespace mapnik diff --git a/src/webp_io.cpp b/src/webp_io.cpp new file mode 100644 index 000000000..53b69bef3 --- /dev/null +++ b/src/webp_io.cpp @@ -0,0 +1,44 @@ +#include + +namespace mapnik { +std::string MAPNIK_DECL webp_encoding_error(WebPEncodingError error) +{ + std::string os; + switch (error) + { + case VP8_ENC_ERROR_OUT_OF_MEMORY: + os = "memory error allocating objects"; + break; + case VP8_ENC_ERROR_BITSTREAM_OUT_OF_MEMORY: + os = "memory error while flushing bits"; + break; + case VP8_ENC_ERROR_NULL_PARAMETER: + os = "a pointer parameter is NULL"; + break; + case VP8_ENC_ERROR_INVALID_CONFIGURATION: + os = "configuration is invalid"; + break; + case VP8_ENC_ERROR_BAD_DIMENSION: + os = "picture has invalid width/height"; + break; + case VP8_ENC_ERROR_PARTITION0_OVERFLOW: + os = "partition is bigger than 512k"; + break; + case VP8_ENC_ERROR_PARTITION_OVERFLOW: + os = "partition is bigger than 16M"; + break; + case VP8_ENC_ERROR_BAD_WRITE: + os = "error while flushing bytes"; + break; + case VP8_ENC_ERROR_FILE_TOO_BIG: + os = "file is bigger than 4G"; + break; + default: + mapnik::util::to_string(os, error); + os = "unknown error (" + os + ")"; + break; + } + return os; +} + +} // namespace mapnik diff --git a/src/webp_reader.cpp b/src/webp_reader.cpp index fa85d6dfc..24b942525 100644 --- a/src/webp_reader.cpp +++ b/src/webp_reader.cpp @@ -110,7 +110,6 @@ class webp_reader : public image_reader void init(); }; -namespace { image_reader* create_webp_reader(char const* data, std::size_t size) { return new webp_reader(data, size); @@ -121,10 +120,11 @@ image_reader* create_webp_reader2(std::string const& filename) return new webp_reader(filename); } -const bool registered = register_image_reader("webp", create_webp_reader); -const bool registered2 = register_image_reader("webp", create_webp_reader2); - -} // namespace +void register_webp_reader() +{ + const bool registered = register_image_reader("webp", create_webp_reader); + const bool registered2 = register_image_reader("webp", create_webp_reader2); +} // ctor template diff --git a/src/wkt/CMakeLists.txt b/src/wkt/CMakeLists.txt index bd6332128..7507fd519 100644 --- a/src/wkt/CMakeLists.txt +++ b/src/wkt/CMakeLists.txt @@ -1,10 +1,9 @@ -add_library(wkt STATIC +target_sources(wkt PRIVATE geometry_to_wkt.cpp mapnik_wkt_generator_grammar.cpp wkt_factory.cpp wkt_grammar_x3.cpp ) -add_library(mapnik::wkt ALIAS wkt) target_include_directories(wkt PRIVATE ${MAPNIK_INCLUDE_PATH} @@ -18,16 +17,12 @@ target_link_libraries(wkt PRIVATE mapnik::core) set_target_properties(wkt PROPERTIES POSITION_INDEPENDENT_CODE ON - DEBUG_POSTFIX "${MAPNIK_DEBUG_POSTFIX}" OUTPUT_NAME "mapnikwkt" PREFIX "lib" IMPORT_PREFIX "lib" VERSION ${MAPNIK_VERSION} # see mapnik target for explanation SOVERSION ${MAPNIK_VERSION} - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" ) mapnik_install(wkt) diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt index 549357a9d..f275df53f 100644 --- a/test/CMakeLists.txt +++ b/test/CMakeLists.txt @@ -1,5 +1,5 @@ -find_package(Boost ${BOOST_MIN_VERSION} REQUIRED COMPONENTS program_options) -find_package(PostgreSQL REQUIRED) +mapnik_find_package(Boost ${BOOST_MIN_VERSION} REQUIRED COMPONENTS program_options) +mapnik_find_package(PostgreSQL REQUIRED) include(FetchContent) @@ -100,11 +100,7 @@ add_executable(mapnik-test-unit unit/vertex_adapter/transform_path_adapter.cpp unit/vertex_adapter/vertex_adapter.cpp ) -set_target_properties(mapnik-test-unit PROPERTIES - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) + target_link_libraries(mapnik-test-unit PUBLIC Catch2::Catch2 mapnik::agg @@ -112,6 +108,7 @@ target_link_libraries(mapnik-test-unit PUBLIC mapnik::json mapnik::wkt PostgreSQL::PostgreSQL + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) # workaround since the "offical" include dir would be file(COPY catch_ext.hpp DESTINATION "${CMAKE_CURRENT_BINARY_DIR}") @@ -119,58 +116,41 @@ file(COPY cleanup.hpp DESTINATION "${CMAKE_CURRENT_BINARY_DIR}") target_include_directories(mapnik-test-unit PRIVATE "${Catch2_SOURCE_DIR}/single_include/catch2" "${CMAKE_CURRENT_BINARY_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}") add_executable(agg_rasterizer_integer_overflow_test standalone/agg_rasterizer_integer_overflow_test.cpp) -set_target_properties(agg_rasterizer_integer_overflow_test PROPERTIES - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) target_link_libraries(agg_rasterizer_integer_overflow_test PUBLIC Catch2::Catch2 mapnik::mapnik mapnik::agg mapnik::json + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) target_include_directories(agg_rasterizer_integer_overflow_test PRIVATE "${Catch2_SOURCE_DIR}/single_include/catch2" "${CMAKE_CURRENT_BINARY_DIR}") add_executable(datasource_registration_test standalone/datasource_registration_test.cpp) -set_target_properties(datasource_registration_test PROPERTIES - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) target_link_libraries(datasource_registration_test PUBLIC Catch2::Catch2 mapnik::mapnik mapnik::agg + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) target_include_directories(datasource_registration_test PRIVATE "${Catch2_SOURCE_DIR}/single_include/catch2" "${CMAKE_CURRENT_BINARY_DIR}") add_executable(font_registration_test standalone/font_registration_test.cpp) -set_target_properties(font_registration_test PROPERTIES - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) target_link_libraries(font_registration_test PUBLIC Catch2::Catch2 mapnik::mapnik mapnik::agg mapnik::json + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) target_include_directories(font_registration_test PRIVATE "${Catch2_SOURCE_DIR}/single_include/catch2" "${CMAKE_CURRENT_BINARY_DIR}") -#not workable since boost::filesystem native returns a wstring and the function taskes a std::string add_executable(map_xml_test standalone/map_xml_test.cpp) -set_target_properties(map_xml_test PROPERTIES - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) target_link_libraries(map_xml_test PUBLIC Catch2::Catch2 mapnik::mapnik mapnik::agg mapnik::json + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) target_include_directories(map_xml_test PRIVATE "${Catch2_SOURCE_DIR}/single_include/catch2" "${CMAKE_CURRENT_BINARY_DIR}") diff --git a/test/standalone/agg_rasterizer_integer_overflow_test.cpp b/test/standalone/agg_rasterizer_integer_overflow_test.cpp index 9ab9cfd95..05d3556f0 100644 --- a/test/standalone/agg_rasterizer_integer_overflow_test.cpp +++ b/test/standalone/agg_rasterizer_integer_overflow_test.cpp @@ -1,6 +1,7 @@ #define CATCH_CONFIG_MAIN #include "catch.hpp" +#include #include #include #include @@ -22,6 +23,7 @@ const std::string TEST_CASE("agg_rasterizer_integer_overflow") { + mapnik::setup(); SECTION("coordinates_do_not_overflow_and_polygon_is_rendered") { auto expected_color = mapnik::color("white"); diff --git a/test/standalone/datasource_registration_test.cpp b/test/standalone/datasource_registration_test.cpp index 9408aba9d..1492b8dad 100644 --- a/test/standalone/datasource_registration_test.cpp +++ b/test/standalone/datasource_registration_test.cpp @@ -1,6 +1,7 @@ #define CATCH_CONFIG_MAIN #include "catch.hpp" +#include #include #include #include @@ -11,6 +12,7 @@ TEST_CASE("datasource_cache") { + mapnik::setup(); SECTION("registration") { try diff --git a/test/standalone/font_registration_test.cpp b/test/standalone/font_registration_test.cpp index b651d525a..e2ce758a3 100644 --- a/test/standalone/font_registration_test.cpp +++ b/test/standalone/font_registration_test.cpp @@ -1,6 +1,7 @@ #define CATCH_CONFIG_MAIN #include "catch.hpp" +#include #include #include #include @@ -13,6 +14,7 @@ TEST_CASE("font") { + mapnik::setup(); SECTION("registration") { try diff --git a/test/standalone/map_xml_test.cpp b/test/standalone/map_xml_test.cpp index bf0e53253..71c37cc76 100644 --- a/test/standalone/map_xml_test.cpp +++ b/test/standalone/map_xml_test.cpp @@ -1,6 +1,7 @@ #define CATCH_CONFIG_MAIN #include "catch.hpp" +#include #include #include #include @@ -137,14 +138,17 @@ void load_map(mapnik::Map& m, bfs::path const& path) } } // anonymous namespace - +#ifndef MAPNIK_STATIC_PLUGINS const bool registered = mapnik::datasource_cache::instance().register_datasources((bfs::path("plugins") / "input").generic_string()); - +#endif TEST_CASE("map xml I/O") { + mapnik::setup(); +#ifndef MAPNIK_STATIC_PLUGINS // make sure plugins are loaded REQUIRE(registered); +#endif // make the tests silent since we intentially test error conditions that are noisy auto const severity = mapnik::logger::instance().get_severity(); diff --git a/test/unit/datasource/csv.cpp b/test/unit/datasource/csv.cpp index 93724d05e..d9734edca 100644 --- a/test/unit/datasource/csv.cpp +++ b/test/unit/datasource/csv.cpp @@ -85,17 +85,15 @@ mapnik::datasource_ptr get_csv_ds(std::string const& file_name, bool strict = tr TEST_CASE("csv") { using mapnik::util::from_u8string; - std::string csv_plugin("./plugins/input/csv.input"); - if (mapnik::util::exists(csv_plugin)) + + // check the CSV datasource is loaded + const bool have_csv_plugin = mapnik::datasource_cache::instance().plugin_registered("csv"); + if (have_csv_plugin) { // make the tests silent since we intentionally test error conditions that are noisy auto const severity = mapnik::logger::instance().get_severity(); mapnik::logger::instance().set_severity(mapnik::logger::none); - // check the CSV datasource is loaded - const std::vector plugin_names = mapnik::datasource_cache::instance().plugin_names(); - const bool have_csv_plugin = std::find(plugin_names.begin(), plugin_names.end(), "csv") != plugin_names.end(); - SECTION("CSV I/O errors") { std::string filename = "does_not_exist.csv"; @@ -125,83 +123,77 @@ TEST_CASE("csv") { for (auto create_index : {false, true}) { - if (have_csv_plugin) - { - std::vector broken; - add_csv_files("test/data/csv/fails", broken); - add_csv_files("test/data/csv/warns", broken); - broken.emplace_back("test/data/csv/fails/does_not_exist.csv"); + std::vector broken; + add_csv_files("test/data/csv/fails", broken); + add_csv_files("test/data/csv/warns", broken); + broken.emplace_back("test/data/csv/fails/does_not_exist.csv"); - for (auto const& path : broken) + for (auto const& path : broken) + { + bool require_fail = true; + if (create_index) { - bool require_fail = true; - if (create_index) + int ret = create_disk_index(path); + int ret_posix = (ret >> 8) & 0x000000ff; + INFO(ret); + INFO(ret_posix); + require_fail = (boost::iends_with(path, "feature_id_counting.csv")) ? false : true; + if (!require_fail) { - int ret = create_disk_index(path); - int ret_posix = (ret >> 8) & 0x000000ff; - INFO(ret); - INFO(ret_posix); - require_fail = (boost::iends_with(path, "feature_id_counting.csv")) ? false : true; - if (!require_fail) - { - REQUIRE(mapnik::util::exists(path + ".index")); - } - } - INFO(path); - if (require_fail) - { - REQUIRE_THROWS(get_csv_ds(path)); - } - else - { - CHECK(bool(get_csv_ds(path))); - } - if (mapnik::util::exists(path + ".index")) - { - CHECK(mapnik::util::remove(path + ".index")); + REQUIRE(mapnik::util::exists(path + ".index")); } } + INFO(path); + if (require_fail) + { + REQUIRE_THROWS(get_csv_ds(path)); + } + else + { + CHECK(bool(get_csv_ds(path))); + } + if (mapnik::util::exists(path + ".index")) + { + CHECK(mapnik::util::remove(path + ".index")); + } } } } // END SECTION SECTION("good files") { - if (have_csv_plugin) - { - std::vector good; - add_csv_files("test/data/csv", good); - add_csv_files("test/data/csv/warns", good); + std::vector good; + add_csv_files("test/data/csv", good); + add_csv_files("test/data/csv/warns", good); - for (auto const& path : good) + for (auto const& path : good) + { + // cleanup in the case of a failed previous run + if (mapnik::util::exists(path + ".index")) { - // cleanup in the case of a failed previous run + mapnik::util::remove(path + ".index"); + } + for (auto create_index : {false, true}) + { + if (create_index) + { + int ret = create_disk_index(path); + int ret_posix = (ret >> 8) & 0x000000ff; + INFO(ret); + INFO(ret_posix); + if (!boost::iends_with(path, + "more_headers_than_column_values.csv")) // mapnik-index won't create + // *.index for 0 features + { + CHECK(mapnik::util::exists(path + ".index")); + } + } + auto ds = get_csv_ds(path, false); + // require a non-null pointer returned + REQUIRE(bool(ds)); if (mapnik::util::exists(path + ".index")) { - mapnik::util::remove(path + ".index"); - } - for (auto create_index : {false, true}) - { - if (create_index) - { - int ret = create_disk_index(path); - int ret_posix = (ret >> 8) & 0x000000ff; - INFO(ret); - INFO(ret_posix); - if (!boost::iends_with(path, - "more_headers_than_column_values.csv")) // mapnik-index won't create - // *.index for 0 features - { - CHECK(mapnik::util::exists(path + ".index")); - } - } - auto ds = get_csv_ds(path, false); - // require a non-null pointer returned - REQUIRE(bool(ds)); - if (mapnik::util::exists(path + ".index")) - { - CHECK(mapnik::util::remove(path + ".index")); - } + CHECK(mapnik::util::remove(path + ".index")); } } } diff --git a/test/unit/datasource/gdal.cpp b/test/unit/datasource/gdal.cpp index e7d13b3dc..e83cd5a52 100644 --- a/test/unit/datasource/gdal.cpp +++ b/test/unit/datasource/gdal.cpp @@ -31,8 +31,8 @@ namespace { mapnik::datasource_ptr get_gdal_ds(std::string const& file_name, boost::optional band) { - std::string gdal_plugin("./plugins/input/gdal.input"); - if (!mapnik::util::exists(gdal_plugin)) + const bool have_gdal_plugin = mapnik::datasource_cache::instance().plugin_registered("gdal"); + if (!have_gdal_plugin) { return mapnik::datasource_ptr(); } diff --git a/test/unit/datasource/geobuf.cpp b/test/unit/datasource/geobuf.cpp index 82859851c..4d371ced6 100644 --- a/test/unit/datasource/geobuf.cpp +++ b/test/unit/datasource/geobuf.cpp @@ -37,8 +37,8 @@ TEST_CASE("Geobuf") { - std::string geobuf_plugin("./plugins/input/geobuf.input"); - if (mapnik::util::exists(geobuf_plugin)) + const bool have_geobuf_plugin = mapnik::datasource_cache::instance().plugin_registered("geobuf"); + if (have_geobuf_plugin) { SECTION("Point") { diff --git a/test/unit/datasource/geojson.cpp b/test/unit/datasource/geojson.cpp index 528c0263b..308e51cff 100644 --- a/test/unit/datasource/geojson.cpp +++ b/test/unit/datasource/geojson.cpp @@ -81,8 +81,8 @@ void iterate_over_features(mapnik::featureset_ptr features) TEST_CASE("geojson") { - std::string geojson_plugin("./plugins/input/geojson.input"); - if (mapnik::util::exists(geojson_plugin)) + const bool have_geojson_plugin = mapnik::datasource_cache::instance().plugin_registered("geojson"); + if (have_geojson_plugin) { SECTION("GeoJSON I/O errors") { diff --git a/test/unit/datasource/ogr.cpp b/test/unit/datasource/ogr.cpp index c6bfc8bb9..8441ecc55 100644 --- a/test/unit/datasource/ogr.cpp +++ b/test/unit/datasource/ogr.cpp @@ -28,12 +28,12 @@ #include #include #include -#include +#include TEST_CASE("ogr") { - std::string geojson_plugin("./plugins/input/ogr.input"); - if (mapnik::util::exists(geojson_plugin)) + const bool have_ogr_plugin = mapnik::datasource_cache::instance().plugin_registered("ogr"); + if (have_ogr_plugin) { SECTION("ogr point feature") { diff --git a/test/unit/datasource/shapeindex.cpp b/test/unit/datasource/shapeindex.cpp index 45aaa2334..76881235b 100644 --- a/test/unit/datasource/shapeindex.cpp +++ b/test/unit/datasource/shapeindex.cpp @@ -98,8 +98,8 @@ int create_shapefile_index(std::string const& filename, bool index_parts, bool s TEST_CASE("invalid shapeindex") { - std::string shape_plugin("./plugins/input/shape.input"); - if (mapnik::util::exists(shape_plugin)) + const bool have_shape_plugin = mapnik::datasource_cache::instance().plugin_registered("shape"); + if (have_shape_plugin) { SECTION("Invalid index") { @@ -141,8 +141,8 @@ TEST_CASE("invalid shapeindex") TEST_CASE("shapeindex") { - std::string shape_plugin("./plugins/input/shape.input"); - if (mapnik::util::exists(shape_plugin)) + const bool have_shape_plugin = mapnik::datasource_cache::instance().plugin_registered("shape"); + if (have_shape_plugin) { SECTION("Index") { diff --git a/test/unit/run.cpp b/test/unit/run.cpp index 8bf68870c..94836d7ed 100644 --- a/test/unit/run.cpp +++ b/test/unit/run.cpp @@ -2,6 +2,7 @@ #include "catch.hpp" #include +#include #include #include #include @@ -20,6 +21,7 @@ int main(int argc, char** argv) session.cli(cli); int result = session.applyCommandLine(argc, argv); + mapnik::setup(); if (!plugin_path.empty()) { if (!mapnik::util::exists(plugin_path)) diff --git a/test/visual/CMakeLists.txt b/test/visual/CMakeLists.txt index 3e7ee4919..0c0c306fc 100644 --- a/test/visual/CMakeLists.txt +++ b/test/visual/CMakeLists.txt @@ -4,17 +4,13 @@ add_executable(mapnik-test-visual runner.cpp run.cpp ) -set_target_properties(mapnik-test-visual PROPERTIES - LIBRARY_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" - ARCHIVE_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}/lib" -) target_link_libraries(mapnik-test-visual PRIVATE Catch2::Catch2 Boost::program_options Boost::filesystem mapnik::mapnik mapnik::agg + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) # needed for cleanup.hpp target_include_directories(mapnik-test-visual PRIVATE "${CMAKE_CURRENT_SOURCE_DIR}/..") diff --git a/test/visual/run.cpp b/test/visual/run.cpp index 846ef6d1c..a352f2c34 100644 --- a/test/visual/run.cpp +++ b/test/visual/run.cpp @@ -23,6 +23,7 @@ #include "runner.hpp" #include "config.hpp" +#include #include #include @@ -166,6 +167,7 @@ int main(int argc, char** argv) return 1; } + mapnik::setup(); #ifdef MAPNIK_LOG std::string log_level(vm["log"].as()); log_levels_map::const_iterator level_iter = log_levels.find(log_level); diff --git a/utils/geometry_to_wkb/CMakeLists.txt b/utils/geometry_to_wkb/CMakeLists.txt index 77cc6ea60..96d888178 100644 --- a/utils/geometry_to_wkb/CMakeLists.txt +++ b/utils/geometry_to_wkb/CMakeLists.txt @@ -1,9 +1,8 @@ add_executable(geometry_to_wkb main.cpp) -target_link_libraries(geometry_to_wkb PRIVATE mapnik::mapnik) - -set_target_properties(geometry_to_wkb PROPERTIES - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" +target_link_libraries(geometry_to_wkb PRIVATE + mapnik::mapnik + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) mapnik_install_utility(geometry_to_wkb) diff --git a/utils/geometry_to_wkb/main.cpp b/utils/geometry_to_wkb/main.cpp index a31ffc93a..4cb2a3d1c 100644 --- a/utils/geometry_to_wkb/main.cpp +++ b/utils/geometry_to_wkb/main.cpp @@ -23,6 +23,7 @@ #include #include +#include #include #include #include @@ -40,6 +41,7 @@ int main(int argc, char** argv) std::cerr << "Geometry to WKB converter\n"; + mapnik::setup(); mapnik::datasource_cache::instance().register_datasources("/opt/mapnik/lib/mapnik/input/"); std::string filename(argv[1]); diff --git a/utils/mapnik-index/CMakeLists.txt b/utils/mapnik-index/CMakeLists.txt index 00c20c1e7..111ac934e 100644 --- a/utils/mapnik-index/CMakeLists.txt +++ b/utils/mapnik-index/CMakeLists.txt @@ -11,10 +11,7 @@ target_link_libraries(mapnik-index PRIVATE mapnik::json mapnik::wkt Boost::program_options -) - -set_target_properties(mapnik-index PROPERTIES - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) mapnik_install_utility(mapnik-index) diff --git a/utils/mapnik-index/mapnik-index.cpp b/utils/mapnik-index/mapnik-index.cpp index b7956607a..c95e001a6 100644 --- a/utils/mapnik-index/mapnik-index.cpp +++ b/utils/mapnik-index/mapnik-index.cpp @@ -24,6 +24,7 @@ #include #include #include +#include #include #include #include @@ -62,6 +63,7 @@ int main(int argc, char** argv) { // using namespace mapnik; namespace po = boost::program_options; + mapnik::setup(); bool verbose = false; bool validate_features = false; unsigned int depth = DEFAULT_DEPTH; diff --git a/utils/mapnik-render/CMakeLists.txt b/utils/mapnik-render/CMakeLists.txt index 49bda1eb1..d148e36e5 100644 --- a/utils/mapnik-render/CMakeLists.txt +++ b/utils/mapnik-render/CMakeLists.txt @@ -5,10 +5,7 @@ add_executable(mapnik-render mapnik-render.cpp) target_link_libraries(mapnik-render PRIVATE mapnik::mapnik Boost::program_options -) - -set_target_properties(mapnik-render PROPERTIES - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) mapnik_install_utility(mapnik-render) diff --git a/utils/mapnik-render/mapnik-render.cpp b/utils/mapnik-render/mapnik-render.cpp index 02afc8e0f..1a05a7795 100644 --- a/utils/mapnik-render/mapnik-render.cpp +++ b/utils/mapnik-render/mapnik-render.cpp @@ -1,3 +1,4 @@ +#include #include #include #include @@ -21,6 +22,8 @@ int main(int argc, char** argv) { namespace po = boost::program_options; + mapnik::setup(); + bool verbose = false; bool auto_open = false; int return_value = 0; diff --git a/utils/ogrindex/CMakeLists.txt b/utils/ogrindex/CMakeLists.txt index e3d18f098..fe9593137 100644 --- a/utils/ogrindex/CMakeLists.txt +++ b/utils/ogrindex/CMakeLists.txt @@ -1,9 +1,8 @@ add_executable(ogrindex ogrindex.cpp) -target_link_libraries(ogrindex PRIVATE mapnik::mapnik) - -set_target_properties(ogrindex PROPERTIES - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" +target_link_libraries(ogrindex PRIVATE + mapnik::mapnik + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) mapnik_install_utility(ogrindex) diff --git a/utils/ogrindex/ogrindex.cpp b/utils/ogrindex/ogrindex.cpp index 0fe3978b4..5c955837f 100644 --- a/utils/ogrindex/ogrindex.cpp +++ b/utils/ogrindex/ogrindex.cpp @@ -27,6 +27,7 @@ #include #include +#include #include #include #include @@ -60,6 +61,7 @@ int main(int argc, char** argv) double ratio = DEFAULT_RATIO; vector ogr_files; + mapnik::setup(); try { po::options_description desc("ogrindex utility"); diff --git a/utils/pgsql2sqlite/CMakeLists.txt b/utils/pgsql2sqlite/CMakeLists.txt index 116dd9c35..3657b31ae 100644 --- a/utils/pgsql2sqlite/CMakeLists.txt +++ b/utils/pgsql2sqlite/CMakeLists.txt @@ -1,6 +1,6 @@ -find_package(Boost ${BOOST_MIN_VERSION} REQUIRED COMPONENTS program_options) -find_package(SQLite3 REQUIRED) -find_package(PostgreSQL REQUIRED) +mapnik_find_package(Boost ${BOOST_MIN_VERSION} REQUIRED COMPONENTS program_options) +mapnik_find_package(SQLite3 REQUIRED) +mapnik_find_package(PostgreSQL REQUIRED) add_executable(pgsql2sqlite main.cpp @@ -13,10 +13,7 @@ target_link_libraries(pgsql2sqlite PRIVATE PostgreSQL::PostgreSQL Boost::program_options mapnik::mapnik -) - -set_target_properties(pgsql2sqlite PROPERTIES - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) mapnik_install_utility(pgsql2sqlite) diff --git a/utils/pgsql2sqlite/main.cpp b/utils/pgsql2sqlite/main.cpp index c87692090..9ab101485 100644 --- a/utils/pgsql2sqlite/main.cpp +++ b/utils/pgsql2sqlite/main.cpp @@ -24,6 +24,7 @@ #include "pgsql2sqlite.hpp" #include #include +#include #include "connection_manager.hpp" #include @@ -46,6 +47,7 @@ int main(int argc, char** argv) po::options_description desc("Postgresql/PostGIS to SQLite3 converter\n Options"); std::string usage = "usage: pgsql2sqlite --dbname db --table planet_osm_line --file osm.sqlite --query \"select * " "from planet_osm_line\""; + mapnik::setup(); try { // clang-format off diff --git a/utils/shapeindex/CMakeLists.txt b/utils/shapeindex/CMakeLists.txt index 520b11d4d..e0e1d93d9 100644 --- a/utils/shapeindex/CMakeLists.txt +++ b/utils/shapeindex/CMakeLists.txt @@ -8,10 +8,7 @@ target_include_directories(shapeindex PRIVATE ../../plugins/input/shape) target_link_libraries(shapeindex PRIVATE Boost::program_options mapnik::mapnik -) - -set_target_properties(shapeindex PROPERTIES - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) mapnik_install_utility(shapeindex) diff --git a/utils/shapeindex/shapeindex.cpp b/utils/shapeindex/shapeindex.cpp index f310a572c..6602257e1 100644 --- a/utils/shapeindex/shapeindex.cpp +++ b/utils/shapeindex/shapeindex.cpp @@ -23,6 +23,7 @@ #include #include #include +#include #include #include #include @@ -58,6 +59,7 @@ int main(int argc, char** argv) double ratio = DEFAULT_RATIO; std::vector shape_files; + mapnik::setup(); try { po::options_description desc("shapeindex utility"); diff --git a/utils/svg2png/CMakeLists.txt b/utils/svg2png/CMakeLists.txt index 7d6e4b657..e2afca177 100644 --- a/utils/svg2png/CMakeLists.txt +++ b/utils/svg2png/CMakeLists.txt @@ -8,10 +8,7 @@ target_link_libraries(svg2png PRIVATE Boost::program_options mapnik::mapnik mapnik::agg -) - -set_target_properties(svg2png PROPERTIES - RUNTIME_OUTPUT_DIRECTORY "${MAPNIK_OUTPUT_DIR}" + ICU::data ICU::i18n ICU::uc # needed for the static build (TODO: why isn't this correctly propagated from mapnik::mapnik?) ) mapnik_install_utility(svg2png) diff --git a/utils/svg2png/svg2png.cpp b/utils/svg2png/svg2png.cpp index 281cd67a9..250e38ba4 100644 --- a/utils/svg2png/svg2png.cpp +++ b/utils/svg2png/svg2png.cpp @@ -26,6 +26,7 @@ #include #include +#include #include #include #include @@ -160,6 +161,7 @@ int main(int argc, char** argv) bool strict = false; int status = 0; std::vector svg_files; + mapnik::setup(); mapnik::logger::instance().set_severity(mapnik::logger::error); double scale_factor = 1.0; std::string usage = "Usage: svg2png [options] ";