Skip to content

Avoid passing Linux-like flags on Windows distros #1096

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 1 commit into from
Feb 16, 2024

Conversation

eramongodb
Copy link
Contributor

Prior attempt: #1094 (patch failure due to base commit issues)

Addresses task failures on Windows distros in the r3.9 waterfall which are interfering with CXX-2818 validation. Verified by this patch.

cl : Command line error D8021: invalid numeric argument '/Wno-deprecated-declarations'

Admittedly I'm not entirely sure why related tasks are succeeding on the main branch waterfall, but these are good changes to make regardless imo.

Copy link
Collaborator

@kevinAlbs kevinAlbs left a comment

Choose a reason for hiding this comment

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

I'm not entirely sure why related tasks are succeeding on the main branch waterfall

I expect it is due to only including -Wno-deprecated-declarations on release branches:

# Ignore deprecation warnings when building on a release branch.
if [ "$(echo "${branch_name:?}" | cut -f2 -d'/')" != "${branch_name:?}" ]; then
  cc_flags+=(-Wno-deprecated-declarations)
  cxx_flags+=(-Wno-deprecated-declarations)
fi

@eramongodb eramongodb merged commit 886b7d9 into mongodb:releases/v3.9 Feb 16, 2024
@eramongodb eramongodb deleted the cxx-releases/v3.9 branch February 16, 2024 20:52
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.

2 participants