-
Notifications
You must be signed in to change notification settings - Fork 125
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
conversion functions from .msg / .srv to .idl #324
Conversation
5af4df6
to
ae4d119
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the only thing that was unexpected is that the scripts had to be run from the package folder.
# Fails
src/ros2/common_interfaces $ ros2 run rosidl_adapter srv2idl.py ./sensor_msgs/srv/SetCameraInfo.srv
# Succeeds
ros2/common_interfaces/sensor_msgs $ ros2 run rosidl_adapter srv2idl.py ./srv/SetCameraInfo.srv
A short doc or more informative error would be helpful in this case.
That is weird. It worked for me with arbitrary paths independent of the cwd. Can you please share your output when it fails. |
|
ae4d119
to
672b7b0
Compare
I was able to reproduce the problem. It should be fixed in 9d2cd71. |
Please retry with the current state
This is the third PR integrating #298 step-by-step.
Builds on top of #323.
Since the functionality provided by this patch isn't actively being used yet there isn't a point in running CI.
The command line script
msg2idl.py
andsrv2idl.py
can be used to manually convert.msg
/.srv
files.