Make strong_ptr transitive dependency

Author: kammce

CMakeLists.txt

@@ -17,6 +17,8 @@ cmake_minimum_required(VERSION 3.28)
 # Generate compile commands for anyone using our libraries.
 set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
 set(CMAKE_COLOR_DIAGNOSTICS ON)
+set(CMAKE_EXPERIMENTAL_EXPORT_PACKAGE_DEPENDENCIES
+    "1942b4fa-b2c5-4546-9385-83f254070067")
 
 project(async_context LANGUAGES CXX)
 
@@ -48,10 +50,10 @@ install(
 
 install(
     EXPORT async_context_targets
-    FILE "libasync_context-config.cmake"
-    NAMESPACE libasync_context::
+    FILE "async_context-config.cmake"
     DESTINATION "lib/cmake"
     CXX_MODULES_DIRECTORY "cxx-modules"
+    EXPORT_PACKAGE_DEPENDENCIES
 )
 
 # ==============================================================================

conanfile.py

@@ -33,7 +33,7 @@ class async_context_conan(ConanFile):
     description = ("Implementation of C++20 coroutines targeting embedded system by eliminating the usage of the global heap and providing a 'context' which contains a coroutine stack frame and other useful utilities for scheduling.")
     topics = ("async", "coroutines", "stack", "scheduling", "scheduler")
     settings = "compiler", "build_type", "os", "arch"
-    exports_sources = "modules/*", "tests/*", "CMakeLists.txt", "LICENSE"
+    exports_sources = "modules/*", "tests/*", "CMakeLists.txt", "*.cmake.in", "LICENSE"
     package_type = "static-library"
     shared = False
 
@@ -89,7 +89,8 @@ def build_requirements(self):
         self.test_requires("boost-ext-ut/2.3.1")
 
     def requirements(self):
-        self.requires("strong_ptr/0.0.0")
+        self.requires("strong_ptr/0.0.0", transitive_libs=True,
+                      transitive_headers=True)
 
     def layout(self):
         cmake_layout(self)

modules/async_context.cppm

@@ -599,7 +599,7 @@ public:
 
   constexpr void resume() const
   {
-    auto active = handle().promise().context().active_handle();
+    auto active = handle().promise().get_context().active_handle();
     active.resume();
   }
 

test_package/.clangd

@@ -0,0 +1,3 @@
+CompileFlags:
+  CompilationDatabase: .
+  BuiltinHeaders: QueryDriver

test_package/CMakeLists.txt

@@ -0,0 +1,45 @@
+# Copyright 2024 - 2025 Khalil Estell and the libhal contributors
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+cmake_minimum_required(VERSION 4.0)
+
+# Generate compile commands for anyone using our libraries.
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+set(CMAKE_COLOR_DIAGNOSTICS ON)
+
+project(test_package LANGUAGES CXX)
+
+# Require Ninja or Visual Studio for modules
+if(NOT CMAKE_GENERATOR MATCHES "Ninja|Visual Studio")
+    message(FATAL_ERROR "C++20 modules require Ninja or Visual Studio generator")
+endif()
+
+find_package(async_context REQUIRED CONFIG)
+
+add_executable(${PROJECT_NAME})
+target_sources(${PROJECT_NAME} PUBLIC
+    FILE_SET CXX_MODULES
+    TYPE CXX_MODULES
+    PRIVATE main.cpp
+)
+target_include_directories(${PROJECT_NAME} PUBLIC .)
+target_compile_features(${PROJECT_NAME} PRIVATE cxx_std_23)
+target_link_libraries(${PROJECT_NAME} PRIVATE async_context)
+
+# Always run this custom target by making it depend on ALL
+add_custom_target(copy_compile_commands ALL
+    COMMAND ${CMAKE_COMMAND} -E copy_if_different
+    ${CMAKE_BINARY_DIR}/compile_commands.json
+    ${CMAKE_SOURCE_DIR}/compile_commands.json
+    DEPENDS ${CMAKE_BINARY_DIR}/compile_commands.json)

test_package/conanfile.py

@@ -0,0 +1,51 @@
+#!/usr/bin/python
+#
+# Copyright 2024 - 2025 Khalil Estell and the libhal contributors
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+from conan import ConanFile
+from conan.tools.build import cross_building
+from conan.tools.cmake import CMake, cmake_layout, CMakeToolchain, CMakeDeps
+from pathlib import Path
+
+
+class TestPackageConan(ConanFile):
+    settings = "os", "arch", "compiler", "build_type"
+    generators = "VirtualRunEnv"
+
+    def build_requirements(self):
+        self.tool_requires("cmake-modules-toolchain/1.0.2")
+
+    def requirements(self):
+        self.requires(self.tested_reference_str)
+
+    def layout(self):
+        cmake_layout(self)
+
+    def generate(self):
+        tc = CMakeToolchain(self)
+        tc.generate()
+
+        deps = CMakeDeps(self)
+        deps.generate()
+
+    def build(self):
+        cmake = CMake(self)
+        cmake.configure()
+        cmake.build()
+
+    def test(self):
+        if not cross_building(self):
+            bin_path = Path(self.cpp.build.bindirs[0]) / "test_package"
+            self.run(bin_path.absolute(), env="conanrun")

test_package/main.cpp

@@ -0,0 +1,78 @@
+// Copyright 2024 - 2025 Khalil Estell and the libhal contributors
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//      http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+#include <cassert>
+
+#include <array>
+#include <chrono>
+#include <coroutine>
+#include <memory_resource>
+#include <span>
+#include <variant>
+
+import async_context;
+
+struct my_scheduler : public async::scheduler
+{
+  int sleep_count = 0;
+
+private:
+  void do_schedule(
+    [[maybe_unused]] async::context& p_context,
+    [[maybe_unused]] async::blocked_by p_block_state,
+    [[maybe_unused]] std::variant<async::sleep_duration, async::context*>
+      p_block_info) override
+  {
+    if (std::holds_alternative<async::sleep_duration>(p_block_info)) {
+      sleep_count++;
+    }
+  }
+};
+
+async::future<void> coro_double_delay(async::context&)
+{
+  using namespace std::chrono_literals;
+  co_await 100ns;
+  co_await 100ns;
+  co_return;
+}
+int main()
+{
+  auto scheduler =
+    mem::make_strong_ptr<my_scheduler>(std::pmr::new_delete_resource());
+  auto buffer = mem::make_strong_ptr<std::array<async::u8, 1024>>(
+    std::pmr::new_delete_resource());
+  auto buffer_span = mem::make_strong_ptr<std::span<async::u8>>(
+    std::pmr::new_delete_resource(), *buffer);
+  async::context my_context(scheduler, buffer_span);
+
+  auto future_delay = coro_double_delay(my_context);
+
+  assert(not future_delay.done());
+
+  future_delay.resume();
+
+  assert(scheduler->sleep_count == 1);
+
+  future_delay.resume();
+
+  assert(scheduler->sleep_count == 2);
+  assert(not scheduler->done());
+
+  future_delay.resume();
+
+  assert(scheduler->done());
+
+  return 0;
+}