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

Merge v1.19 into master #1562

Closed
wants to merge 2 commits into from

Conversation

mongodb-php-bot
Copy link
Collaborator

Merge new changes from v1.19 into master.

Commits

Resolving conflicts

To resolve any conflicts, check out the temporary branch and run the following command:
git merge master

Ignoring changes

To ignore from the remote branch, first reset the temporary branch to master and manually merge using the `ours` merge strategy:
git reset --hard master
git merge --strategy=ours v1.19

Then, push the temporary branch to upate the pull request.

jmikola and others added 2 commits May 22, 2024 09:46
* PHPC-2382: Allow static builds with php-src

Prefer PHP_VERSION_ID for static builds, and fall back to invoking php-config otherwise.

mongocrypt.h previously used a path relative to the PHPC project root, which is not compatible with static builds. The current path is relative to src/, which is explicitly defined as an include path in config.m4.

* Preserve CPPFLAGS before modification by PlatformFlags.m4

PlatformFlags.m4 modifies CPPFLAGS in order to make checks in subsequent M4 scripts consistent with PHP_MONGODB_BUNDLED_CFLAGS. Restoring its original value after executing all M4 scripts will avoid unintended side effects on a static PHP build.

* Remove obsolete STD_CFLAGS assignments

STD_CFLAGS was never appended to PHP_MONGODB_BUNDLED_CFLAGS after these M4 scripts were run, so it's safe to assume they had no effect and can be removed entirely.

For CheckCompiler.m4, bumping the GCC requirement to 4.6 (still quite old) will ensure compatibility for libmongoc's BEGIN_IGNORE_DEPRECATIONS macro.

* Prefix names of internal CFLAGS vars in config.m4

* Remove obsolete comment in config.m4

This comment should have been removed in d8d30e5

* Include CPPFLAGS in build config output

Co-authored-by: Andreas Braun <[email protected]>
@mongodb-php-bot mongodb-php-bot enabled auto-merge May 22, 2024 13:53
@alcaeus alcaeus closed this May 22, 2024
auto-merge was automatically disabled May 22, 2024 13:54

Pull request was closed

@alcaeus
Copy link
Member

alcaeus commented May 22, 2024

Closing in favour of merging up v1.19 once #1561 has been merged to v1.19.

@alcaeus alcaeus deleted the merge-v1.19-into-master-1716386016542 branch May 22, 2024 13:54
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