-
Notifications
You must be signed in to change notification settings - Fork 27
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
Colcon build failed if a package set the requirement of ament_index_cpp #86
Comments
(Microsoft) I'm working on this. I'm staging a fix in https://github.com/ms-iot/ament_index/tree/humble, and will PR when I can verify it. This repros when compiling resource_retriever. |
@ooeygui Thanks for this information. Correct. I've got the first error with resource_retriever. I've tried to skip this package, then I've noticed that almost every package with ament_index_cpp got the same error. |
@tomcattiger1230 Thanks for confirming. I'm root causing it. |
The CMake which ships with Visual Studio is later. Cmake recently changed _expectedTarget to _cmake_expected_target. In # extract the target names associated with the export
set(_regex "foreach\\(_expectedTarget (.+)\\)") However, the cmake files themselves reference foreach(_cmake_expected_target IN ITEMS ament_index_cpp::ament_index_cpp)
list(APPEND _cmake_expected_targets "${_cmake_expected_target}")
if(TARGET "${_cmake_expected_target}")
list(APPEND _cmake_targets_defined "${_cmake_expected_target}")
else()
list(APPEND _cmake_targets_not_defined "${_cmake_expected_target}")
endif()
endforeach() And here's the change: |
is there any suggestion to correct this? Thanks. |
There is a fix in progress - ament/ament_cmake#395 |
Actually, I'm going to close this one anyway since this is a duplicate of ament/ament_cmake#396 |
-- Found ament_index_cpp: 1.4.0 (/Users/xxx/Develop/ros2/ros2_humble/install/share/ament_index_cpp/cmake)
CMake Error at /Users/xxx/Develop/ros2/ros2_humble/install/share/ament_index_cpp/cmake/ament_cmake_export_targets-extras.cmake:18 (message):
Failed to find exported target names in
'/Users/xxx/Develop/ros2/ros2_humble/install/share/ament_index_cpp/cmake/export_ament_index_cppExport.cmake'
Call Stack (most recent call first):
/Users/xxx/Develop/ros2/ros2_humble/install/share/ament_index_cpp/cmake/ament_index_cppConfig.cmake:41 (include)
CMakeLists.txt:14 (find_package)
The text was updated successfully, but these errors were encountered: