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

Update RHEL7/Centos7/OracleLinux7 dockerfiles #829

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

jepio
Copy link
Member

@jepio jepio commented Dec 5, 2024

Description

Update build container dockerfiles for Centos 7 (and derivative) to fix an issue preventing commandrunner and samples/cpp from building.

Rapidjson has a google test submodule that it installs. This module has a version of google test that does not match the one we expect and overwrites the version that we clone ourselves. Disable installation of rapidjson's googletest by setting a cmake variable and order the gtest installation after rapidjson to ensure google test is installed correctly.

Centos 7 mirror lists have been decomissioned but we can still access a snapshot of the repo - we just have to overwrite the repo definitions. This is necessary to get the containers to build at all.

Checklist

  • I have read the contribution guidelines.
  • All unit tests are passing.
  • I have merged the latest main branch prior to this PR submission.
  • I submitted this PR against the main branch.

- Rapidjson has a google test submodule that it installs. This module is
  incompatible with our usage of google test, and conflicts with the version
  that we clone ourselves. Disable installations of rapidjson's googletest and
  reorder to ensure google test is built last.
- Centos 7 mirror lists have been decomissioned but we can still access a snapshot
  of the repo - we just have to overwrite the repo definitions.
@jepio jepio requested a review from a team as a code owner December 5, 2024 16:16
Copy link
Contributor

@MariusNi MariusNi left a comment

Choose a reason for hiding this comment

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

:shipit:

@MariusNi MariusNi merged commit c1a3f6e into Azure:main Dec 5, 2024
116 of 121 checks passed
@jepio jepio deleted the update-centos7-dockerfiles branch December 6, 2024 09:03
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