fixup readme; rm hack

RobotLocomotion · Apr 18, 2024 · c5d012c · c5d012c
1 parent 454739b
commit c5d012c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/bazel_ros2_rules/ros2/README.md b/bazel_ros2_rules/ros2/README.md
@@ -207,9 +207,12 @@ From the above two examples (at present), the following features are in
     directly, but instead leverage the C-level interfaces, we get into an
     awkward ground of mutually exclusive memory / resource management
     paradigms.
-- Affordances for `ros2 launch`
 
 The other repos, however, have the following that `bazel_ros2_rules` is
 missing:
 
 - Launching from containers (Docker, Apptainer)
+
+Some common features might be:
+
+- Bazel affordances for `ros2 launch`
diff --git a/ros2_example_bazel_installed/ros2_example_apps/roslaunch_eg_nodes/eg_talker.py b/ros2_example_bazel_installed/ros2_example_apps/roslaunch_eg_nodes/eg_talker.py
@@ -37,10 +37,6 @@ def main():
     while rclpy.ok() and not node.is_done():
         executor.spin_once(timeout_sec=1e-3)
 
-    # Avoid odd error:
-    #   cannot use Destroyable because destruction was requested
-    executor._sigint_gc = None
-
 
 if __name__ == "__main__":
     main()