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

docs: prepare for 2.10.1 release #4279

Merged
merged 5 commits into from Oct 31, 2022

Conversation

henryiii
Copy link
Collaborator

@henryiii henryiii commented Oct 24, 2022

Description

Work on #4216 exposed deadlocks previously mistaken for flakes. This condition existed at least for a couple years and will be further debugged after the 2.10.1 release.

Closes #4125. CC @wjakob, @rwgk, @Skylion007, @ax3l.

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
Copy link
Collaborator

@rwgk rwgk left a comment

Choose a reason for hiding this comment

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

To the other maintainers: please vote your mind.

I feel digging into the suspected ODR question is most important at the moment, expecting it's only "hours" (ballpark) of effort to get more clarity, not "days". So that's what I'll do now.

I'm not opposed to this release anymore after coming to believe that the internals bump won't help, but I still feel more clarity is needed.

docs/changelog.rst Outdated Show resolved Hide resolved

Changes:
Using at least this version is recommended if you need Python 3.11 support
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change:


This release has one change (PR #4119) that improves compatibility with Python 3.11 for users of pybind11/embed.h.


I think it's important to highlight, but I'd let people decide by themselves how important this is to them. (I assume it's not important to most users.)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Reworded to clarify that this the first version to support embedding 3.11.

docs/changelog.rst Outdated Show resolved Hide resolved
docs/changelog.rst Outdated Show resolved Hide resolved
@rwgk
Copy link
Collaborator

rwgk commented Oct 25, 2022

@henryiii I could make the release tomorrow, I think I remember enough + the documentation that I can figure out all the steps. Please let me know if that would be helpful.

@henryiii
Copy link
Collaborator Author

That would be, I won't have a consistent amount of free time to make the release in the next few days. Changes above look fine, and we ideally want one more PR (the GIL_whatever_we_called_it) one. And maybe your error export one, though I should point out that's not an issue reported by users, so I'd be careful about making it too critical - if it's a largish change, it needs to sit in master a bit.

@rwgk
Copy link
Collaborator

rwgk commented Oct 25, 2022

That would be, I won't have a consistent amount of free time to make the release in the next few days.

Thanks! I'll take care of it.

Changes above look fine, and we ideally want one more PR (the GIL_whatever_we_called_it) one.

Will take care of it.

And maybe your error export one,

I looked at the list of my open PRs but it didn't really click: which PR? This one? #4072 (comment)

Just the visibility fixes, but not the cmake change? (I was thinking myself it's best to split the PR.)

@henryiii
Copy link
Collaborator Author

#4072 needs to wait and get reviewed properly, and is not a target for 2.10.x. I just meant whatever you are working on for the ODR worries in follow up to the ABI issue, if you think it should go into 2.10.

@rwgk
Copy link
Collaborator

rwgk commented Oct 25, 2022

#4072 needs to wait and get reviewed properly, and is not a target for 2.10.x. I just meant whatever you are working on for the ODR worries in follow up to the ABI issue, if you think it should go into 2.10.

Oh good. I think "trimming back the scope #2999" (to reduce the scope for ODR worries) does not have to go into 2.10. So just this PR and #4216, then we're good.

@henryiii
Copy link
Collaborator Author

Waiting on, and including, #4297.

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
docs/changelog.rst Outdated Show resolved Hide resolved
Copy link
Collaborator

@rwgk rwgk left a comment

Choose a reason for hiding this comment

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

Thanks! I really like the Halloween date for the release.
Please go ahead merge #4297 if it looks good to you.

@rwgk
Copy link
Collaborator

rwgk commented Oct 31, 2022

I changed (simplified) the PR description, in case someone looks here later. The previous version was outdated.

Copy link
Collaborator

@ax3l ax3l left a comment

Choose a reason for hiding this comment

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

Thank you 🙏

@ax3l ax3l added the docs Docs or GitHub info label Oct 31, 2022
@henryiii henryiii merged commit 252ed8f into pybind:master Oct 31, 2022
@henryiii henryiii deleted the henryiii/chore/2.10.1 branch October 31, 2022 18:11
henryiii added a commit that referenced this pull request Oct 31, 2022
* docs: prepare for 2.10.1 release

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>

* Update changelog.rst

* docs: update changelog with final list of PRs

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>

* Update docs/changelog.rst

* chore: one more changelog bump

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
rwgk pushed a commit that referenced this pull request Nov 18, 2022
* docs: prepare for 2.10.1 release

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>

* Update changelog.rst

* docs: update changelog with final list of PRs

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>

* Update docs/changelog.rst

* chore: one more changelog bump

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Docs or GitHub info
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[PLAN] Tracking issue for 2.10.1
4 participants