GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||||||||||||||||||||||
GeezerVison is showing github.com — Support "bpo-" in Misc/NEWS by brettcannon · Pull Request #1 · python/cpython Conversation
Member
No description provided.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Carreau
reviewed
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That's on purpose as the first capture group is the prefix which doesn't matter. Probably should change the regex to use a non-capture group.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sure, you can either you use
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah, you're right. Now fixed.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsMake the prefix match non-capturing and fix a bug where a backslash was left into the substitution.
Carreau
approved these changes
zware
reviewed
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Shouldn't it be
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. My patch for another issue will address this, I think 🤔
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I went ahead and updated here since I'm already changing it.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok :)
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
zware
reviewed
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd suggest making it
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
zware
approved these changes
Mariatta
referenced
this pull request
in Mariatta/cpython
(cherry picked from commit 79ab8be)
Mariatta
referenced
this pull request
in Mariatta/cpython
Change the url to 3.5 (cherry picked from commit 79ab8be) # Conflicts: # Doc/tools/extensions/pyspecific.py
Mariatta
referenced
this pull request
in Mariatta/cpython
Change the url to 2.7 (cherry picked from commit 79ab8be)
Mariatta
added a commit
that referenced
this pull request
tiran
pushed a commit
that referenced
this pull request
* Added support for CAN_ISOTP protocol * Added unit tests for CAN ISOTP * Updated documentation for ISO-TP protocol * Removed trailing whitespace in documentation * Added blurb NEWS.d file * updated Misc/ACKS * Fixed broken unit test that was using isotp const outside of skippable section * Removed dependecy over third party project * Added implementation for getsockname + unit tests * Missing newline at end of ACKS file * Accidentally inserted a type in ACKS file * Followed tiran changes review #1 recommendations * Added spaces after comma
GadgetSteve
referenced
this pull request
in GadgetSteve/cpython
* Added support for CAN_ISOTP protocol * Added unit tests for CAN ISOTP * Updated documentation for ISO-TP protocol * Removed trailing whitespace in documentation * Added blurb NEWS.d file * updated Misc/ACKS * Fixed broken unit test that was using isotp const outside of skippable section * Removed dependecy over third party project * Added implementation for getsockname + unit tests * Missing newline at end of ACKS file * Accidentally inserted a type in ACKS file * Followed tiran changes review #1 recommendations * Added spaces after comma
native-api
pushed a commit
to native-api/cpython
that referenced
this pull request
Clucier17
reviewed
This was referenced
barneygale
added a commit
to barneygale/cpython
that referenced
this pull request
eyzmeng
pushed a commit
to rapidcow/cpython
that referenced
this pull request
Update email.message.rst
Kaushalt2004
added a commit
to Kaushalt2004/cpython
that referenced
this pull request
Kaushalt2004
added a commit
to Kaushalt2004/cpython
that referenced
this pull request
…new__ override, rely on __add__ only. Clarify with reviewer note. (python#1 branch)
Krishna-web-hub
added a commit
to Krishna-web-hub/cpython
that referenced
this pull request
Krishna-web-hub
added a commit
to Krishna-web-hub/cpython
that referenced
this pull request
Co-authored-by: Bénédikt Tran <[email protected]>
SonicField
added a commit
to SonicField/cpython
that referenced
this pull request
Add timing for all intermediate GC phases to diagnose the 50-80ms gap between timed phases and total gc.collect() time. New timing fields: - scan_heap_ns: parallel scan_heap phase - disable_deferred_ns: disable deferred refcounting loop - find_weakrefs_ns: find_weakref_callbacks - stw1_ns: StartTheWorld python#1 - objs_decref_ns: cleanup_worklist(objs_to_decref) - weakref_callbacks_ns: call_weakref_callbacks - finalize_ns: finalize_garbage - stw2_ns: StopTheWorld python#2 - resurrection_ns: handle_resurrected_objects - freelists_ns: _PyGC_ClearAllFreeLists - clear_weakrefs_ns: clear_weakrefs - stw3_ns: StartTheWorld python#3 All values exposed via gc.get_parallel_stats()['phase_timing'].
savannahostrowski
pushed a commit
that referenced
this pull request
…140574) * gh-140281: Doc: Update free-threading how-to * gh-140281: Doc: Update free-threading how-to * Fix trailing whitespace * doc fixing of the cpython fixes#84116 * Docs: Document help and aliases for argparse.add_parser() (gh-84116) * Docs: Document help and aliases for argparse.add_parser() (gh-84116) * Fix trailing whitespace * Fix trailing whitespace * Fix trailing whitespace * Fix trailing whitespace * Fix trailing whitespace * Fix trailing whitespace * Fix trailing errors * Fix trailing errors and spaces * Fix docutils formatting, NEWS ref, and trailing whitespace * Docs: Update argparse.rst and add NEWS entry * Delete Doc/howto/free-threading-python.rst * Delete Misc/NEWS.d/next/Documentation/2025-10-25-00-49-43.gh-issue-140281.tuMQUe.rst * adding the depreceated tag * The error indexing was fixed * Fix trailing whitespace * Restore missing free-threading documentation * fixing some minor error * fixing some minor error part 2 * fixing some minor error part 3 * Fix NEWS entry format * Final cleanup of NEWS entry format 2 * changes in the argparse.rst * Remove unnecessary NEWS entry * Fixing the issue as requested * Added the Changes done before. * done the changes * done the changes#1 * Apply suggestion from @picnixz Co-authored-by: Bénédikt Tran <[email protected]> * Apply suggestion from @picnixz Co-authored-by: Bénédikt Tran <[email protected]> * Done the changes * Done the new changes * The versionadded is rectified * Update Doc/library/argparse.rst Co-authored-by: Stan Ulbrych <[email protected]> * Docs editing * Docs fixing whitespace * Docs rectifiying * little bit rectification * Indentation rectification * Indentation rectification 1 * Apply suggestion from @picnixz Co-authored-by: Bénédikt Tran <[email protected]> * deprecated rectification * Remove mistakenly added NEWS entry * Update Doc/library/argparse.rst Co-authored-by: Bénédikt Tran <[email protected]> * changes #1 Co-authored-by: Bénédikt Tran <[email protected]> * Update Doc/library/argparse.rst 2 Co-authored-by: Bénédikt Tran <[email protected]> * removed useless thing Co-authored-by: Bénédikt Tran <[email protected]> * changed according to the request Co-authored-by: Bénédikt Tran <[email protected]> * Updated the test part --------- Co-authored-by: Bénédikt Tran <[email protected]> Co-authored-by: Stan Ulbrych <[email protected]>
thunder-coding
pushed a commit
to thunder-coding/cpython
that referenced
this pull request
…er() (python#140574) * pythongh-140281: Doc: Update free-threading how-to * pythongh-140281: Doc: Update free-threading how-to * Fix trailing whitespace * doc fixing of the cpython fixes#84116 * Docs: Document help and aliases for argparse.add_parser() (pythongh-84116) * Docs: Document help and aliases for argparse.add_parser() (pythongh-84116) * Fix trailing whitespace * Fix trailing whitespace * Fix trailing whitespace * Fix trailing whitespace * Fix trailing whitespace * Fix trailing whitespace * Fix trailing errors * Fix trailing errors and spaces * Fix docutils formatting, NEWS ref, and trailing whitespace * Docs: Update argparse.rst and add NEWS entry * Delete Doc/howto/free-threading-python.rst * Delete Misc/NEWS.d/next/Documentation/2025-10-25-00-49-43.gh-issue-140281.tuMQUe.rst * adding the depreceated tag * The error indexing was fixed * Fix trailing whitespace * Restore missing free-threading documentation * fixing some minor error * fixing some minor error part 2 * fixing some minor error part 3 * Fix NEWS entry format * Final cleanup of NEWS entry format 2 * changes in the argparse.rst * Remove unnecessary NEWS entry * Fixing the issue as requested * Added the Changes done before. * done the changes * done the changes#1 * Apply suggestion from @picnixz Co-authored-by: Bénédikt Tran <[email protected]> * Apply suggestion from @picnixz Co-authored-by: Bénédikt Tran <[email protected]> * Done the changes * Done the new changes * The versionadded is rectified * Update Doc/library/argparse.rst Co-authored-by: Stan Ulbrych <[email protected]> * Docs editing * Docs fixing whitespace * Docs rectifiying * little bit rectification * Indentation rectification * Indentation rectification 1 * Apply suggestion from @picnixz Co-authored-by: Bénédikt Tran <[email protected]> * deprecated rectification * Remove mistakenly added NEWS entry * Update Doc/library/argparse.rst Co-authored-by: Bénédikt Tran <[email protected]> * changes python#1 Co-authored-by: Bénédikt Tran <[email protected]> * Update Doc/library/argparse.rst 2 Co-authored-by: Bénédikt Tran <[email protected]> * removed useless thing Co-authored-by: Bénédikt Tran <[email protected]> * changed according to the request Co-authored-by: Bénédikt Tran <[email protected]> * Updated the test part --------- Co-authored-by: Bénédikt Tran <[email protected]> Co-authored-by: Stan Ulbrych <[email protected]>
SonicField
added a commit
to SonicField/cpython
that referenced
this pull request
…ython#1) PyModule_Create stores a pointer to the PyModuleDef in the module object's md_def field. jit_module_noslotsdef was a stack-local in jit::initialize(), so md_def dangled after the function returned. GC's module_traverse then read from dead stack memory, causing heap corruption under sustained GC pressure (the 7/17 single-process crash). Fix: make jit_module_noslotsdef static so it survives the function call.
SonicField
added a commit
to SonicField/cpython
that referenced
this pull request
Push 44 W3 Step 5 (final): self-test script that validates the W3 R4
oracle's diagnostic capability via 3 falsifiers + synthetic injection.
Per theologian 2026-04-22 02:30:08Z W3 spec falsifier:
"inject a synthetic refcount bug into the C path — scripts/
rc_diff_oracle.sh produces non-empty output. If diff is empty under
injection, oracle is non-functional and W3 has not delivered the
diagnostic capability it promised."
Script structure:
--check mode: pre-conditions only (3 nm-based assertions)
default: pre-conditions + Phase A (clean diff = empty) + Phase B
(injected diff = non-empty) + post-restore verification
FALSIFIERS validated:
python#1 — production python has 0 rc_oracle symbols (RC_ORACLE undefined,
dispatcher block ABSENT from compiled binary). VERIFIED via
--check mode at HEAD a99db92: 'PASS: production python has 0
rc_oracle symbols'.
python#2 — libphoenix_rc_oracle.a exports rc_oracle_run T-symbol. VERIFIED:
'PASS: scratch lib exports rc_oracle_run (C entry point)'.
python#3 — python_rc_cpp has rc_oracle_run T-symbol. PENDING — operator must
run out-of-band link via the recipe in scripts/build_oracle.sh
tail (per supervisor 02:51:14Z option (c)).
INJECTION mechanism:
Phase B comments out the FIRST 'phx_rc_emit_incref' call site in
Python/jit/hir/refcount_pass_c.c via sed. Rebuilds C path. Runs
rc_diff_oracle.sh and asserts non-empty diff. Restores via backup +
rebuild. Final verification: post-restore diff is empty.
Injection target chosen for stability: phx_rc_emit_incref is a
high-frequency call across all refcount_insertion paths; commenting
one removes one Incref → guaranteed runtime divergence under
Py_REF_DEBUG (under-count → leak) AND under PYDEBUG (use-after-free).
Injection is reversible: cp $INJECT_TARGET.oracle_backup back +
rebuild. trap EXIT ensures restore even on script error.
Per supervisor 02:51:14Z option (c): out-of-band link of python_rc_cpp
is operator-driven (not embedded in this script), since W3 is a
one-shot oracle resurrection; no permanent Makefile.pre.in touch.
Pre-commit verification:
bash -n: SYNTAX OK
scripts/rc_oracle_self_test.sh --check: pre-conditions python#1, python#2 PASS;
python#3 expected to FAIL until operator runs the link recipe.
Push 44 W3 batch is now 2 commits (per supervisor 02:51:14Z amended):
a99db92 — W3 Steps 1-4 bundled (scratch lib + dispatcher + adapter)
THIS COMMIT — Step 5 self-test
ABBA cap 15 → 17. MANDATORY ABBA + auto-compile re-experiment after
push 44 lands per pre-authorization.
Process lesson applied (per supervisor 02:51:14Z): explicit
'git add scripts/rc_oracle_self_test.sh' (single file) + 'git diff
--cached --stat' verification BEFORE commit, to avoid the over-broad
staging that produced a99db92's bundled scope.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
ReviewersUh oh!
There was an error while loading. Please reload this page. AssigneesLabels
None yet
ProjectsMilestoneNo milestoneDevelopmentSuccessfully merging this pull request may close these issues. Uh oh!
There was an error while loading. Please reload this page. Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later. | ||||||||||||||||||||||||||||||||||||
| You are visitor 01402503 — GeezerVison does not store the pages you look up. Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required. |