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

binding/query.c: use PyErr_Format instead of _PyErr_FormatFromCause #284

Closed
wants to merge 2 commits into from

Conversation

arthurzam
Copy link

The _PyErr_FormatFromCause function is a private function, which was never documented. While it existed for a long time, it was removed in Python 3.13 python/cpython#106320.

Use instead the public function PyErr_Format, which is also part of the stable API.

The `_PyErr_FormatFromCause` function is a private function, which was
never documented. While it existed for a long time, it was removed in
Python 3.13 [1].

Use instead the public function `PyErr_Format`, which is also part of
the stable API.

[1] python/cpython#106320

Signed-off-by: Arthur Zamarin <[email protected]>
Signed-off-by: Arthur Zamarin <[email protected]>
@ObserverOfTime
Copy link
Member

PyErr_Format does not include the cause.

@arthurzam
Copy link
Author

PyErr_Format does not include the cause.

OK, so which other function should we use? I can modify it as needed.

@ObserverOfTime
Copy link
Member

It's complicated. I'll look into it during the weekend.

@arthurzam
Copy link
Author

Thank you. I leave it to your discretion if you want to just close this PR, or override it, or anything else.

@ObserverOfTime
Copy link
Member

Fixed in 2f7889d

@chenrui333
Copy link

@ObserverOfTime any chance cutting a new release? Thanks!

@ObserverOfTime
Copy link
Member

It's in v0.23.1.

@chenrui333
Copy link

sorry, I missed it, the 0.23.1 works for me, thanks!

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