Skip to content

Use std::filesystem in find_library and add more test#221

Open
ahcorde wants to merge 2 commits intorollingfrom
ahcorde/rolling/filesystem_find_library
Open

Use std::filesystem in find_library and add more test#221
ahcorde wants to merge 2 commits intorollingfrom
ahcorde/rolling/filesystem_find_library

Conversation

@ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Mar 9, 2026

Description

Use std::filesystem instead of strings and add more test

Did you use Generative AI?

Claude Sonnet 4.6

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde self-assigned this Mar 9, 2026
@ahcorde
Copy link
Contributor Author

ahcorde commented Mar 9, 2026

Pulls: #221
Gist: https://gist.githubusercontent.com/ahcorde/fb3d63720ce3bc41c4a64135ad3f6c41/raw/5db531a2270a47370f5accdd410a6ad0436eeb8d/ros2.repos
BUILD args: --packages-above-and-dependencies rcpputils
TEST args: --packages-above rcpputils
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/18415

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>

Fix windows

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde force-pushed the ahcorde/rolling/filesystem_find_library branch from c0f5878 to 2f037d8 Compare March 10, 2026 22:47
@ahcorde
Copy link
Contributor Author

ahcorde commented Mar 10, 2026

Pulls: #221
Gist: https://gist.githubusercontent.com/ahcorde/0af1d33112197cc61cd9278acdf9a7b9/raw/5db531a2270a47370f5accdd410a6ad0436eeb8d/ros2.repos
BUILD args: --packages-above-and-dependencies rcpputils
TEST args: --packages-above rcpputils
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/18429

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

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