* Daniele E. Domenichelli <[email protected]> [2016-08-11 11:24]: > cmake_minimum_required(VERSION 2.8.9) > project(foo) > > find_package(YARP REQUIRED) > find_package(std_msgs REQUIRED) > > yarp_generate_bindings(GENERATED_SRCS > "${std_msgs_DATAROOTDIR}/std_msgs/msg/String.msg" > "${std_msgs_DATAROOTDIR}/std_msgs/msg/ColorRGBA.msg)
Where is ${std_msgs_DATAROOTDIR} defined?
> $CATKIN_GLOBAL_SHARE_DESTINATION will require catkin therefore I
> cannot use it.
>
> $CMAKE_INSTALL_DATAROOTDIR is generated by GNUInstallDirs.cmake and
> it is by default "share" (relative) this means that if ROS packages are
> not installed in "/usr" (for example because they are installed from
> the ROS repository in "/opt") or if CMAKE_INSTALL_PREFIX of the module
> is not set to "/usr" (the default value is "/usr/local", therefore it
> will have to be set manually), the messages will not be found.
I see your point, but I'm nut sure about the right solution here. My
thinking is:
- If you want msg from the default path, $CMAKE_INSTALL_DATAROOTDIR
should work (modulo the CMAKE_INSTALL_PREFIX).
- If you want them from /usr/local, you should set it manually.
- If you want it from the ROS packages in /opt, I would propose to
include catkin and use $CATKIN_GLOBAL_SHARE_DESTINATION, as there
could be multiple versions in /opt and sourcing setup.sh/catkin should
give you the right one.
So maybe a cmake script would be useful after all :).
> > Maybe we will need some more dependencies
> > (or recommends) on this package, like for rosmsg for example and we
> > should add it to the ros-core-dev meta package. What do you think?
>
> I agree.
Could you go ahead and add it?
> Perhaps also the libstd-msgs-dev package should recommend it?
Not sure here, does it change the usability of libstd-msgs-dev somehow
(didn't test)?
> Perhaps should we add an extra meta package ros-core-msgs?
Though about it as well, do you have a use case for it? Otherwise I
wouldn't do it.
> Should I go on and do the same for the other message packages?
Yes please :). Once we are fine with it, I will upload the new packages
(note that we have to go through the new queue).
Cheers Jochen
signature.asc
Description: PGP signature
