feat: Replace outcome with Boost's version of outcome to remove dependence on quickcpplib.

CMakeLists.txt

@@ -39,6 +39,11 @@ set(CMAKE_EXPORT_COMPILE_COMMANDS
     FORCE
 )
 
+find_package(Boost REQUIRED)
+if(Boost_FOUND)
+    message(STATUS "Found Boost ${Boost_VERSION}.")
+endif()
+
 if(YSTDLIB_CPP_IS_TOP_LEVEL)
     # Include dependency settings if the project isn't being included as a subproject.
     # NOTE: We mark the file optional because if the user happens to have the dependencies
@@ -53,13 +58,6 @@ if(BUILD_TESTING AND YSTDLIB_CPP_BUILD_TESTING)
     set(YSTDLIB_CPP_ENABLE_TESTS ON)
 endif()
 
-find_package(outcome REQUIRED)
-if(outcome_FOUND)
-    message(STATUS "Found outcome.")
-else()
-    message(FATAL_ERROR "Could not find libraries for outcome.")
-endif()
-
 if(YSTDLIB_CPP_ENABLE_TESTS)
     find_package(Catch2 3.8.0 REQUIRED)
     if(Catch2_FOUND)

src/ystdlib/error_handling/CMakeLists.txt

@@ -7,7 +7,7 @@ cpp_library(
         Result.hpp
         utils.hpp
     PUBLIC_LINK_LIBRARIES
-        outcome::hl
+        Boost::headers
     TESTS_SOURCES
         test/constants.hpp
         test/test_ErrorCode.cpp

src/ystdlib/error_handling/Result.hpp

@@ -3,10 +3,10 @@
 
 #include <system_error>
 
-#include <outcome/config.hpp>
-#include <outcome/std_result.hpp>
-#include <outcome/success_failure.hpp>
-#include <outcome/try.hpp>
+#include <boost/outcome/config.hpp>
+#include <boost/outcome/std_result.hpp>
+#include <boost/outcome/success_failure.hpp>
+#include <boost/outcome/try.hpp>
 
 namespace ystdlib::error_handling {
 /**
@@ -20,34 +20,16 @@ namespace ystdlib::error_handling {
  * @tparam ErrorType The type used to represent errors.
  */
 template <typename ReturnType, typename ErrorType = std::error_code>
-using Result = OUTCOME_V2_NAMESPACE::std_result<ReturnType, ErrorType>;
+using Result = BOOST_OUTCOME_V2_NAMESPACE::std_result<ReturnType, ErrorType>;
 
 /**
  * @return A value indicating successful completion of a function that returns a void result (i.e.,
  * `Result<void, E>`).
  */
-[[nodiscard]] inline auto success() -> OUTCOME_V2_NAMESPACE::success_type<void> {
-    return OUTCOME_V2_NAMESPACE::success();
+[[nodiscard]] inline auto success() -> BOOST_OUTCOME_V2_NAMESPACE::success_type<void> {
+    return BOOST_OUTCOME_V2_NAMESPACE::success();
 }
 
-/**
- * A function-style macro that emulates Rust’s try (`?`) operator for error propagation.
- *
- * @param expr An expression that evaluates to a `Result` object.
- *
- * Behavior:
- * - If `expr` represents an error (i.e., `expr.has_error()` returns true), the macro performs an
- *   early return from the enclosing function with the contained error.
- * - Otherwise, it unwraps and yields the successful value as an rvalue reference (`expr.value()`).
- *
- * NOTE: This macro is only supported on GCC and Clang due to reliance on compiler-specific
- * extensions.
- */
-#ifdef OUTCOME_TRYX
-    // NOLINTNEXTLINE(cppcoreguidelines-macro-usage)
-    #define YSTDLIB_ERROR_HANDLING_TRYX(expr) OUTCOME_TRYX(expr)
-#endif
-
 /**
  * A function-style macro for propagating errors from expressions that evaluate to a void result
  * (`Result<void, E>`).
@@ -60,7 +42,7 @@ using Result = OUTCOME_V2_NAMESPACE::std_result<ReturnType, ErrorType>;
  * - Otherwise, execution continues normally.
  */
 // NOLINTNEXTLINE(cppcoreguidelines-macro-usage)
-#define YSTDLIB_ERROR_HANDLING_TRYV(expr) OUTCOME_TRYV(expr)
+#define YSTDLIB_ERROR_HANDLING_TRY(expr) BOOST_OUTCOME_TRY(expr)
 }  // namespace ystdlib::error_handling
 
 #endif  // YSTDLIB_ERROR_HANDLING_RESULT_HPP

src/ystdlib/error_handling/test/test_Result.cpp

@@ -50,7 +50,7 @@ TEST_CASE("test_result_void", "[error_handling][Result]") {
 
 TEST_CASE("test_result_void_in_main", "[error_handling][Result]") {
     auto main_func = [&](bool is_error) -> Result<void> {
-        YSTDLIB_ERROR_HANDLING_TRYV(cVoidFunc(is_error));
+        YSTDLIB_ERROR_HANDLING_TRY(cVoidFunc(is_error));
         return success();
     };
     auto const main_no_error{main_func(false)};
@@ -74,7 +74,7 @@ TEST_CASE("test_result_int", "[error_handling][Result]") {
 
 TEST_CASE("test_result_int_in_main", "[error_handling][Result]") {
     auto main_func = [&](bool is_error) -> Result<void> {
-        YSTDLIB_ERROR_HANDLING_TRYV(cIntFunc(is_error));
+        YSTDLIB_ERROR_HANDLING_TRY(cIntFunc(is_error));
         return success();
     };
     auto const main_no_error{main_func(false)};
@@ -88,7 +88,8 @@ TEST_CASE("test_result_int_in_main", "[error_handling][Result]") {
 
 TEST_CASE("test_result_int_propagate", "[error_handling][Result]") {
     auto main_func = [&](bool is_error) -> Result<int> {
-        return YSTDLIB_ERROR_HANDLING_TRYX(cIntFunc(is_error));
+        YSTDLIB_ERROR_HANDLING_TRY(cIntFunc(is_error));
+        return success();
     };
     auto const main_no_error{main_func(false)};
     REQUIRE_FALSE(main_no_error.has_error());
@@ -112,7 +113,7 @@ TEST_CASE("test_result_unique_ptr", "[error_handling][Result]") {
 
 TEST_CASE("test_result_unique_ptr_in_main", "[error_handling][Result]") {
     auto main_func = [&](bool is_error) -> Result<void> {
-        YSTDLIB_ERROR_HANDLING_TRYV(cUniquePtrFunc(is_error));
+        YSTDLIB_ERROR_HANDLING_TRY(cUniquePtrFunc(is_error));
         return success();
     };
     auto const main_no_error{main_func(false)};
@@ -126,7 +127,8 @@ TEST_CASE("test_result_unique_ptr_in_main", "[error_handling][Result]") {
 
 TEST_CASE("test_result_unique_ptr_propagate", "[error_handling][Result]") {
     auto main_func = [&](bool is_error) -> Result<std::unique_ptr<int>> {
-        return YSTDLIB_ERROR_HANDLING_TRYX(cUniquePtrFunc(is_error));
+        YSTDLIB_ERROR_HANDLING_TRY(cUniquePtrFunc(is_error));
+        return success();
     };
     auto const main_no_error{main_func(false)};
     REQUIRE_FALSE(main_no_error.has_error());

taskfile.yaml

@@ -4,7 +4,7 @@ includes:
   build: "./taskfiles/build.yaml"
   deps: "./taskfiles/deps.yaml"
   lint: "./taskfiles/lint.yaml"
-  utils: "tools/yscope-dev-utils/taskfiles/utils.yaml"
+  utils: "tools/yscope-dev-utils/exports/taskfiles/utils/utils.yaml"
 
 vars:
   G_BUILD_DIR: "{{.ROOT_DIR}}/build"

taskfiles/build.yaml

@@ -10,7 +10,7 @@ tasks:
     deps:
       - "init"
     cmds:
-      - task: ":utils:cmake-build"
+      - task: ":utils:cmake:build"
         vars:
           BUILD_DIR: "{{.G_BUILD_DIR}}"
 
@@ -21,7 +21,7 @@ tasks:
     deps:
       - "init"
     cmds:
-      - task: ":utils:cmake-build"
+      - task: ":utils:cmake:build"
         vars:
           BUILD_DIR: "{{.G_BUILD_DIR}}"
           TARGETS:
@@ -49,7 +49,7 @@ tasks:
   clean:
     desc: "Removes all built artifacts."
     deps:
-      - task: ":utils:cmake-clean"
+      - task: ":utils:cmake:clean"
         vars:
           BUILD_DIR: "{{.G_BUILD_DIR}}"
 
@@ -59,7 +59,7 @@ tasks:
       - ":deps:install-all"
     run: "once"
     cmds:
-      - task: ":utils:cmake-generate"
+      - task: ":utils:cmake:generate"
         vars:
           BUILD_DIR: "{{.G_BUILD_DIR}}"
           SOURCE_DIR: "{{.ROOT_DIR}}"