From 488a4c31de746fc4f8e3d230c57cf41e6be3b157 Mon Sep 17 00:00:00 2001 From: Ezra Brooks Date: Thu, 5 Nov 2026 09:34:36 -0700 Subject: [PATCH] lyrical: plotjuggler needs python in host plotjuggler embeds the Python interpreter (plotjuggler_app links Python3::Python for its Python custom transforms) without declaring a Python dependency in package.xml. Today every lyrical package has python in host anyway, so this changes nothing yet; it keeps plotjuggler built per Python once packages without Python content stop getting python (vinca's conditional_python_dependency). --- distros/lyrical/patch/dependencies.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/distros/lyrical/patch/dependencies.yaml b/distros/lyrical/patch/dependencies.yaml index e7e37c3f5..a359cbe30 100644 --- a/distros/lyrical/patch/dependencies.yaml +++ b/distros/lyrical/patch/dependencies.yaml @@ -120,7 +120,8 @@ plotjuggler: # PlotJuggler 3.17 is Qt5-only (find_package(Qt5 ...)); the Qt rosdep keys map to Qt6 here remove_host: ["qt6-main"] remove_run: ["qt6-main"] - add_host: ["libxcb", "${{ 'elfutils' if linux }}", "ros2-ros-workspace", "qt-main"] + # python: embeds the interpreter (links libpython) without declaring it + add_host: ["libxcb", "${{ 'elfutils' if linux }}", "ros2-ros-workspace", "qt-main", "python"] add_run: ["qt-main"] plotjuggler_ros: # Qt5-only like plotjuggler, which it links