search

maliput / maliput_malidrive

Open-source ready OpenDrive backend for Maliput
BSD 3-Clause "New" or "Revised" License
1 stars 2 forks source link

Replaces AMENT_CURRENT_PREFIX by COLCON_PREFIX_PATH #141

Closed agalbachicar closed 3 years ago

agalbachicar commented 3 years ago

AMENT_CURRENT_PREFIX points to /opt/ros/foxy in foxy whereas it points to the full workspace/install/package_name in dashing. COLCON_PREFIX_PATH remains to point to the full workspace/install path

Part of ToyotaResearchInstitute/maliput_infrastructure#196 -- see this comment

stonier commented 3 years ago

Is there any documentation to definition of AMENT_CURRENT_PREFIX and why it changed? I couldn't quickly find anything on a google search.

agalbachicar commented 3 years ago

I spent a considerable amount of time trying to find it and could not find any.

I did ask to colleagues and they pointed me to ament_index for this usage. I think we can do it later on after this is solved in the workspace.

agalbachicar commented 3 years ago

I merged in this PR to continue with the task. If there is any better approach I am happy to revert it and implement the right one.