Fix rolling CI: deprecated subscription callback and -fcoroutines

- Use std::shared_ptr<const MsgT> for subscription callbacks (rolling
  deprecated the non-const SharedPtr overload)
- Only pass -fcoroutines for GCC; Clang enables coroutines by default
  and warns on the unused flag during clang-tidy

Author: otamachan

rclcpp_async/CMakeLists.txt

@@ -8,7 +8,10 @@ if(NOT CMAKE_CXX_STANDARD)
 endif()
 
 if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
-  add_compile_options(-Wall -Wextra -Wpedantic -Werror=deprecated-declarations -fcoroutines)
+  add_compile_options(-Wall -Wextra -Wpedantic -Werror=deprecated-declarations)
+endif()
+if(CMAKE_COMPILER_IS_GNUCXX)
+  add_compile_options(-fcoroutines)
 endif()
 
 find_package(ament_cmake REQUIRED)

rclcpp_async/include/rclcpp_async/co_context.hpp

@@ -186,7 +186,7 @@ class CoContext
   {
     auto stream = std::make_shared<TopicStream<MsgT>>(*this);
     stream->sub_ = node_.template create_subscription<MsgT>(
-      topic, qos, [s = stream](typename MsgT::SharedPtr msg) {
+      topic, qos, [s = stream](std::shared_ptr<const MsgT> msg) {
         if (s->closed_) {
           return;
         }

rclcpp_async/include/rclcpp_async/topic_stream.hpp

@@ -37,7 +37,7 @@ template <typename MsgT>
 class TopicStream
 {
   CoContext & ctx_;
-  std::queue<typename MsgT::SharedPtr> queue_;
+  std::queue<std::shared_ptr<const MsgT>> queue_;
   std::coroutine_handle<> waiter_;
   typename rclcpp::Subscription<MsgT>::SharedPtr sub_;
   bool closed_ = false;
@@ -49,7 +49,7 @@ class TopicStream
 
   struct NextAwaiter
   {
-    using SharedPtr = typename MsgT::SharedPtr;
+    using SharedPtr = std::shared_ptr<const MsgT>;
 
     TopicStream & stream;
     std::stop_token token;

rclcpp_async_example/CMakeLists.txt

@@ -8,7 +8,10 @@ if(NOT CMAKE_CXX_STANDARD)
 endif()
 
 if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
-  add_compile_options(-Wall -Wextra -Wpedantic -Werror=deprecated-declarations -fcoroutines)
+  add_compile_options(-Wall -Wextra -Wpedantic -Werror=deprecated-declarations)
+endif()
+if(CMAKE_COMPILER_IS_GNUCXX)
+  add_compile_options(-fcoroutines)
 endif()
 
 find_package(ament_cmake REQUIRED)