Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AMENT_PREFIX_PATH uses a semicolon instead of a colon #1104

Open
jdiez17 opened this issue Mar 5, 2024 · 1 comment
Open

AMENT_PREFIX_PATH uses a semicolon instead of a colon #1104

jdiez17 opened this issue Mar 5, 2024 · 1 comment
Assignees

Comments

@jdiez17
Copy link
Contributor

jdiez17 commented Mar 5, 2024

Describe the bug
The two directories in AMENT_PREFIX_PATH set in ros_ament_cmake should be separated by a semicolon, not a colon.

I noticed this during my work on building workspaces using colcon. I had this error from ament_cmake:

| [5.673s] colcon.colcon_ros.prefix_path.ament WARNING The path '/build/tmp/work/cortexa57-poky-linux/ros-rust-example/1.0+gitAUTOINC+3e1126a711-r0/recipe-sysroot/usr;/build/t
mp/work/cortexa57-poky-linux/ros-rust-example/1.0+gitAUTOINC+3e1126a711-r0/recipe-sysroot-native/usr' in the environment variable AMENT_PREFIX_PATH doesn't exist 
@robwoolley
Copy link
Collaborator

Thanks @jdiez17 for reporting this.

For future reference, we discovered that references in the code indicate that AMENT_PREFIX_PATH use the operating systems path delimiter, whereas CMAKE_PREFIX_PATH is meant to accept lists which in CMake are stored as strings using semicolon as the delimiter.

References:

@robwoolley robwoolley self-assigned this Apr 13, 2024
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

No branches or pull requests

2 participants