Skip to content
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

AP_DDS: Enable DDS by default #25649

Open
Ryanf55 opened this issue Nov 28, 2023 · 1 comment · May be fixed by #25651
Open

AP_DDS: Enable DDS by default #25649

Ryanf55 opened this issue Nov 28, 2023 · 1 comment · May be fixed by #25651
Assignees
Labels
Milestone

Comments

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Nov 28, 2023

Feature request

Is your feature request related to a problem? Please describe.

Enable DDS by default on SITL and build server. If you want to develop, you can optionally install the tool.

Describe the solution you'd like

Want AP_DDS_PREGEN directory for the generated files. If microxrceddsgen doesn't exist (at configure time), and if it doens't, then link in the pregen files. If you do generate, it will have to write in the build tree. Change include path based on flag USE_PREGEN.

Need a libraries/Tools/AP_DDS_run_pregen.

Describe alternatives you've considered

Require all devs to install microxrceddsgen.

Platform
[x] All
[ ] AntennaTracker
[ ] Copter
[ ] Plane
[ ] Rover
[ ] Submarine

Additional context

N/A

@Ryanf55 Ryanf55 added ROS For-4.5 Planned for 4.5 release labels Nov 28, 2023
@Ryanf55 Ryanf55 self-assigned this Nov 28, 2023
@Ryanf55 Ryanf55 added this to DDS/ROS2 Nov 28, 2023
@Ryanf55 Ryanf55 moved this to 🔖 Ready in DDS/ROS2 Nov 28, 2023
@Ryanf55 Ryanf55 linked a pull request Nov 28, 2023 that will close this issue
@Ryanf55
Copy link
Collaborator Author

Ryanf55 commented Feb 17, 2024

@Ryanf55 Ryanf55 removed the For-4.5 Planned for 4.5 release label May 1, 2024
@Ryanf55 Ryanf55 added this to the DDS 4.6 milestone May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🔖 Ready
Development

Successfully merging a pull request may close this issue.

1 participant