Remove ament_export_include_directories and ament_export_libraries calls

[fujitatomoya]

These packages already call ament_export_targets(), so the old-style CMake variable exports are redundant.

rclcpp_action and rclcpp_lifecycle relied on the old-style rclcpp_INCLUDE_DIRS variable to give cppcheck include hints. Take the include directories from the rclcpp::rclcpp target instead so the hint keeps working.

Fixes #3285

Description

Fixes # (issue)

Is this user-facing behavior change?

No

Did you use Generative AI?

Yes, Claude Fable 5.1

Additional Information

[fujitatomoya]

this would silently become empty after the rclcpp change. now it reads the include directories off the modern target instead.

[github-actions]

ABI Compliance Check

✅ Verdict: compatible

Library	Verdict	Summary
libcomponent_manager.so	✅ compatible	No ABI changes detected.
librclcpp.so	✅ compatible	No ABI changes detected.
librclcpp_action.so	✅ compatible	No ABI changes detected.
librclcpp_lifecycle.so	✅ compatible	No ABI changes detected.

✅ libcomponent_manager.so — full abidiff report

Compared:

• Base: lib-base/libcomponent_manager.so
• Head: lib-pr/libcomponent_manager.so @ 09169cb

(empty report — no differences printed by abidiff)

✅ librclcpp.so — full abidiff report

Compared:

• Base: lib-base/librclcpp.so
• Head: lib-pr/librclcpp.so @ 09169cb

(empty report — no differences printed by abidiff)

✅ librclcpp_action.so — full abidiff report

Compared:

• Base: lib-base/librclcpp_action.so
• Head: lib-pr/librclcpp_action.so @ 09169cb

(empty report — no differences printed by abidiff)

✅ librclcpp_lifecycle.so — full abidiff report

Compared:

• Base: lib-base/librclcpp_lifecycle.so
• Head: lib-pr/librclcpp_lifecycle.so @ 09169cb

(empty report — no differences printed by abidiff)

_{Updated for commit 09169cb · suppressions: /home/runner/work/_temp/ros2-abi-suppressions.txt}

[anshika170-cmyk]

Hi I want to work on this