Delete ament_export_include_directories() and ament_export_libraries() calls
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 80/100
- Issue type
- Refactor
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- cmake, cpp
- Domain
- build-system
Research direction
The issue points to the CMakeLists.txt file. Look for calls to ament_export_include_directories() and ament_export_libraries() in that file and delete them, ensuring ament_export_targets() is already present. Verify the changes by building the package to confirm no regressions.
Written by the indexing model from the issue text.
Description
We use ament_export_targets() to export modern CMake targets. Downstream packages should use these modern CMake targets instead of old CMake variables. Because the packages below already call ament_export_targets(), we can delete the ament_export_include_directories() and ament_export_libraries() calls.
Modify the following files and delete the ament_export_include_directories() and ament_export_libraries() calls:
CMakeLists.txt
- Dominant language
- C++
- Stars
- 37
- Forks
- 59
- Avg merge
- 21h 9m
- Merged PRs (30d)
- 1
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ros2/rcpputils
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
ros2/message_filters#338 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
subsurface/subsurface#4984 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
google-ai-edge/LiteRT-LM#3739 ·