Migrate boost filesystem to std filesystem (#253)

Author: awegrzyn

.github/workflows/build.yml

@@ -10,7 +10,7 @@ jobs:
     - name: Install boost, curl and lcov
       run: |
         sudo apt-get update
-        sudo apt-get install -yq libcurl4-openssl-dev libboost-system1.71-dev libboost-filesystem1.71-dev libboost-test1.71-dev libboost-program-options1.71-dev lcov
+        sudo apt-get install -yq libcurl4-openssl-dev libboost-system1.71-dev libboost-test1.71-dev libboost-program-options1.71-dev lcov
     - name: Run CMake
       run: mkdir build && cd build && cmake .. -DCMAKE_BUILD_TYPE=Debug
     - name: Build project

CMakeLists.txt

@@ -17,7 +17,7 @@ endif()
 
 # Define project
 project(Monitoring
-  VERSION 3.8.5
+  VERSION 3.8.6
   DESCRIPTION "O2 Monitoring library"
   LANGUAGES CXX
 )
@@ -59,7 +59,7 @@ endif()
 
 list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_LIST_DIR}/cmake")
 
-find_package(Boost REQUIRED COMPONENTS unit_test_framework program_options system filesystem)
+find_package(Boost REQUIRED COMPONENTS unit_test_framework program_options system)
 find_package(ApMon MODULE)
 find_package(CURL MODULE)
 find_package(RdKafka CONFIG)
@@ -137,7 +137,6 @@ target_link_libraries(Monitoring
     Boost::boost
   PRIVATE
     Boost::system
-    Boost::filesystem
     pthread
     $<$<BOOL:${ApMon_FOUND}>:ApMon::ApMon>
     $<$<BOOL:${RdKafka_FOUND}>:RdKafka::rdkafka++>
@@ -241,7 +240,7 @@ foreach (test ${TEST_SRCS})
   add_executable(${test_name} ${test})
   target_link_libraries(${test_name}
     PRIVATE
-      Monitoring Boost::unit_test_framework Boost::filesystem
+      Monitoring Boost::unit_test_framework
   )
   add_test(NAME ${test_name} COMMAND ${test_name})
   set_tests_properties(${test_name} PROPERTIES TIMEOUT 60)

src/Transports/Unix.cxx

@@ -15,7 +15,7 @@
 
 #include "Unix.h"
 #include <string>
-#include <boost/filesystem.hpp>
+#include <filesystem>
 #include "../MonLogger.h"
 
 namespace o2
@@ -29,7 +29,7 @@ namespace transports
 #if defined(BOOST_ASIO_HAS_LOCAL_SOCKETS)
 Unix::Unix(const std::string& socketPath) : mSocket(mIoService), mEndpoint(socketPath)
 {
-  if (!boost::filesystem::exists(socketPath)) {
+  if (!std::filesystem::exists(socketPath)) {
     MonLogger::Get(Severity::Error) << "Unix socket path is invalid: " << socketPath << MonLogger::End();
   } else {
     mSocket.open();

test/testMonitoringFactory.cxx

@@ -15,8 +15,7 @@
 #define BOOST_TEST_DYN_LINK
 #include <boost/test/unit_test.hpp>
 #include <boost/lexical_cast.hpp>
-#include <boost/filesystem/path.hpp>
-#include <boost/filesystem/operations.hpp>
+#include <filesystem>
 
 #include "../include/Monitoring/MonitoringFactory.h"
 
@@ -52,7 +51,7 @@ BOOST_AUTO_TEST_CASE(Noop)
 #ifdef O2_MONITORING_WITH_APPMON
 BOOST_AUTO_TEST_CASE(ApMon)
 {
-  boost::filesystem::path configPath = boost::filesystem::canonical(".");
+  std::filesystem::path configPath = std::filesystem::canonical(".");
   auto Monitoring = MonitoringFactory::Get("apmon://" + configPath.string() + "/ApMon.conf");
   monitoring->send({10, "myCrazyMetric"});
 }