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

CMake Asan on GHA and Fixes for CMake on Scoreboard #3382

Merged
merged 4 commits into from Mar 8, 2022

Conversation

iguessthislldo
Copy link
Member

  • Readd CMake Asan Build & Test Removed in GitHub Actions: Remove CMake from ASan build #2836
  • Support gcov in CMake
  • Default OPENDDS_DEBUG and OPENDDS_INLINE to true, which are default build features.
  • Pass through toolchain file in install test for build on the scoreboard.

Use common mechanism to inject compiler and linker arguments that is
also used with sanitizers.

Also
- Default `OPENDDS_DEBUG` and `OPENDDS_INLINE` to true, which are
  default build features.
- Passthrough toolchain file in install test for build on the
  scoreboard.
Copy link
Member

@mitza-oci mitza-oci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this needs to be resolved before merging:

test_u20_p1_asan_sec

tests/cmake/install/CMakeLists.txt run_test.pl clean-env
/home/runner/work/OpenDDS/OpenDDS/OpenDDS/tests/cmake/build/install/the-install-prefix/bin/opendds_install_test_user: error while loading shared libraries: libTAO_PortableServer.so: cannot open shared object file: No such file or directory

@iguessthislldo
Copy link
Member Author

Looks like this needs to be resolved before merging:

test_u20_p1_asan_sec

tests/cmake/install/CMakeLists.txt run_test.pl clean-env
/home/runner/work/OpenDDS/OpenDDS/OpenDDS/tests/cmake/build/install/the-install-prefix/bin/opendds_install_test_user: error while loading shared libraries: libTAO_PortableServer.so: cannot open shared object file: No such file or directory

This is DOCGroup/ACE_TAO#1790.

@mitza-oci
Copy link
Member

Looks like this needs to be resolved before merging:

This is DOCGroup/ACE_TAO#1790.

I thought we had a workaround in place?

@iguessthislldo
Copy link
Member Author

We did, but not on the asan build and I didn't think to add it originally.

@mitza-oci mitza-oci merged commit 43c46ed into OpenDDS:master Mar 8, 2022
mitza-oci added a commit that referenced this pull request Mar 11, 2022
CMake Asan on GHA and Fixes for CMake on Scoreboard

(cherry picked from commit 43c46ed)
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.

None yet

2 participants