fix: build compilation issue

.github/workflows/linux-build-test.yml

@@ -24,7 +24,7 @@ jobs:
               g++ \
               cmake \
               ninja-build \
-              libboost-locale-dev libboost-thread-dev libboost-filesystem-dev libboost-log-dev libboost-stacktrace-dev \
+              libboost-locale-dev libboost-thread-dev libboost-filesystem-dev libboost-log-dev libboost-stacktrace-dev libboost-container-dev \
               libssl-dev \
               libgstreamer1.0-dev libgstreamer-plugins-base1.0-dev \
               libwayland-dev libwayland-server0 libinput-dev libxkbcommon-dev libgbm-dev \
@@ -173,7 +173,7 @@ jobs:
           sudo apt-get install -y libunwind-dev
           sudo apt-get install -y \
               ninja-build \
-              libboost-locale-dev libboost-thread-dev libboost-filesystem-dev libboost-log-dev libboost-stacktrace-dev \
+              libboost-locale-dev libboost-thread-dev libboost-filesystem-dev libboost-log-dev libboost-stacktrace-dev libboost-container-dev \
               libssl-dev \
               libgstreamer1.0-dev libgstreamer-plugins-base1.0-dev \
               libwayland-dev libwayland-server0 libinput-dev libxkbcommon-dev libgbm-dev \

docker/wolf.Dockerfile

@@ -14,7 +14,7 @@ RUN apt-get update -y && \
     ccache \
     git \
     clang \
-    libboost-thread-dev libboost-locale-dev libboost-filesystem-dev libboost-log-dev libboost-stacktrace-dev \
+    libboost-thread-dev libboost-locale-dev libboost-filesystem-dev libboost-log-dev libboost-stacktrace-dev libboost-container-dev \
     libwayland-dev libwayland-server0 libinput-dev libxkbcommon-dev libgbm-dev \
     libcurl4-openssl-dev \
     libssl-dev \

docs/modules/dev/pages/manual_build.adoc

@@ -131,7 +131,7 @@ apt-get install -y --no-install-recommends \
     ccache \
     git \
     clang \
-    libboost-thread-dev libboost-locale-dev libboost-filesystem-dev libboost-log-dev libboost-stacktrace-dev \
+    libboost-thread-dev libboost-locale-dev libboost-filesystem-dev libboost-log-dev libboost-stacktrace-dev libboost-container-dev \
     libwayland-dev libwayland-server0 libinput-dev libxkbcommon-dev libgbm-dev \
     libcurl4-openssl-dev \
     libssl-dev \

src/core/src/platforms/all/docker/CMakeLists.txt

@@ -11,20 +11,6 @@ target_sources(wolf_docker
         docker/json_formatters.hpp
         docker/docker.cpp)
 
-FetchContent_Declare(
-        boost_json
-        GIT_REPOSITORY https://github.com/boostorg/json.git
-        GIT_TAG "boost-1.75.0")
-
-FetchContent_GetProperties(boost_json)
-if (NOT boost_json_POPULATED)
-    FetchContent_Populate(boost_json)
-
-    add_library(boost_json INTERFACE)
-    target_include_directories(boost_json INTERFACE ${boost_json_SOURCE_DIR}/include)
-    target_link_libraries_system(wolf_docker PRIVATE boost_json)
-endif ()
-
 find_package(CURL) # REQUIRED UnixSockets removed because it doesn't work in Github Actions
 target_link_libraries_system(wolf_docker
         PRIVATE

src/core/src/platforms/all/docker/docker/docker.cpp

@@ -1,5 +1,5 @@
-#include <boost/json/src.hpp>
 #include <core/docker.hpp>
+#include <helpers/utils.hpp>
 #include <curl/curl.h>
 #include <docker/formatters.hpp>
 #include <docker/json_formatters.hpp>
@@ -9,6 +9,7 @@
 
 namespace wolf::core::docker {
 using namespace ranges;
+using namespace utils;
 namespace json = boost::json;
 
 enum METHOD {
@@ -104,23 +105,12 @@ req(CURL *handle,
   }
 }
 
-json::value parse(std::string_view json) {
-  json::error_code ec;
-  auto parsed = json::parse({json.data(), json.size()}, ec);
-  if (!ec) {
-    return parsed;
-  } else {
-    logs::log(logs::error, "Error while parsing JSON: {} \n {}", ec.message(), json);
-    return json::object(); // Returning an empty object should allow us to continue most of the times
-  }
-}
-
 std::optional<Container> DockerAPI::get_by_id(std::string_view id) const {
   if (auto conn = docker_connect(socket_path)) {
     auto url = fmt::format("http://localhost/{}/containers/{}/json", DOCKER_API_VERSION, id);
     auto raw_msg = req(conn.value().get(), GET, url);
     if (raw_msg && raw_msg->first == 200) {
-      auto json = parse(raw_msg->second);
+      auto json = parse_json(raw_msg->second);
       return json::value_to<Container>(json);
     } else if (raw_msg) {
       logs::log(logs::warning, "[CURL] error {} - {}", raw_msg->first, raw_msg->second);
@@ -135,7 +125,7 @@ std::vector<Container> DockerAPI::get_containers(bool all) const {
     auto url = fmt::format("http://localhost/{}/containers/json{}", DOCKER_API_VERSION, all ? "?all=true" : "");
     auto raw_msg = req(conn.value().get(), GET, url);
     if (raw_msg && raw_msg->first == 200) {
-      auto json = parse(raw_msg->second);
+      auto json = parse_json(raw_msg->second);
       auto containers = json::value_to<std::vector<json::value>>(json);
       return containers                                                            //
              | ranges::views::transform([this](const json::value &container) {     //
@@ -173,7 +163,7 @@ std::optional<Container> DockerAPI::create(const Container &container,
       exposed_ports[fmt::format("{}/{}", port.public_port, port.type == docker::TCP ? "tcp" : "udp")] = json::object();
     }
 
-    auto post_params = parse(custom_params).as_object();
+    auto post_params = parse_json(custom_params).as_object();
     post_params["Image"] = container.image;
     merge_array(&post_params, "Env", json::value_from(container.env).as_array());
 
@@ -190,7 +180,7 @@ std::optional<Container> DockerAPI::create(const Container &container,
     auto json_payload = json::serialize(post_params);
     auto raw_msg = req(conn.value().get(), POST, url, json_payload);
     if (raw_msg && raw_msg->first == 201) {
-      auto json = parse(raw_msg->second);
+      auto json = parse_json(raw_msg->second);
       auto created_id = json.at("Id").as_string();
       return get_by_id(std::string_view{created_id.data(), created_id.size()});
     } else if (raw_msg && raw_msg->first == 404) { // 404 returned when the image is not present
@@ -335,7 +325,7 @@ bool DockerAPI::exec(std::string_view id, const std::vector<std::string_view> &c
     auto raw_msg = req(conn.value().get(), POST, api_url, json_payload);
     if (raw_msg && raw_msg->first == 201) {
       // Exec request created, start it
-      auto json = parse(raw_msg->second);
+      auto json = parse_json(raw_msg->second);
       std::string exec_id = json.at("Id").as_string().data();
       api_url = fmt::format("http://localhost/{}/exec/{}/start", DOCKER_API_VERSION, exec_id);
       post_params = json::object{{"Detach", false}, {"Tty", false}};
@@ -347,7 +337,7 @@ bool DockerAPI::exec(std::string_view id, const std::vector<std::string_view> &c
         api_url = fmt::format("http://localhost/{}/exec/{}/json", DOCKER_API_VERSION, exec_id);
         raw_msg = req(conn.value().get(), GET, api_url);
         if (raw_msg && raw_msg->first == 200) {
-          json = parse(raw_msg->second);
+          json = parse_json(raw_msg->second);
           auto exit_code = json.at("ExitCode").as_int64();
           if (exit_code != 0) {
             logs::log(logs::warning, "Docker exec failed ({}), {}", exit_code, console);

src/core/src/platforms/all/helpers/CMakeLists.txt

@@ -24,9 +24,17 @@ FetchContent_MakeAvailable(fmtlib)
 target_link_libraries_system(wolf_helpers INTERFACE fmt::fmt-header-only)
 
 # Boost for logging
-find_package(Boost REQUIRED COMPONENTS log)
+find_package(Boost REQUIRED COMPONENTS log container)
 include_directories(${Boost_INCLUDE_DIRS})
 target_link_libraries(wolf_helpers INTERFACE ${Boost_LIBRARIES})
 
+FetchContent_Declare(
+        boost_json
+        GIT_REPOSITORY https://github.com/boostorg/json.git
+        GIT_TAG "boost-1.75.0")
+
+FetchContent_MakeAvailable(boost_json)
+target_link_libraries(wolf_helpers INTERFACE Boost::json)
+
 # All users of this library will need at least C++17
 target_compile_features(wolf_helpers INTERFACE cxx_std_17)

src/core/src/platforms/all/helpers/helpers/utils.hpp

@@ -1,6 +1,8 @@
 #pragma once
 #include <algorithm>
 #include <boost/endian.hpp>
+#include <boost/json.hpp>
+#include <helpers/logger.hpp>
 #include <range/v3/view.hpp>
 #include <sstream>
 #include <stdlib.h>
@@ -111,4 +113,16 @@ map_to_string(const std::map<std::string, std::string> &m, char val_separator =
   return ss.str();
 }
 
+namespace json = boost::json;
+inline json::value parse_json(std::string_view json) {
+  json::error_code ec;
+  auto parsed = json::parse({json.data(), json.size()}, ec);
+  if (!ec) {
+    return parsed;
+  } else {
+    logs::log(logs::error, "Error while parsing JSON: {} \n {}", ec.message(), json);
+    return json::object(); // Returning an empty object should allow us to continue most of the times
+  }
+}
+
 } // namespace utils

src/moonlight-server/CMakeLists.txt

@@ -65,20 +65,6 @@ else ()
 endif ()
 FetchContent_MakeAvailable(cpptrace)
 
-FetchContent_Declare(
-        boost_json
-        GIT_REPOSITORY https://github.com/boostorg/json.git
-        GIT_TAG "boost-1.75.0")
-
-FetchContent_GetProperties(boost_json)
-if (NOT boost_json_POPULATED)
-    FetchContent_Populate(boost_json)
-
-    add_library(boost_json INTERFACE)
-    target_include_directories(boost_json INTERFACE ${boost_json_SOURCE_DIR}/include)
-endif ()
-target_link_libraries_system(wolf_runner PRIVATE boost_json)
-
 ##############################
 
 find_package(Threads REQUIRED)

src/moonlight-server/runners/docker.hpp

@@ -1,5 +1,4 @@
 #pragma once
-#include <boost/json/src.hpp>
 #include <boost/thread/thread.hpp>
 #include <chrono>
 #include <control/control.hpp>
@@ -194,9 +193,8 @@ void RunDocker::run(std::size_t session_id,
   auto final_json_opts = this->base_create_json;
   if (get_vendor(render_node) == NVIDIA && !utils::get_env("NVIDIA_DRIVER_VOLUME_NAME")) {
     logs::log(logs::info, "NVIDIA_DRIVER_VOLUME_NAME not set, assuming nvidia driver toolkit is installed..");
-    boost::json::error_code ec;
-    auto parsed_json = boost::json::parse({final_json_opts.data(), final_json_opts.size()}, ec).as_object();
-    if (!ec) {
+    {
+      auto parsed_json = utils::parse_json(final_json_opts).as_object();
       auto default_gpu_config = boost::json::array{                    // [
                                                    boost::json::object{// {
                                                                        {"Driver", "nvidia"},