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 — Translate WASI signal dispositions through libc sentinels by youknowdot · Pull Request #9024 · RustPython/RustPython Conversation
Contributor
ScopeTranslate Python's WASI uses function-table pointers for its signal sentinels. Treating them as the usual integer values lets an ignored synchronous signal trap and lets an uncatchable signal registration appear to succeed. A typed host boundary keeps Python's numeric values distinct from libc's default, ignored, and custom handlers. This is a narrow Python 3.14 compatibility extraction from #8954. It does not include the separate wakeup-fd error-reporting changes. Regression coverageThe existing signal snippet adds synchronous custom delivery, replacement with VerificationActual RustPython modules were built from current main bd8fdb7 and the isolated commit 59f9e3c, using the same source station, official Under Wasmtime 49.0.0:
The updated snippet also passes the repository pytest runner on CPython 3.14.7 and the Linux RustPython baseline. Normal commit hooks pass. Native Linux Clippy with warnings denied, the workspace compile-only aggregate, and the separate C-API compile-only aggregate all pass on the exact isolated source tree. Broad runtime suites were deliberately not run during this extraction. Reference implementations: CPython 3.14 signalmodule.c, wasi-libc signal constants, and wasi-libc signal implementation. AI assistance: OpenAI Codex assisted with source comparison, implementation, regression design, and automated verification under the user's direction. No manual validation is claimed.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsKeep Python disposition values distinct from WASI function-table pointers. Recognize wasi-libc SIG_ERR before updating Python handlers and preserve ignored/default/custom dispositions. Add synchronous signal and rejected-registration regressions to the existing snippet. AI assistance covered source investigation, implementation, and automated reference/baseline tests. Isolated patched runtime validation is pending. No human validation is claimed. Assisted-by: Codex:model-version-unavailable
|
There was an error while loading. Please reload this page.
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Merging this PR will not alter performance
Comparing Footnotes
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
cc @jiwahn
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: 59f9e3ccd7
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| previous = signal.signal(signal.SIGILL, synchronous_handler) | ||
| try: | ||
| signal.raise_signal(signal.SIGILL) |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Remove the added signal test block
This starts a new signal exercise and the following assertions in an existing test file, but the repository's test-code policy permits only adding an expected-failure decorator or removing one and its TODO once the test passes, and explicitly forbids modifying test logic or assertions. Remove this added snippet coverage rather than editing this test file.
AGENTS.md reference: AGENTS.md:L275-L279
Useful? React with 👍 / 👎.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.