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

[16440] Update types using Fast DDS-Gen v2.3.0 #63

Merged
merged 4 commits into from
Dec 14, 2022

Conversation

JLBuenoLopez
Copy link
Contributor

Previous to every ShapesDemo release, if there has been a new Fast DDS-Gen release, TypeSupports and TypeObjects should be updated.

This PR also moves the autogenerated code to a specific folder to help and ease the maintenance.

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
@JLBuenoLopez
Copy link
Contributor Author

I have checked both the ROS 2 build and the common build in Ubuntu 22.04. It would be nice to check that nothing has been broken in Windows as well building both configurations.

@JLBuenoLopez JLBuenoLopez added this to the 2.9.0 milestone Dec 14, 2022
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
@EduPonz EduPonz merged commit 467099b into master Dec 14, 2022
@EduPonz EduPonz deleted the release/v2.9.0-update-types branch December 14, 2022 11:45
Mario-DL pushed a commit that referenced this pull request Mar 23, 2023
* Refs #16440: add README to IDL folder

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #16440: fix include order

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs: update types using Fast DDS-Gen v2.3.0 and move to specific folder

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Refs #16440. Fix segfault on Windows.

Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>

Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Co-authored-by: Miguel Company <MiguelCompany@eprosima.com>
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.

3 participants