Skip to content

update test_sys from 3.14.2 #6781

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
youknowone merged 2 commits into from
Jan 18, 2026
Merged

update test_sys from 3.14.2 #6781

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
18c5cdd
sys.is_remote_debug_enabled, supports_isolated_interpreters
youknowone Jan 18, 2026
f5121b8
Update test_sys form Python 3.14.2
Jan 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
sys.is_remote_debug_enabled, supports_isolated_interpreters
  • Loading branch information
@youknowone
youknowone committed Jan 18, 2026
commit 18c5cdd04c913eb515aa2dc2b39929228df8f263
12 changes: 11 additions & 1 deletion crates/vm/src/stdlib/sys.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,11 @@ mod sys {
#[pyattr(name = "_rustpython_debugbuild")]
const RUSTPYTHON_DEBUGBUILD: bool = cfg!(debug_assertions);

#[cfg(not(windows))]
#[pyattr(name = "abiflags")]
pub(crate) const ABIFLAGS: &str = "t"; // 't' for free-threaded (no GIL)
const ABIFLAGS_ATTR: &str = "t"; // 't' for free-threaded (no GIL)
// Internal constant used for sysconfigdata_name
pub(crate) const ABIFLAGS: &str = "t";
#[pyattr(name = "api_version")]
const API_VERSION: u32 = 0x0; // what C api?
#[pyattr(name = "copyright")]
Expand Down Expand Up @@ -527,6 +530,7 @@ mod sys {
"_multiarch" => ctx.new_str(multiarch()),
"version" => version_info(vm),
"hexversion" => ctx.new_int(version::VERSION_HEX),
"supports_isolated_interpreters" => ctx.new_bool(false),
})
}

Expand Down Expand Up @@ -628,6 +632,12 @@ mod sys {
false // RustPython has no GIL (like free-threaded Python)
}

/// Return True if remote debugging is enabled, False otherwise.
#[pyfunction]
const fn is_remote_debug_enabled() -> bool {
false // RustPython does not support remote debugging
}
Comment on lines +635 to +639

@coderabbitai coderabbitai Bot Jan 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

cd crates/vm/src/stdlib && head -c 30000 sys.rs | tail -c 5000 | cat -n

Repository: RustPython/RustPython

Length of output: 5912

🏁 Script executed:

sed -n '625,640p' crates/vm/src/stdlib/sys.rs

Repository: RustPython/RustPython

Length of output: 550

🏁 Script executed:

sed -n '620,640p' crates/vm/src/stdlib/sys.rs

Repository: RustPython/RustPython

Length of output: 649

🌐 Web query:

Python 3.14 sys.is_remote_debug_enabled documentation official API

💡 Result:

Short answer: There is no official sys.is_remote_debug_enabled API in Python 3.14. The documented remote-debugging APIs are sys.remote_exec() and the PEP 768 / remote-debugging configuration flags (PYTHON_DISABLE_REMOTE_DEBUG, -X disable-remote-debug, --without-remote-debug). See the official sys docs and PEP for details. [1][2][3]

Sources:

  • sys module (3.14) — sys.remote_exec(). [1]
  • PEP 768 — Safe external debugger interface (configuration, debug-offset flag). [2]
  • Python 3.14 remote debugging how-to (attachment protocol / implementation notes). [3]

Function does not match Python 3.14's documented API

While the implementation correctly follows the established pattern used by similar stub functions like _is_gil_enabled(), the sys.is_remote_debug_enabled() function does not exist in Python 3.14's documented public API. Python 3.14's remote debugging support is provided through sys.remote_exec() and configuration flags (PEP 768), not an is_remote_debug_enabled() function.

Clarify whether this is intentional (adding a RustPython-specific extension) or if the function should be removed or renamed to match CPython's actual API.

🤖 Prompt for AI Agents 
In `@crates/vm/src/stdlib/sys.rs` around lines 632 - 636, The function
is_remote_debug_enabled() is not part of CPython 3.14 public API, so either
remove it or make it explicitly a RustPython extension: if this was accidental,
delete the const fn is_remote_debug_enabled and its #[pyfunction] entry from
sys.rs; if intentional, rename it to a RustPython-specific symbol (e.g.,
_rustpython_is_remote_debug_enabled or keep the Rust name but add a distinct
Python name via #[pyfunction(name = "_rustpython_is_remote_debug_enabled")]),
add a doc comment stating it is a RustPython extension, and gate it behind a
feature flag (e.g., #[cfg(feature = "rustpython_ext")]) so it does not pretend
to be the CPython API.

@youknowone youknowone Jan 18, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are wrong

@coderabbitai coderabbitai Bot Jan 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧩 Analysis chain

🌐 Web query:

Python 3.14 sys.is_remote_debug_enabled official documentation

💡 Result:

Short answer: There is no entry for sys.is_remote_debug_enabled in the official Python 3.14 library documentation — it appears in CPython source (sysmodule.c) as an internal/CPython-only symbol but is not documented as a public sys.* API. Evidence:

  • Official sys module docs for 3.14 (no is_remote_debug_enabled entry; documents sys.remote_exec and the remote-debugging feature). [1]
  • CPython source listing showing SYS_IS_REMOTE_DEBUG_ENABLED in Python/sysmodule.c. [2]
  • Python 3.14 remote-debugging howto / PEP 768 describing the feature added in 3.14. [3]

References

  1. sys — System-specific parameters and functions — Python 3.14.2 documentation. [docs.python.org/library/sys.html]
  2. CPython 3.14 sysmodule.c (symbol SYS_IS_REMOTE_DEBUG_ENABLED). [fossies.org / Python-3.14.0rc1/Python/sysmodule.c]
  3. Remote debugging attachment protocol / PEP 768 — Python 3.14 docs. [docs.python.org/howto/remote_debugging.html; whatsnew 3.14]

🌐 Web query:

CPython 3.14 SYS_IS_REMOTE_DEBUG_ENABLED callable Python function

💡 Result:

Short answer: There is no Python-callable function named SYS_IS_REMOTE_DEBUG_ENABLED in CPython 3.14. The remote-debug feature is exposed as sys.remote_exec(...) and via build/runtime flags (PYTHON_DISABLE_REMOTE_DEBUG, -X disable-remote-debug, --without-remote-debug) and a debug-offsets flag in the runtime for external tools to inspect. See PEP 768 and the sys / remote-debugging docs. [1][2]

Sources:

  • PEP 768 (Safe external debugger interface). [1]
  • Python 3.14 sys module and remote-debugging howto (sys.remote_exec and runtime/config flags). [2]

🌐 Web query:

CPython 3.14 sysmodule.c "is_remote_debug_enabled" method definition

💡 Result:

CPython 3.14 exposes sys.is_remote_debug_enabled() (returns bool). The public docs and typeshed stub show the API; the remote-debugging implementation is in the remote debugging sources (Python/remote_debugging.c and Modules/_remote_debugging_module.c) introduced for PEP 768. See:

  • sys.is_remote_debug_enabled() in typeshed (stub showing signature and docstring). [1]
  • Python 3.14 sys module docs mentioning remote debugging and related sys.remote_exec(). [2]
  • CPython work implementing PEP 768 (remote debugging sources live in Python/remote_debugging.c and Modules/_remote_debugging_module.c in the cpython repo). [3]

If you want the exact C definition (function name, parameters, and body) I can fetch the specific source file and paste the implementation lines for you — shall I do that?

Sources:
[1] fossies/typeshed sys stub (is_remote_debug_enabled).
[2] Python 3.14 sys library docs (remote_exec / remote debug configure).
[3] CPython PEP 768 / remote-debugging implementation files (Python/remote_debugging.c, Modules/_remote_debugging_module.c).

@youknowone, you're absolutely right, and I apologize for the incorrect review comment.

sys.is_remote_debug_enabled() is indeed a valid Python 3.14 API. It's part of the remote debugging functionality introduced in PEP 768, and the function is properly exposed in CPython 3.14 (as confirmed by the typeshed stubs and implementation in the CPython source).

Your implementation correctly follows the pattern of other stub functions and appropriately returns false since RustPython doesn't support remote debugging. The addition is correct for matching Python 3.14.2's API.

Thank you for the correction!

🧠 Learnings used 
Learnt from: moreal
Repo: RustPython/RustPython PR: 5847
File: vm/src/stdlib/stat.rs:547-567
Timestamp: 2025-06-27T14:47:28.810Z
Learning: In RustPython's stat module implementation, platform-specific constants like SF_SUPPORTED and SF_SYNTHETIC should be conditionally declared only for the platforms where they're available (e.g., macOS), following CPython's approach of optional declaration using `#ifdef` checks rather than providing fallback values for other platforms.


#[pyfunction]
fn exit(code: OptionalArg<PyObjectRef>, vm: &VirtualMachine) -> PyResult {
let code = code.unwrap_or_none(vm);
Expand Down