[16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) by JLBuenoLopez · Pull Request #3078 · eProsima/Fast-DDS

JLBuenoLopez · 2022-11-08T15:23:43Z

Description

Ubuntu Jammy upgrades several dependencies and build tools so some warnings are issued when building in that platform.

GCC 11 is used instead of GCC 9: a non initialized variable warning is issued with the newest compiler.
OpenSSL 3.0 is used instead of OpenSSL 1.1.1. Several deprecation notices are issued when forcing Asio as thirdparty because Fast DDS uses Asio 1.18.1 (same as Ubuntu Jammy) and it is not until Asio 1.22 that the deprecation warnings are fixed.
Doxygen 1.9 also deprecates some configuration options included with Doxygen 1.8.
Finally, the project name in the roadmap is fixed using the current Fast DDS name.

Contributor Checklist

Commit messages follow the project guidelines.
N/A The code follows the style guidelines of this project.
N/A Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added.
N/A Any new/modified methods have been properly documented using Doxygen.
N/A Fast DDS test suite has been run locally.
N/A Changes are ABI compatible.
N/A Changes are API compatible.
N/A Documentation builds and tests pass locally.
N/A New feature has been added to the versions.md file (if applicable).
N/A New feature has been documented/Current behavior is correctly described in the documentation.

Reviewer Checklist

Check contributor checklist is correct.
Check CI results: changes do not issue any warning.
Check CI results: failing tests are unrelated with the changes.

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

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

JLBuenoLopez · 2022-12-16T08:24:38Z

@Mergifyio backport 2.8.x 2.7.x 2.6.x 2.1.x

* Refs #16125: update doxygen configuration file Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: update library name in Roadmap Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: remove warning: variable may be used uninitialized Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: remove OpenSSL deprecation warnings when forcing Asio as thirdparty Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> (cherry picked from commit d024c10)

* Refs #16125: update doxygen configuration file Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: update library name in Roadmap Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: remove warning: variable may be used uninitialized Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: remove OpenSSL deprecation warnings when forcing Asio as thirdparty Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> (cherry picked from commit d024c10) # Conflicts: # include/fastdds/rtps/transport/TCPAcceptorSecure.h # src/cpp/rtps/transport/TCPChannelResourceSecure.h # src/cpp/rtps/transport/TCPTransportInterface.h # test/unittest/statistics/rtps/RTPSStatisticsTests.cpp

mergify · 2022-12-16T08:25:54Z

backport 2.8.x 2.7.x 2.6.x 2.1.x

✅ Backports have been created

Details

#3155 [16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) (backport #3078) has been created for branch 2.8.x
#3156 [16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) (backport #3078) has been created for branch 2.7.x
#3157 [16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) (backport #3078) has been created for branch 2.6.x
#3158 [16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) (backport #3078) has been created for branch 2.1.x

* Several fixes to remove warnings in Ubuntu Jammy (22.04) (#3078) * Refs #16125: update doxygen configuration file Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: update library name in Roadmap Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: remove warning: variable may be used uninitialized Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: remove OpenSSL deprecation warnings when forcing Asio as thirdparty Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> (cherry picked from commit d024c10) * Refs #16328: fix linux warnings Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> --------- Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> Co-authored-by: José Luis Bueno López <69244257+JLBuenoLopez-eProsima@users.noreply.github.com> Co-authored-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

* Several fixes to remove warnings in Ubuntu Jammy (22.04) (#3078) * Refs #16125: update doxygen configuration file Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: update library name in Roadmap Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: remove warning: variable may be used uninitialized Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> * Refs #16125: remove OpenSSL deprecation warnings when forcing Asio as thirdparty Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> (cherry picked from commit d024c10) * Refs #16329: fix Ubuntu warnings Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> --------- Signed-off-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com> Co-authored-by: José Luis Bueno López <69244257+JLBuenoLopez-eProsima@users.noreply.github.com> Co-authored-by: JLBuenoLopez-eProsima <joseluisbueno@eprosima.com>

JLBuenoLopez added 4 commits November 8, 2022 09:05

Refs #16125: update doxygen configuration file

b710ef0

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

Refs #16125: update library name in Roadmap

7e35d53

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

Refs #16125: remove warning: variable may be used uninitialized

3b51027

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

Refs #16125: remove OpenSSL deprecation warnings when forcing Asio as…

0543677

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

JLBuenoLopez added the no-test Skip CI tests if PR marked with this label label Nov 8, 2022

JLBuenoLopez mentioned this pull request Nov 8, 2022

[15964] Skip ASAN job if skip-ci or no-test label is set #3077

Merged

4 tasks

EduPonz added this to the v2.9.0 milestone Nov 15, 2022

MiguelCompany changed the title ~~[16125] Several fixes to remove warnings in Ubuntu Jammy (22.04)~~ [16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) Nov 30, 2022

MiguelCompany approved these changes Nov 30, 2022

View reviewed changes

jsan-rt added the ready-to-merge Ready to be merged. CI and changes have been reviewed and approved. label Dec 7, 2022

MiguelCompany merged commit d024c10 into master Dec 7, 2022

MiguelCompany deleted the feature/jammy-updates branch December 7, 2022 17:56

mergify bot mentioned this pull request Dec 16, 2022

[16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) (backport #3078) #3155

Merged

mergify bot mentioned this pull request Dec 16, 2022

[16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) (backport #3078) #3156

Merged

This was referenced Dec 16, 2022

[16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) (backport #3078) #3157

Merged

[16329] Several fixes to remove warnings in Ubuntu Jammy (22.04) (backport #3078) #3158

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[16329] Several fixes to remove warnings in Ubuntu Jammy (22.04)#3078

[16329] Several fixes to remove warnings in Ubuntu Jammy (22.04)#3078
MiguelCompany merged 4 commits intomasterfrom
feature/jammy-updates

JLBuenoLopez commented Nov 8, 2022

Uh oh!

JLBuenoLopez commented Dec 16, 2022

Uh oh!

mergify bot commented Dec 16, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

JLBuenoLopez commented Nov 8, 2022

Description

Contributor Checklist

Reviewer Checklist

Uh oh!

JLBuenoLopez commented Dec 16, 2022

Uh oh!

mergify bot commented Dec 16, 2022

✅ Backports have been created

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants