Skip to content

Migrate Goby from Boost.Filesystem to C++17 std::filesystem#391

Open
Copilot wants to merge 3 commits into3.0from
copilot/remove-boost-filesystem
Open

Migrate Goby from Boost.Filesystem to C++17 std::filesystem#391
Copilot wants to merge 3 commits into3.0from
copilot/remove-boost-filesystem

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 7, 2026

Now that Goby requires C++17, Boost.Filesystem is redundant and should no longer be part of runtime codepaths or exported dependency requirements. This PR replaces active filesystem usage with std::filesystem and removes the Boost filesystem component from build/package metadata.

  • Code migration to std::filesystem

    • Replaced boost::filesystem includes/types/APIs in:
      • src/apps/acomms/store_server/store_server.cpp
      • src/apps/middleware/goby_tool/unified_log_tool.h
      • src/apps/middleware/log_tool/log_tool.cpp
      • src/apps/zeromq/liaison/liaison.cpp
      • src/middleware/application/configuration_reader.cpp
      • src/moos/goby_moos_app.h
    • Updated path/string handling (.native()/Boost-specific behavior) to std::filesystem equivalents.
    • Removed legacy BOOST_FILESYSTEM_VERSION conditionals that are no longer relevant.
  • Build/export dependency cleanup

    • Removed filesystem from Boost component requirements in:
      • src/CMakeLists.txt
      • src/goby-config.cmake.in
    • Downstream consumers now resolve only the remaining Boost components actually used.
  • Representative change

    // before
    #include <boost/filesystem.hpp>
    boost::filesystem::path p(input);
    if (boost::filesystem::exists(p)) { ... }
    
    // after
    #include <filesystem>
    std::filesystem::path p(input);
    if (std::filesystem::exists(p)) { ... }

Copilot AI and others added 2 commits May 7, 2026 22:13
Agent-Logs-Url: https://github.com/GobySoft/goby3/sessions/c0c252d4-470f-4634-9b59-8487f9927141

Co-authored-by: tsaubergine <732276+tsaubergine@users.noreply.github.com>
Agent-Logs-Url: https://github.com/GobySoft/goby3/sessions/c0c252d4-470f-4634-9b59-8487f9927141

Co-authored-by: tsaubergine <732276+tsaubergine@users.noreply.github.com>
@tsaubergine tsaubergine marked this pull request as ready for review May 7, 2026 22:17
Copilot AI changed the title [WIP] Remove boost::filesystem and use std::filesystem Migrate Goby from Boost.Filesystem to C++17 std::filesystem May 7, 2026
Copilot AI requested a review from tsaubergine May 7, 2026 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use std::filesystem and remove boost::filesystem

2 participants