This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Tiny patch to cmdline docs
Type: Stage:
Components: Documentation Versions: Python 2.6
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: belopolsky, georg.brandl, tim.golden
Priority: normal Keywords: patch

Created on 2008-03-05 14:31 by tim.golden, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
doc-using-cmdline-r61249.patch tim.golden, 2008-03-05 14:31 Patch against r62149 of cmdline.rst
Messages (8)
msg63276 - (view) Author: Tim Golden (tim.golden) * (Python committer) Date: 2008-03-05 14:31
The docs for the PYTHONPATH var indicate that its items are separated by
colons. In fact they're separated by whatever's customary for the O/S.
Patch attached.
msg63281 - (view) Author: Alexander Belopolsky (belopolsky) * (Python committer) Date: 2008-03-05 15:07
PYTHONPATH variable is likely to be defined by sysadmins who may not know 
what os.pathsep is.  Maybe it is better to say "OS-dependent separator 
(';' on Windows, ':' on Linux and other UNIX-like OSes or ',' on some 
less-known systems.)"
msg63282 - (view) Author: Tim Golden (tim.golden) * (Python committer) Date: 2008-03-05 15:12
Alexander Belopolsky wrote:
> Alexander Belopolsky added the comment:
> 
> PYTHONPATH variable is likely to be defined by sysadmins who may not know 
> what os.pathsep is.  Maybe it is better to say "OS-dependent separator 
> (';' on Windows, ':' on Linux and other UNIX-like OSes or ',' on some 
> less-known systems.)"

Maybe, but I have made the doc reference a link to the os.pathsep
doc which basically says what you just did.
msg63286 - (view) Author: Alexander Belopolsky (belopolsky) * (Python committer) Date: 2008-03-05 16:27
> .. but I have made the doc reference a link to the os.pathsep

I knew you would say that :-).  I was making my comment out of real life 
experience: sysadmins rarely know python language, but can do good job 
administering python installations.  For better or worse, the target 
audience for PYTHONPATH documentation is likely not to be interested in 
being educated in the python programming language.  Having to lookup the 
meaning of os.pathsep just to understand the format of PYTHONPATH is a distraction. Another problem with the link is that cmdline page is 
likely to be printed out for a quick reference and the link will 
obviously not work in the printout.
msg63287 - (view) Author: Tim Golden (tim.golden) * (Python committer) Date: 2008-03-05 16:32
Alexander Belopolsky wrote:
> Alexander Belopolsky added the comment:
> 
>> .. but I have made the doc reference a link to the os.pathsep
> 
> I knew you would say that :-).  I was making my comment out of real life 
> experience: sysadmins rarely know python language, but can do good job 
> administering python installations.  For better or worse, the target 
> audience for PYTHONPATH documentation is likely not to be interested in 
> being educated in the python programming language.  Having to lookup the 
> meaning of os.pathsep just to understand the format of PYTHONPATH is a distraction. Another problem with the link is that cmdline page is 
> likely to be printed out for a quick reference and the link will 
> obviously not work in the printout.

I don't feel strongly about it. Feel free to propose an alternative
wording for the patch :)
msg63289 - (view) Author: Alexander Belopolsky (belopolsky) * (Python committer) Date: 2008-03-05 16:52
> Feel free to propose an alternative wording for the patch 

I thought I already did in my first post.  The complete sentence should 
read:

"""
The format is the same as the shell's :envvar:`PATH`: one or more 
directory pathnames separated by an OS-dependent character (';' on 
Windows, ':' on Linux and other UNIX-like OSes or ',' on some 
less-known systems.)
"""
msg63290 - (view) Author: Tim Golden (tim.golden) * (Python committer) Date: 2008-03-05 17:01
Alexander Belopolsky wrote:
> Alexander Belopolsky added the comment:
> 
>> Feel free to propose an alternative wording for the patch 
> 
> I thought I already did in my first post.  The complete sentence should 
> read:
> 
> """
> The format is the same as the shell's :envvar:`PATH`: one or more 
> directory pathnames separated by an OS-dependent character (';' on 
> Windows, ':' on Linux and other UNIX-like OSes or ',' on some 
> less-known systems.)
> """

Ah. Lazily, I meant: feel free to upload a patch containing the
wording you think good.

TJG
msg63292 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2008-03-05 19:32
I took the best of both worlds and committed in r61255.
History
Date User Action Args
2022-04-11 14:56:31adminsetgithub: 46492
2008-03-05 19:32:06georg.brandlsetstatus: open -> closed
resolution: fixed
messages: + msg63292
2008-03-05 17:01:12tim.goldensetmessages: + msg63290
2008-03-05 16:52:19belopolskysetmessages: + msg63289
2008-03-05 16:32:18tim.goldensetmessages: + msg63287
2008-03-05 16:27:19belopolskysetmessages: + msg63286
2008-03-05 15:12:06tim.goldensetmessages: + msg63282
2008-03-05 15:07:50belopolskysetnosy: + belopolsky
messages: + msg63281
2008-03-05 14:31:29tim.goldencreate