[issue33095] Cross-reference isolated mode from relevant locations

[Karthikeyan Singaravelan]

Karthikeyan Singaravelan  added the comment:

Closing since PRs were merged. Thanks for the review.

--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33095] Cross-reference isolated mode from relevant locations

[Julien Palard]

Julien Palard  added the comment:


New changeset 210dc3bb37ec57a8b8b16ffe0881181138a6e973 by Julien Palard in 
branch '3.7':
[3.7] bpo-33095: Add reference to isolated mode in -m and script option 
(GH-7764) (GH-16181)
https://github.com/python/cpython/commit/210dc3bb37ec57a8b8b16ffe0881181138a6e973


--
nosy: +mdk

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33095] Cross-reference isolated mode from relevant locations

[miss-islington]

miss-islington  added the comment:


New changeset 07186c3959a61c3d73b5ccae431cdd32afe70e5e by Miss Islington (bot) 
in branch '3.8':
bpo-33095: Add reference to isolated mode in -m and script option (GH-7764)
https://github.com/python/cpython/commit/07186c3959a61c3d73b5ccae431cdd32afe70e5e


--
nosy: +miss-islington

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33095] Cross-reference isolated mode from relevant locations

[Julien Palard]

Change by Julien Palard :


--
pull_requests: +15791
pull_request: https://github.com/python/cpython/pull/16181

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33095] Cross-reference isolated mode from relevant locations

[miss-islington]

Change by miss-islington :


--
pull_requests: +15790
pull_request: https://github.com/python/cpython/pull/16180

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33095] Cross-reference isolated mode from relevant locations

[Nick Coghlan]

Nick Coghlan  added the comment:


New changeset bdd6945d4dbd1fe6a7fcff95f7d6908db7d791a1 by Nick Coghlan (Xtreak) 
in branch 'master':
bpo-33095: Add reference to isolated mode in -m and script option (GH-7764)
https://github.com/python/cpython/commit/bdd6945d4dbd1fe6a7fcff95f7d6908db7d791a1


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33095] Cross-reference isolated mode from relevant locations

[Karthikeyan Singaravelan]

Change by Karthikeyan Singaravelan :


--
pull_requests: +7372
stage: needs patch -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33095] Cross-reference isolated mode from relevant locations

[Nick Coghlan]

Nick Coghlan  added the comment:

xtreak: if you're familiar with GitHub workflows, then we now accept docs and 
code updates as GitHub PRs: https://devguide.python.org/pullrequest/

The parts about running the tests aren't exactly relevant here, what's more 
relevant is building the documentation: 
https://devguide.python.org/documenting/#building-the-documentation

In relation to your draft patch, I think that's a good starting point, but for 
-c and -m, it's the current directory that gets omitted, rather than "the 
script's directory" (that isn't well defined when anything other than a path is 
passed in).

For making "-" and 

[issue33095] Cross-reference isolated mode from relevant locations

[Karthikeyan Singaravelan]

Karthikeyan Singaravelan  added the comment:

There are four parts where I could see sys.path manipulation being mentioned 
while running scripts : 

-c option (https://docs.python.org/3/using/cmdline.html#cmdoption-c)
-m option (https://docs.python.org/3/using/cmdline.html#cmdoption-m)
-  (input option) 

[issue33095] Cross-reference isolated mode from relevant locations

[Jakub Wilk]

Change by Jakub Wilk :


--
nosy:  -jwilk

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33095] Cross-reference isolated mode from relevant locations

[Nick Coghlan]

New submission from Nick Coghlan :

In https://bugs.python.org/issue33053#msg313966, jwilk noted that it isn't 
obvious from https://docs.python.org/3/using/cmdline.html#cmdoption-m how to 
keep the current directory from being added to `sys.path` when using the -m 
switch. The answer is to pass the `-I` switch as well (to activate isolated 
mode), but there's no cross reference to help readers discover that fact.

https://docs.python.org/3/using/cmdline.html#id2 is the main documentation for 
isolated mode, so the steps needed to close this issue are:

1. At least add a reference from the -m switch documentation to the -I switch 
documentation
2. Review the other parts of the `using` docs that describe how `sys.path` is 
initialised, and reference the -I switch documentation where relevant

--
assignee: docs@python
components: Documentation
keywords: easy
messages: 314022
nosy: docs@python, jwilk, ncoghlan
priority: normal
severity: normal
stage: needs patch
status: open
title: Cross-reference isolated mode from relevant locations
type: enhancement
versions: Python 3.6, Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com