-
Notifications
You must be signed in to change notification settings - Fork 762
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
Support new Fast-CDR v2.0.0 [19456] #3828
Conversation
ff87f8e
to
5769a3e
Compare
@richiprosima please test this |
47d07ec
to
206c271
Compare
@richiprosima please test this |
1 similar comment
@richiprosima please test this |
@richiprosima please test linux |
@richiprosima please test this |
@richiprosima please test linux |
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.
The top-level CMakeLists.txt should be updated to require fastcdr version above v2
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.
First partial review. Still a lot to review
examples/cpp/dds/ContentFilteredTopicExample/MyCustomFilter.hpp
Outdated
Show resolved
Hide resolved
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.
Review until commit 4804ba68f8c913dec2fe3debada6944b34cabed4
5048f8f
to
1f7f80b
Compare
@MiguelCompany done in 006e71401 |
@richiprosima please test this |
@richiprosima please test this |
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.
Only a couple of comments regarding the TypeObject types...
Also, I think the versions.md should be updated with the API break in MEMBER_INVALID. I have updated the checklist including the N/A points. Check if you agree.
Signed-off-by: Ricardo González Moreno <[email protected]>
2d803c8
to
7d0278c
Compare
@richiprosima please test this |
Signed-off-by: Ricardo González Moreno <[email protected]>
@richiprosima please test this |
1 similar comment
@richiprosima please test this |
@richiprosima please test mac |
3 similar comments
@richiprosima please test mac |
@richiprosima please test mac |
@richiprosima please test mac |
Signed-off-by: Ricardo González Moreno <[email protected]>
@richiprosima please test this |
3 similar comments
@richiprosima please test this |
@richiprosima please test this |
@richiprosima please test this |
6849cee
to
22a5ed4
Compare
Signed-off-by: Ricardo González Moreno <[email protected]>
22a5ed4
to
9c76dcc
Compare
@richiprosima please test this |
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.
Thanks for the last minute changes! In order to improve the release process I think that we should take into account the following in the future:
- The infrastructure should be updated periodically. Maybe this could be done a month before the release date (but never in the same week).
- If a new tool has to be installed in such a short notice, it is important to look into the tool's dependencies and take into account that if some development tool is upgraded, the tool should not be installed until after the release.
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.
LGTM
Description
Contributor Checklist
versions.md
file (if applicable).Related documentation PR: Documentation about Fast-CDR v2.0.0 and new annotations [19458] Fast-DDS-docs#550
Reviewer Checklist