Use absolute topic name for rosout (#549)

Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
ros2 · Dec 6, 2019 · 024c3ee · 024c3ee
1 parent 73d2c4c
commit 024c3ee
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/rcl/src/rcl/logging_rosout.c b/rcl/src/rcl/logging_rosout.c
@@ -33,7 +33,7 @@ extern "C"
 {
 #endif
 
-#define ROSOUT_TOPIC_NAME "rosout"
+#define ROSOUT_TOPIC_NAME "/rosout"
 
 /* Return RCL_RET_OK from this macro because we won't check throughout rcl if rosout is
  * initialized or not and in the case it's not we want things to continue working.

diff --git a/rcl/test/rcl/test_logging_rosout.cpp b/rcl/test/rcl/test_logging_rosout.cpp
@@ -95,7 +95,7 @@ class TEST_FIXTURE_P_RMW (TestLoggingRosoutFixture)
     // create rosout subscription
     const rosidl_message_type_support_t * ts =
       ROSIDL_GET_MSG_TYPE_SUPPORT(rcl_interfaces, msg, Log);
-    const char * topic = "rosout";
+    const char * topic = "/rosout";
     this->subscription_ptr = new rcl_subscription_t;
     *this->subscription_ptr = rcl_get_zero_initialized_subscription();
     rcl_subscription_options_t subscription_options = rcl_subscription_get_default_options();