Skip to content

CXX-2661 Add macos-1100 tasks and move ubuntu1404 -> ubuntu2204 #974

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

Merged
merged 4 commits into from
Jun 15, 2023

Conversation

eramongodb
Copy link
Contributor

@eramongodb eramongodb commented Jun 9, 2023

Description

As a first step for CXX-2661, adds macos-1100 equivalent tasks for existing macos-1014 tasks. Removal of old macos-1014 tasks would fully resolve CXX-2661.

Additionally, tasks on ubuntu1404 were moved to ubuntu2204 per distro guidelines, resolving CXX-2657 and ongoing failures in the waterfall.

The Apple Clang compiler on the macos-1100 distro appears to have a new warning enabled, -Wrange-loop-analysis, which are also addressed by this PR.

Verified by this patch.

Boost Polyfill Detection

Example projects were updated to replace Windows-only special handling of the conditional Boost dependency, and instead generically detect the use of the Boost polyfill library via the CMakeCXXSymbolExists module to detect the presence of the BSONCXX_POLY_USE_BOOST preprocessor macro. This avoids example project build failure when compiling with Boost on MacOS.

Note: this only applies to the example projects using the deprecated CMake project config files, as the newer CMake project config files automatically handles the Boost dependency detection and inclusion (see #968 (review) for more context).

@eramongodb eramongodb self-assigned this Jun 9, 2023
@eramongodb eramongodb merged commit 418e219 into mongodb:master Jun 15, 2023
@eramongodb eramongodb deleted the cxx-2662 branch June 15, 2023 20:26
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