eigen-conversions: also depend on cmake-modules
Without the dependency on cmake-modules, `bitbake eigen-conversions` can possibly fail with: ``` | CMake Error at /home/lukas/dev/openembedded.org/openembedded-core/build/tmp-glibc/work/i586-oe-linux/eigen-conversions/1.11.8-r0/recipe-sysroot-native/opt/ros/indigo/share/catkin/cmake/catkinConfig.cmake:83 (find_package): | Could not find a package configuration file provided by "cmake_modules" | with any of the following names: | | cmake_modulesConfig.cmake | cmake_modules-config.cmake | | Add the installation prefix of "cmake_modules" to CMAKE_PREFIX_PATH or set | "cmake_modules_DIR" to a directory containing one of the above files. If | "cmake_modules" provides a separate development package or SDK, be sure it | has been installed. | Call Stack (most recent call first): | CMakeLists.txt:5 (find_package) | | | -- Configuring incomplete, errors occurred! ``` The failure only occurs if cmake-modules has not been installed before eigen-conversions is configured. Hence, the regular regression testing with `bitbake core-image-ros-world`, which builds many packages in parallel, did not uncover this because make-modules was usually installed before eigen-conversions was configured. However, the issue is clearly reproducible with `bitbake eigen-conversions cmake-modules -c cleanall && bitbake eigen-conversions` The missing dependency was probably introduced by the automatic recipe updates without checking for new dependencies. Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
This commit is contained in:
parent
30d5e698b8
commit
a14c2138f6
|
@ -3,6 +3,6 @@ SECTION = "devel"
|
||||||
LICENSE = "BSD"
|
LICENSE = "BSD"
|
||||||
LIC_FILES_CHKSUM = "file://package.xml;beginline=15;endline=15;md5=d566ef916e9dedc494f5f793a6690ba5"
|
LIC_FILES_CHKSUM = "file://package.xml;beginline=15;endline=15;md5=d566ef916e9dedc494f5f793a6690ba5"
|
||||||
|
|
||||||
DEPENDS = "geometry-msgs libeigen orocos-kdl std-msgs"
|
DEPENDS = "cmake-modules geometry-msgs libeigen orocos-kdl std-msgs"
|
||||||
|
|
||||||
require geometry.inc
|
require geometry.inc
|
||||||
|
|
Loading…
Reference in New Issue