Skip to content

[smart_holder] git merge master #3284

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 6 commits into from
Sep 20, 2021
Merged

Conversation

rwgk
Copy link
Collaborator

@rwgk rwgk commented Sep 20, 2021

Helper/scratch PR for running the CI.

Mainly to test the change in .github/workflows/ci_sh_def.yml

All other changes were tested previously via the helper/scratch PR #3282.

pre-commit-ci bot and others added 5 commits September 13, 2021 15:58
updates:
- [github.com/asottile/pyupgrade: v2.25.0 → v2.26.0](asottile/pyupgrade@v2.25.0...v2.26.0)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Eliminate duplicate TLS keys for loader_life_support stack

This revises the existing fix for
pybind#2765 in
pybind#3237 to reduce the amount of
TLS storage used.

The shared TLS key is stored in two different ways, depending on
`PYBIND11_INTERNALS_VERSION`.  If `PYBIND11_INTERNALS_VERSION ==
4` (as is currently set), the TLS key is stored in the
`internal::shared_data` map to avoid breaking ABI compatibility.  If
`PYBIND11_INTERNALS_VERSION > 4`, the TLS key is stored directly in
the `internals` struct.

* Fix test_pytypes.py::test_issue2361 failure on PyPy3.7

* Add github actions tests for unstable ABI
@rwgk
Copy link
Collaborator Author

rwgk commented Sep 20, 2021

Merging, after squashing two commits.

@rwgk rwgk marked this pull request as ready for review September 20, 2021 14:22
@rwgk rwgk requested a review from henryiii as a code owner September 20, 2021 14:22
@rwgk rwgk merged commit 10350ab into pybind:smart_holder Sep 20, 2021
@rwgk rwgk deleted the sh_merge_master branch September 20, 2021 14:23
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.

4 participants