Skip to content

[PR #13457/e54b79cb backport][3.15] Build the websocket reader_c extension without the reader_c.py symlink - #13469

Merged
Dreamsorcerer merged 1 commit into
3.15from
patchback/backports/3.15/e54b79cbae69e24d5e87469613e97a3d9bec0d4b/pr-13457
Aug 17, 2026
Merged

[PR #13457/e54b79cb backport][3.15] Build the websocket reader_c extension without the reader_c.py symlink#13469
Dreamsorcerer merged 1 commit into
3.15from
patchback/backports/3.15/e54b79cbae69e24d5e87469613e97a3d9bec0d4b/pr-13457

Conversation

@patchback

@patchback patchback Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

This is a backport of PR #13457 as merged into master (e54b79c).

What do these changes do?

Removes the aiohttp/_websocket/reader_c.py symlink; the extension is now compiled directly from reader_py.py using cython --module-name aiohttp._websocket.reader_c. The symlink only existed so Cython would pair the source with reader_c.pxd and emit the right module name, but after #13388 the wheel build materializes it as a regular file in site-packages, which coverage then reports as a new 0% file.

Are there changes in behavior for the user?

No runtime changes; dists no longer contain a stray reader_c.py file, reader_c only exists as the compiled extension.

Is it a substantial burden for the maintainers to support this?

No, it is one less special case; the Makefile rule is the only thing that changed.

Related issue number

Follow up to #13388

Checklist

  • I think the code is well written
  • Unit tests for the changes exist
    • N/A, build tooling only, covered by the existing websocket test suite with extensions enabled
  • Documentation reflects the changes
    • N/A
  • If you provide code modification, please add yourself to CONTRIBUTORS.txt
    • N/A, already listed
  • Add a new news fragment into the CHANGES/ folder
    • name it <issue_or_pr_num>.<type>.rst (e.g. 588.bugfix.rst)

Drafted with Claude Code; reviewed by bdraco.

@greptile-apps

greptile-apps Bot commented Aug 17, 2026

Copy link
Copy Markdown

Confidence Score: 5/5

The PR appears safe to merge, with no concrete build, runtime, packaging, or security failure identified.

The changed build rule preserves the extension name expected by setup and imports, while the existing distribution pipeline generates the C source before packaging; no actionable regression remains.

Reviews (1): Last reviewed commit: "Build the websocket reader_c extension w..." | Re-trigger Greptile

@codecov

codecov Bot commented Aug 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.20%. Comparing base (7977dcf) to head (9912337).
⚠️ Report is 1 commits behind head on 3.15.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             3.15   #13469      +/-   ##
==========================================
+ Coverage   95.66%   96.20%   +0.54%     
==========================================
  Files         160      159       -1     
  Lines       51183    50894     -289     
  Branches     2828     2771      -57     
==========================================
- Hits        48964    48963       -1     
+ Misses       2039     1751     -288     
  Partials      180      180              
Flag Coverage Δ
CI-GHA 96.17% <0.00%> (+0.54%) ⬆️
OS-Linux 95.95% <0.00%> (+0.54%) ⬆️
OS-Windows 93.68% <0.00%> (+0.52%) ⬆️
OS-macOS 95.13% <0.00%> (+0.54%) ⬆️
Py-3.10 95.30% <0.00%> (+0.53%) ⬆️
Py-3.11 95.58% <0.00%> (+0.53%) ⬆️
Py-3.12 95.66% <0.00%> (+0.54%) ⬆️
Py-3.13 95.65% <0.00%> (+0.55%) ⬆️
Py-3.14 95.74% <0.00%> (+0.54%) ⬆️
Py-3.14t 94.75% <0.00%> (+0.53%) ⬆️
Py-pypy-3.11 94.61% <0.00%> (+0.53%) ⬆️
VM-macos 95.13% <0.00%> (+0.54%) ⬆️
VM-ubuntu 95.95% <0.00%> (+0.54%) ⬆️
VM-windows 93.68% <0.00%> (+0.52%) ⬆️
cython-coverage 78.05% <0.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@Dreamsorcerer
Dreamsorcerer merged commit 347f922 into 3.15 Aug 17, 2026
47 of 48 checks passed
@Dreamsorcerer
Dreamsorcerer deleted the patchback/backports/3.15/e54b79cbae69e24d5e87469613e97a3d9bec0d4b/pr-13457 branch August 17, 2026 21:50
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.

2 participants