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

Bump required C driver to 1.25.0 #1056

Merged
merged 6 commits into from
Nov 8, 2023

Conversation

kevinAlbs
Copy link
Collaborator

@kevinAlbs kevinAlbs commented Nov 8, 2023

Summary

This PR bumps the required C driver to 1.25.0 and includes additional changes intended before the 3.9.0 release:

  • Fix documented version of the auto C driver download introduction (3.9.0, not 3.8.1).
  • Remove extra step for updating versions in Dockerfile.
  • Remove check_docker_file_versions.
  • Remove reference to removed Evergreen variant.
  • Add -pthread to pkg-config example projects to work around CDRIVER-4776

Background & Motivation

Updating MONGOCXX_VERSION in Dockerfile before tagging a release may result in error downloading the not-yet-released tarball. This PR assumes it is preferable to update versions in the Dockerfiles after tagging a release.

"Ubuntu 18.04 with minimum libmongoc" was removed in PR mongodb#1049. Testing with minimum libmongoc is now the default.
This step is listed in later in `Docker Image Build and Publish`.

Building the Docker images requires downloading the C++ release tarball. Bump after tagging.
The Docker versions are bumped after tagging the release. This check may not be expected to succeed during a release.
It is introduced in 3.9.0 as part of CXX-2695.
Copy link
Contributor

@kkloberdanz kkloberdanz left a comment

Choose a reason for hiding this comment

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

Looks good!

This is a workaround until CDRIVER-4776 is resolved.
@kevinAlbs
Copy link
Collaborator Author

Updating to C driver 1.25.0 resulted in an error building example projects with pkg-config using the static libraries. e9f0f9d applies a workaround. CDRIVER-4776 tracks a fix in the C driver.

@kevinAlbs kevinAlbs merged commit 19a6099 into mongodb:master Nov 8, 2023
57 of 59 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants