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

Revert "remove exec_depend on behaviortree_cpp_v3" #3336

Merged
merged 1 commit into from
Dec 22, 2022

Conversation

SteveMacenski
Copy link
Member

@SteveMacenski SteveMacenski commented Dec 22, 2022

Reverts #3279 @Aposhian

Causes build failure in build farm https://build.ros2.org/job/Hbin_uJ64__nav2_behaviors__ubuntu_jammy_amd64__binary/22/consoleFull

16:41:26 -- Found nav2_behavior_tree: 1.1.4 (/opt/ros/humble/share/nav2_behavior_tree/cmake)
16:41:27 CMake Error at /opt/ros/humble/share/nav2_behavior_tree/cmake/ament_cmake_export_dependencies-extras.cmake:21 (find_package):
16:41:27   By not providing "Findbehaviortree_cpp_v3.cmake" in CMAKE_MODULE_PATH this
16:41:27   project has asked CMake to find a package configuration file provided by
16:41:27   "behaviortree_cpp_v3", but CMake did not find one.
16:41:27 
16:41:27   Could not find a package configuration file provided by
16:41:27   "behaviortree_cpp_v3" with any of the following names:
16:41:27 
16:41:27     behaviortree_cpp_v3Config.cmake
16:41:27     behaviortree_cpp_v3-config.cmake
16:41:27 
16:41:27   Add the installation prefix of "behaviortree_cpp_v3" to CMAKE_PREFIX_PATH
16:41:27   or set "behaviortree_cpp_v3_DIR" to a directory containing one of the above
16:41:27   files.  If "behaviortree_cpp_v3" provides a separate development package or
16:41:27   SDK, be sure it has been installed.
16:41:27 Call Stack (most recent call first):
16:41:27   /opt/ros/humble/share/nav2_behavior_tree/cmake/nav2_behavior_treeConfig.cmake:41 (include)
16:41:27   CMakeLists.txt:11 (find_package)
16:41:27 
16:41:27 

@SteveMacenski SteveMacenski marked this pull request as ready for review December 22, 2022 19:47
@SteveMacenski SteveMacenski merged commit bd0614b into main Dec 22, 2022
@SteveMacenski SteveMacenski deleted the revert-3279-remove-unnecessary-deps branch December 22, 2022 19:48
SteveMacenski added a commit that referenced this pull request Dec 22, 2022
SteveMacenski added a commit that referenced this pull request Dec 22, 2022
* Revert "remove exec_depend on behaviortree_cpp_v3 (#3279)" (#3336)

This reverts commit 90ae5cc.

* bump to 1.1.5 for release
@Aposhian
Copy link
Contributor

Hm. That's weird. I'll look into this a little more closely after the holidays. But this seems to be a failure of the build farm to install the build_depend.

andrewlycas pushed a commit to StratomInc/navigation2 that referenced this pull request Feb 23, 2023
shrijitsingh99 pushed a commit to moss-ag/navigation2 that referenced this pull request Mar 4, 2023
…on#3337)

* Revert "remove exec_depend on behaviortree_cpp_v3 (ros-navigation#3279)" (ros-navigation#3336)

This reverts commit 90ae5cc.

* bump to 1.1.5 for release
shrijitsingh99 pushed a commit to moss-ag/navigation2 that referenced this pull request Mar 4, 2023
…on#3337)

* Revert "remove exec_depend on behaviortree_cpp_v3 (ros-navigation#3279)" (ros-navigation#3336)

This reverts commit 90ae5cc.

* bump to 1.1.5 for release
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.

2 participants