Skip to content

Docs: Clarify that -I is not a security boundary#150790

Open
StanFromIreland wants to merge 2 commits into
python:mainfrom
StanFromIreland:I-doc-clarify
Open

Docs: Clarify that -I is not a security boundary#150790
StanFromIreland wants to merge 2 commits into
python:mainfrom
StanFromIreland:I-doc-clarify

Conversation

@StanFromIreland
Copy link
Copy Markdown
Member

No description provided.

Comment thread Doc/using/cmdline.rst

.. option:: -E

Ignore all ``PYTHON*`` environment variables, e.g.
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.

Should I also add the note here, I was split?

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.

I don't think it is necessary since the section also includes the "see also the -I option", but it is up to you and the core team

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd rather see a more positive description of what this is for - e.g. "this is useful for running commands that should not take into account the user's current environment settings, but does not necessarily mean that the current environment will have no effect." Once that's in, maybe it'll make sense to also say what it's not, but I don't think it's real helpful to jump straight to what it isn't without saying what it is.

@read-the-docs-community
Copy link
Copy Markdown

read-the-docs-community Bot commented Jun 2, 2026

Documentation build overview

📚 cpython-previews | 🛠️ Build #32957524 | 📁 Comparing fc4d7d7 against main (86867ed)

  🔍 Preview build  

3 files changed
± library/sys.html
± using/cmdline.html
± whatsnew/changelog.html

Comment thread Doc/using/cmdline.rst Outdated
Co-authored-by: Bartosz Sławecki <bartosz@ilikepython.com>
@StanFromIreland
Copy link
Copy Markdown
Member Author

@zooba, any thoughts here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

4 participants