Repository navigation
Add support python 3.15 - #1954
p-r-a-v-i-n wants to merge 2 commits into
Conversation
|
pls see #1963 |
|
I would need some time to get back on this, may be till 10th oct. |
d6d3ab8 to
b4e66ae
Compare
|
rgd the CI fails: this is more likely a PyPy ABI/compatibility issue in the cryptography binary than a bug in Autobahn or Twisted. The key clue is the undefined symbol PyPy_DecRef.
|
|
should we keep the |
|
my view: as good OSS citizens, we should track down this issue properly (is it cryptography or pypy?), and inform upstream, file an issue that reproduces the issue with minimal venv (without autobahn), and wait for upstream to fix. until then, I don't want to introduce even more complexity into our CI just to have cpy315 and not pypy312. that's just my personal view and judgement, one might very well disagree! but we're dealing with such stuff all the time:( call it unfortunate, call it business as usual, call it that's just how the world is;) eg here is another example (unrelated to autobahn-python, but related to crossbar): crossbario/wamp-xbr#209 |
|
I respect what you said and I totally agree with you and actually this how i reached here for solving similar problem in other project.
I reproduced this without Autobahn or Twisted using only |
|
@oberstet Pyo3 closes that issue, what we should do now next ? I used AI for writing that Issue. Bcs reproducing that issue is not a brainy work, it a friction work and AI is useful to reduce this friction. |
|
@p-r-a-v-i-n here is my view:
tldr
digging deeper, level 1The fix is already released — it shipped in PyO3 0.29.3 (tagged 30 Sep 2026), not in 0.29.4. The maintainer's version number looks slightly off. The commits that fix #6498:
Here is the before and after for the symbol in your traceback: There is a What this means for autobahn-python: your build used cryptography 50.0.2, which pulled in pyo3 0.29.2 from its lockfile. The fix is to build cryptography against pyo3 ≥ 0.29.3. You can wait for a cryptography release that bumps it, or patch its Sources:
digging deeper, level 2I'll trace the pyo3 bump in cryptography's lockfile and check which tags contain it. It isn't in any release yet. The bump is only on
One correction to "lower floor": Until cryptography 50.0.3 or 51 ships, you have two ways to build on PyPy 3.12:
I checked release tags in the repo rather than PyPI directly, because PyPI blocked the fetch. cryptography tags every release, so 50.0.2 should be the latest. Sources:
|
Thanks, I really like your attitude. Initially i thought this work is just changing the versions from previous references but it is more than that. I think I rushed in the process too much. I should have invest time in digging. I will go through your suggestions slowly. thank you! |
blocked until cryptography 50.0.3 or 51 ships
Description
Please explain the changes you are proposing.
Related Issue(s)
Closes or relates to #
Checklist
the style guidelines of this project
is effective or that my feature works
updated the changelog
in this PR
AI Assistance Disclosure File
file at
.audit/<branch-name>.mdright in this PR. Thedisclosure file must follow the exact format and content as
described below. Your PR will not be accepted without a
disclosure file.
Example 1 file contents of your disclosure file
.audit/<branch-name>.md:OR
Example 2 file contents of your disclosure file
.audit/<branch-name>.md:Submitting code generated primarily by AI, or for which you
cannot claim human authorship, is not permitted. See
AI Policy
for details.
Example 1 OR Example 2 show the only valid two variants. You
cannot have both or none of the first two marks checked, and you
must always have the last tick checked.
Well, "must" if you want your PR to be accepted and ultimately
merged that is. Of course you are always free to
Go ahead! Fork my Day. (TM);) This is Open-source.