8000 Feat/read ament prefix path by nonanonno · Pull Request #13 · nonanonno/ros2_d · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Feat/read ament prefix path #13

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

Merged
merged 5 commits into from
Nov 27, 2021
Merged

Feat/read ament prefix path #13

merged 5 commits into from
Nov 27, 2021

Conversation

nonanonno
Copy link
Owner

This PR adds message generation from ROS2 packages under AMENT_PREFIX_PATH. To do that, the following additional changes are introduced.

  • Parse default value in IDL file
  • Add feature to find ROS2 packages under AMENT_PREFIX_PATH
  • Update example

Previous implementation cannot parse IDL file that has other annotation
than `verbatim`. To solve this problem, the following changes are
applied.

- More general annotation rule
- Enable parsing number literals in annotation

In addition, to check these features, unittest was changed and
refactored.
Support default value parsing.
ROS2 packages can be found by seeing AMENT_PREFIX_PATH. Thus this commit
adds a feature to refer AMENT_PREFIX_PATH and to find ROS2 message
packages under AMENT_PREFIX_PATH. In addition, generating all message
packages found is supported.
@nonanonno nonanonno self-assigned this Nov 27, 2021
@nonanonno nonanonno merged commit ef38f84 into main Nov 27, 2021
@nonanonno nonanonno deleted the feat/read-ament_prefix_path branch November 27, 2021 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0