vasek/setup-resource-tweaks: updated dependencies; added several twea…

…ks to mapproxy-setup-resource to allow remote mapproxy signalling

mapproxy/src/calipers/CMakeLists.txt

@@ -10,7 +10,7 @@ set(mp-calipers_SOURCES
 
 add_library(mp-calipers ${mp-calipers_SOURCES})
 target_link_libraries(mp-calipers ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mp-calipers ${MODULE_DEFINITIONS})
+target_compile_definitions(mp-calipers PRIVATE ${MODULE_DEFINITIONS})
 buildsys_library(mp-calipers)
 
 # ------------------------------------------------------------------------
@@ -29,7 +29,7 @@ set(mapproxy-calipers_SOURCES
 
 add_executable(mapproxy-calipers ${mapproxy-calipers_SOURCES})
 target_link_libraries(mapproxy-calipers mp-calipers ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-calipers ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-calipers PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(mapproxy-calipers)
 set_target_version(mapproxy-calipers ${vts-mapproxy_VERSION})
 

mapproxy/src/generatevrtwo/CMakeLists.txt

@@ -10,7 +10,7 @@ set(mp-generatevrtwo_SOURCES
 
 add_library(mp-generatevrtwo ${mp-generatevrtwo_SOURCES})
 target_link_libraries(mp-generatevrtwo ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mp-generatevrtwo ${MODULE_DEFINITIONS})
+target_compile_definitions(mp-generatevrtwo PRIVATE ${MODULE_DEFINITIONS})
 buildsys_library(mp-generatevrtwo)
 
 # ------------------------------------------------------------------------
@@ -29,7 +29,7 @@ set(generatevrtwo_SOURCES
 
 add_executable(generatevrtwo ${generatevrtwo_SOURCES})
 target_link_libraries(generatevrtwo mp-generatevrtwo ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(generatevrtwo ${MODULE_DEFINITIONS})
+target_compile_definitions(generatevrtwo PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(generatevrtwo)
 set_target_version(generatevrtwo ${vts-mapproxy_VERSION})
 

mapproxy/src/mapproxy/CMakeLists.txt

@@ -82,7 +82,7 @@ define_module(LIBRARY mapproxy-core
 add_library(mapproxy-core STATIC ${mapproxy-core_SOURCES})
 buildsys_library(mapproxy-core)
 target_link_libraries(mapproxy-core ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-core ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-core PRIVATE ${MODULE_DEFINITIONS})
 
 # ------------------------------------------------------------------------
 # GDAL machinery
@@ -106,7 +106,7 @@ define_module(LIBRARY mapproxy-gdal
 add_library(mapproxy-gdal STATIC ${mapproxy-gdal_SOURCES})
 buildsys_library(mapproxy-gdal)
 target_link_libraries(mapproxy-gdal ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-gdal ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-gdal PRIVATE ${MODULE_DEFINITIONS})
 
 # ------------------------------------------------------------------------
 # Main mapproxy binary
@@ -225,7 +225,7 @@ add_executable(mapproxy
   ${mapproxy_OL_SOURCES}
   )
 target_link_libraries(mapproxy mapproxy-core ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(mapproxy)
 set_target_version(mapproxy ${vts-mapproxy_VERSION})
 

mapproxy/src/mapproxy/mapproxy.cpp

@@ -33,43 +33,43 @@ namespace fs = boost::filesystem;
 bool Mapproxy::supportsReferenceFrame(const std::string &referenceFrame)
 {
     const auto reply
-        (ctrl_.command("supports-reference-frame", referenceFrame));
+        (ctrl_->command("supports-reference-frame", referenceFrame));
     if (reply.empty()) {
         LOGTHROW(err2, std::runtime_error)
             << "Invalid reply from mapproxy.";
     }
 
-    return ctrl_.parseBoolean(reply.front());
+    return ctrl_->parseBoolean(reply.front());
 }
 
 bool Mapproxy::has(const Resource::Id &resourceId) const
 {
-    const auto reply(ctrl_.command("has-resource", resourceId.referenceFrame
+    const auto reply(ctrl_->command("has-resource", resourceId.referenceFrame
                                    , resourceId.group, resourceId.id));
     if (reply.empty()) {
         LOGTHROW(err2, std::runtime_error)
             << "Invalid reply from mapproxy.";
     }
 
-    return ctrl_.parseBoolean(reply.front());
+    return ctrl_->parseBoolean(reply.front());
 }
 
 bool Mapproxy::isReady(const Resource::Id &resourceId) const
 {
-    const auto reply(ctrl_.command("is-resource-ready"
+    const auto reply(ctrl_->command("is-resource-ready"
                                    , resourceId.referenceFrame
                                    , resourceId.group, resourceId.id));
     if (reply.empty()) {
         LOGTHROW(err2, std::runtime_error)
             << "Invalid reply from mapproxy.";
     }
 
-    return ctrl_.parseBoolean(reply.front());
+    return ctrl_->parseBoolean(reply.front());
 }
 
 std::string Mapproxy::url(const Resource::Id &resourceId) const
 {
-    const auto reply(ctrl_.command("resource-url", resourceId.referenceFrame
+    const auto reply(ctrl_->command("resource-url", resourceId.referenceFrame
                                    , resourceId.group, resourceId.id));
     if (reply.empty()) {
         LOGTHROW(err2, std::runtime_error)
@@ -81,26 +81,26 @@ std::string Mapproxy::url(const Resource::Id &resourceId) const
 
 std::uint64_t Mapproxy::updateResources()
 {
-    auto reply(ctrl_.command("update-resources"));
+    auto reply(ctrl_->command("update-resources"));
     if (reply.size() < 2) { return 0; }
     return boost::lexical_cast<std::uint64_t>(reply[1]);
 }
 
 bool Mapproxy::updatedSince(std::uint64_t timestamp) const
 {
-    const auto reply(ctrl_.command("updated-since", timestamp));
+    const auto reply(ctrl_->command("updated-since", timestamp));
     if (reply.empty()) {
         LOGTHROW(err2, std::runtime_error)
             << "Invalid reply from mapproxy.";
     }
 
-    return ctrl_.parseBoolean(reply.front());
+    return ctrl_->parseBoolean(reply.front());
 }
 
 bool Mapproxy::updatedSince(const Resource::Id &resourceId
                             , std::uint64_t timestamp, bool nothrow) const
 {
-    const auto reply(ctrl_.command("updated-since", timestamp
+    const auto reply(ctrl_->command("updated-since", timestamp
                                    , resourceId.referenceFrame
                                    , resourceId.group, resourceId.id
                                    , nothrow ? "true" : "false"));
@@ -109,5 +109,5 @@ bool Mapproxy::updatedSince(const Resource::Id &resourceId
             << "Invalid reply from mapproxy.";
     }
 
-    return ctrl_.parseBoolean(reply.front());
+    return ctrl_->parseBoolean(reply.front());
 }

mapproxy/src/mapproxy/mapproxy.hpp

@@ -40,8 +40,8 @@
  */
 class Mapproxy {
 public:
-    Mapproxy(const boost::filesystem::path &ctrl)
-        : ctrl_(ctrl)
+    Mapproxy(const std::string &ctrl)
+        : ctrl_(service::ctrlClientFactory(ctrl))
     {}
 
     /** Is given reference frame supported?
@@ -77,8 +77,7 @@ class Mapproxy {
         const;
 
 private:
-    // mutable because we want to mark non-changing functions as const
-    mutable service::CtrlClient ctrl_;
+    std::shared_ptr<service::CtrlClientBase> ctrl_;
 };
 
 

mapproxy/src/mapproxy/test/CMakeLists.txt

@@ -12,6 +12,6 @@ set(generate-tileindex_SOURCES
 
 add_executable(mapproxy-generate-tileindex ${generate-tileindex_SOURCES})
 target_link_libraries(mapproxy-generate-tileindex ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-generate-tileindex ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-generate-tileindex PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(mapproxy-generate-tileindex)
 set_target_version(mapproxy-generate-tileindex ${vts-mapproxy_VERSION})

mapproxy/src/rf-mask/CMakeLists.txt

@@ -12,7 +12,7 @@ set(rf-mask_SOURCES
 
 add_executable(mapproxy-rf-mask ${rf-mask_SOURCES})
 target_link_libraries(mapproxy-rf-mask ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-rf-mask ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-rf-mask PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(mapproxy-rf-mask)
 set_target_version(mapproxy-rf-mask ${vts-mapproxy_VERSION})
 

mapproxy/src/setup-resource/CMakeLists.txt

@@ -12,7 +12,7 @@ set(setup-resource_SOURCES
 
 add_executable(mapproxy-setup-resource ${setup-resource_SOURCES})
 target_link_libraries(mapproxy-setup-resource ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-setup-resource ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-setup-resource PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(mapproxy-setup-resource)
 set_target_version(mapproxy-setup-resource ${vts-mapproxy_VERSION})
 

mapproxy/src/setup-resource/main.cpp

@@ -100,10 +100,14 @@ struct Config {
 
     fs::path mapproxyDataRoot;
     fs::path mapproxyDefinitionDir;
-    fs::path mapproxyCtrl;
+    std::string mapproxyCtrl;
 
     bool parallel = true;
 
+    struct {
+        boost::optional<fs::path> datasetHome;
+    } override;
+
     Config()
         : format(RasterFormat::jpg)
         , transparent(false)
@@ -195,6 +199,10 @@ void SetupResource::configuration(po::options_description &cmdline
          , "Desired bottom LOD. The actual bottom might be deeper in "
          "case of more detail dataset.")
 
+        ("override.datasetHome", po::value<fs::path>()
+         , "Dataset home path override if the default is undesirable. "
+         "Relative to --dataRoot. Use with caution.")
+
         ("background", po::value<vrtwo::Color>()
         , "Optional background. If whole warped tile contains this "
          "color it is left empty in the output. Solid dataset with this color "
@@ -215,7 +223,7 @@ void SetupResource::configuration(po::options_description &cmdline
          , po::value(&config_.mapproxyDefinitionDir)->required()
          , "Path to mapproxy resource definition directory.")
         ("mapproxy.ctrl", po::value(&config_.mapproxyCtrl)->required()
-         , "Path to mapproxy control socket.")
+         , "Mapproxy control socket path/CTRL URI.")
         ;
 
     pd.add("dataset", 1)
@@ -249,6 +257,11 @@ void SetupResource::configure(const po::variables_map &vars)
         config_.background = vars["background"].as<vrtwo::Color>();
     }
 
+    if (vars.count("override.datasetHome")) {
+        config_.override.datasetHome = vars["override.datasetHome"]
+            .as<fs::path>();
+    }
+
     // absolutize destination paths
     config_.mapproxyDataRoot = fs::absolute(config_.mapproxyDataRoot);
     config_.mapproxyDefinitionDir
@@ -827,8 +840,9 @@ int SetupResource::run()
     const auto datasetFileName(fs::path(dataset_).filename());
 
     const auto datasetHome
-        (utility::addExtension(datasetFileName
-                               , "." + md5sum(dataset_)));
+        (config_.override.datasetHome.value_or
+         (utility::addExtension(datasetFileName
+                                , "." + md5sum(dataset_))));
 
     const auto rootDir(config.mapproxyDataRoot / datasetHome);
     const auto baseDatasetPath("original-dataset" / datasetFileName);
@@ -851,7 +865,7 @@ int SetupResource::run()
 
                 if (linkDataset_ == DatasetLink::relative) {
                     // relativize already absolute path
-                    dst = utility::lexically_relative(dst, rootDir);
+                    dst = dst.lexically_relative(rootDir / ".");
                 }
 
                 // link

mapproxy/src/tiling/CMakeLists.txt

@@ -10,7 +10,7 @@ set(mp-tiling_SOURCES
 
 add_library(mp-tiling ${mp-tiling_SOURCES})
 target_link_libraries(mp-tiling ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mp-tiling ${MODULE_DEFINITIONS})
+target_compile_definitions(mp-tiling PRIVATE ${MODULE_DEFINITIONS})
 buildsys_library(mp-tiling)
 
 # ------------------------------------------------------------------------
@@ -29,7 +29,7 @@ set(mapproxy-tiling_SOURCES
 
 add_executable(mapproxy-tiling ${mapproxy-tiling_SOURCES})
 target_link_libraries(mapproxy-tiling mp-tiling ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-tiling ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-tiling PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(mapproxy-tiling)
 set_target_version(mapproxy-tiling ${vts-mapproxy_VERSION})
 
@@ -39,7 +39,7 @@ set(sub-tiling_SOURCES
 
 add_executable(mapproxy-sub-tiling ${sub-tiling_SOURCES})
 target_link_libraries(mapproxy-sub-tiling mp-tiling ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-sub-tiling ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-sub-tiling PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(mapproxy-sub-tiling)
 set_target_version(mapproxy-sub-tiling ${vts-mapproxy_VERSION})
 

mapproxy/src/tools/CMakeLists.txt

@@ -14,7 +14,7 @@ set(mapproxy-ti2mmti_SOURCES
 
 add_executable(mapproxy-ti2mmti ${mapproxy-ti2mmti_SOURCES})
 target_link_libraries(mapproxy-ti2mmti ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-ti2mmti ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-ti2mmti PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(mapproxy-ti2mmti)
 set_target_version(mapproxy-ti2mmti ${vts-mapproxy_VERSION})
 
@@ -26,6 +26,6 @@ set(mapproxy-querymmti_SOURCES
 
 add_executable(mapproxy-querymmti ${mapproxy-querymmti_SOURCES})
 target_link_libraries(mapproxy-querymmti ${MODULE_LIBRARIES})
-buildsys_target_compile_definitions(mapproxy-querymmti ${MODULE_DEFINITIONS})
+target_compile_definitions(mapproxy-querymmti PRIVATE ${MODULE_DEFINITIONS})
 buildsys_binary(mapproxy-querymmti)
 set_target_version(mapproxy-querymmti ${vts-mapproxy_VERSION})