From 4295045c02ec431656e0ff6270ca87d77ef0c91b Mon Sep 17 00:00:00 2001
From: Bjar Ne <gleichdick@users.noreply.github.com>
Date: Tue, 3 Aug 2021 14:47:14 +0000
Subject: [PATCH] add note regarding SYSTEM usage

Signed-off-by: Bjar Ne <gleichdick@users.noreply.github.com>
---
 .../cmake/ament_target_dependencies.cmake                     | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/ament_cmake_target_dependencies/cmake/ament_target_dependencies.cmake b/ament_cmake_target_dependencies/cmake/ament_target_dependencies.cmake
index 50bb69c7..0614a5a1 100644
--- a/ament_cmake_target_dependencies/cmake/ament_target_dependencies.cmake
+++ b/ament_cmake_target_dependencies/cmake/ament_target_dependencies.cmake
@@ -31,6 +31,10 @@
 #   with a SYSTEM keyword, followed by an INTERFACE or PUBLIC keyword.
 #   If it starts with a SYSTEM keyword, it will be used in
 #   target_include_directories() calls.
+#   Do not use the SYSTEM keyword for other ROS dependencies
+#   (targets build in a ROS workspace, to be precise),
+#   as build errors could arise when overlaying workspaces.
+#   See `here <https://github.com/ros-planning/geometric_shapes/pull/186#discussion_r623989443>`_ for details.
 #   If it starts (or follows) with an INTERFACE or PUBLIC keyword,
 #   this keyword will be used in the target_*() calls.
 # :type ARGN: list of strings