Skip to content

Commit 13d4006

Browse files
nbbrooksclaude
andcommitted
Remove redundant Qt discovery in moveit_setup_framework
The rviz2-version-based Qt selection already finds Qt before any rviz or moveit dependency, so the second find_package(Qt6) block was redundant and could pull in Qt6 even on the Qt5 path. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent b7f4d35 commit 13d4006

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

moveit_setup_assistant/moveit_setup_framework/CMakeLists.txt

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,6 @@ else()
2323
endif()
2424
find_package(moveit_core REQUIRED)
2525
find_package(moveit_ros_planning REQUIRED)
26-
# Find Qt before packages that transitively depend on it (rviz, visualization)
27-
find_package(Qt6 QUIET COMPONENTS Core Widgets)
28-
if(NOT Qt6_FOUND)
29-
find_package(Qt5Core REQUIRED)
30-
find_package(Qt5Widgets REQUIRED)
31-
endif()
3226
find_package(moveit_ros_visualization REQUIRED)
3327
find_package(pluginlib REQUIRED)
3428
find_package(rclcpp REQUIRED)

0 commit comments

Comments
 (0)