Repository navigation
Conversation
Register files read by py_compile! and py_freeze! with include_bytes! dependency markers, preserving existing compilation behavior. Add a standalone Cargo regression for existing inputs and documented boundaries. OpenAI dot/Codex assisted with investigation, implementation, regression utilities, test execution, automated review, and this commit message. The exact model identifier was not exposed. Human review and manual verification remain pending for this fork-only review draft. Assisted-by: OpenAI Codex:model identifier unavailable
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Track files read by
py_compile!andpy_freeze!with inert, fully qualifiedinclude_bytes!dependency markers so editing an existing embedded Python input triggers a Cargo rebuild. Includes recursive frozen-directory inputs, skipped invalid inputs, and a standalone regression driver without a consumer build script. This is the scoped fix for upstream issue 5065.This fork-only draft is for human review in
moreal/RustPython. It contains exactly three files and one commit based on06007c81fa99e813cbf4583dab326764ba0ef017, the freshly verified targetmainand the exact fully validated base. No source adaptation was required.Validation on this base
test_compile: 185 run, 51 skipped.The supplied source patch SHA256 is
4c183532977e8f8781975635e00da1eb7742d702e7d93665fbd79c5b0aa362ff.Run the standalone regression from the patched checkout (new output directory; Cargo dependencies already cached):
python3 extra_tests/embedded_macro_rebuilds.py --repo . --output /tmp/embedded-rebuilds-NEWSupplemental checks on the fork's earlier
7cd4a54ebase also confirmed the original stale behavior (56/56 checks in--expect-stalemode), followed by 61/61 fixed-regression checks, configured hooks, and focused Clippy. Those results are separate from the current-base validation above.Known tracking boundaries
Validation is Linux-only with Rust 1.99.0. Windows/macOS execution, human/manual verification, and separate C/Python C-API integration beyond Cargo tests remain unperformed. This draft is not a ready-to-merge claim.
AI assistance
OpenAI dot/Codex assisted with investigation, implementation, regression utilities, test execution, automated review, and this description. The exact model identifier was not exposed, so none is asserted. Human review/use and the upstream AI-policy submission gate remain pending; policy compliance is not claimed by opening this review draft in the requested fork.