search

osrf / ros_dds

(Deprecated) Prototype system using DDS as the middleware for a ROS like API.
Apache License 2.0
42 stars 16 forks source link

Add local devel path to idlpp include. #20

Closed gerkey closed 10 years ago

gerkey commented 10 years ago

This change is needed in the case when you're building in a workspace that is separate from the one in which genidlcpp was built.

@dirk-thomas

dirk-thomas commented 10 years ago

+1 The current implementation has several flaws regarding exporting the correct information for a DDS message package, chaining of workspaces etc. We can merge this fix but I would recommend to only build everything in a single workspace and not spend time on supporting the full set of cases possible with catkin.

wjwwood commented 10 years ago

+1

gerkey commented 10 years ago

It's a fair point about not supporting all possible configurations, but I think that this one is worth addressing, as I expect that it'll be pretty common. I ran into because I built and installed ros_dds/prototype, then built ros_tutorials/turtlesim against that installation. I would do the same to build other packages against ros_dds/prototype.

dirk-thomas commented 10 years ago

Please be aware that the installation target including exporting information to the CMake config file has not been tested and is likely missing information somewhere in the chain.