Skip to content

Conversation

@bsutherland333
Copy link
Member

An installation path has changed for Jazzy, including it with ament_target_dependencies is necessary to compile: ament/ament_index#84

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a compilation issue with Jazzy by adding the necessary ament_index_cpp dependency to CMakeLists.txt files. This addresses a change in the installation path for Jazzy that requires explicit inclusion of ament_index_cpp as documented in the linked GitHub issue.

  • Added ament_index_cpp dependency to two executables that require it for compilation
  • Reformatted one dependency list for better readability

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
rosplane_gcs/CMakeLists.txt Added ament_index_cpp dependency and reformatted the dependency list for rviz_waypoint_publisher
rosplane/CMakeLists.txt Added ament_index_cpp dependency to the estimator executable

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@bsutherland333 bsutherland333 self-assigned this Sep 16, 2025
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@iandareid iandareid merged commit bb5f9bd into main Sep 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants