search

jhu-lcsr-forks / rtt_ros_integration

Orocos-ROS integration libraries and tools
1 stars 2 forks source link

rtt_roscomm: Boost header generation broken with genmsg 0.4.22 #16

Closed meyerj closed 10 years ago

meyerj commented 10 years ago

ros/genmsg#32 has broken typekit generation as ${${package}_MESSAGE_FILES} variable contains relative paths now. The current Ubuntu repositories serve genmsg 0.4.22 including that patch since today.

We have to check if this can be considered a bug in genmsg or if there is a simple way to get the absolute path in a generic way.

Possible solution (not tested yet):

if(NOT IS_ABSOLUTE "${FILE}")
  set(FILE "${${package}_DIR}/${FILE}")
endif()
jbohren commented 10 years ago

Blah. Yeah, this might be an acceptable solution. It definitely seems easier than having to go through the process of changing genmsg.

meyerj commented 10 years ago

I guess it's not a bug. There have been huge efforts to ban every occurence of absolute paths from the install-space in order to make it relocatable: https://github.com/ros/catkin/issues/490.

jbohren commented 10 years ago

I guess it's not a bug. There have been huge efforts to ban every occurence of absolute paths from the install-space in order to make it relocatable: ros/catkin#490.

Yeah, that's what it seems like. At least this shouldn't be too hard to fix. Meanwhile, did you have a chance to look at #5 ?