Roundup Robot added the comment:
New changeset ff9636af9505 by Terry Jan Reedy in branch '3.2':
Issue #17415: Clarify 'this' referent by moving containing sentence just after
http://hg.python.org/cpython/rev/ff9636af9505
New changeset bceb81b0016e by Terry Jan Reedy in branch '2.7':
Issue
Terry J. Reedy added the comment:
If foo is a symbolic link, changing A/foo/../B to A/B could change the
meaning, so I am sure the rewrite is correct. I used Ezio's version with a few
more changes. I particular, I changed 'This collapsing', which is slightly
awkward anyway, to 'This string
Serhiy Storchaka added the comment:
os.path.normpath() works not only with strings but with bytes objects too.
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17415
Terry J. Reedy added the comment:
The top of the os doc makes it clear that path = byte string or unicode string
and I meant 'string' in that generalized sense. I think 'string' is used that
way elsewhere in the 3.x docs. I though of 'text manipulation', but as the doc
again makes clear, unix
Changes by Terry J. Reedy tjre...@udel.edu:
--
assignee: docs@python - terry.reedy
stage: needs patch - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17415
Gurmeet Singh added the comment:
At the moment, I running late for creating a program - so I apologize that
immediately I would not able to work on creating a patch. Later though I may be
able to. I am sorry.
To avoid wasted work I think patch should be created only after someone (do not
Ezio Melotti added the comment:
I will correct that if you make me aware about it!
Try to use meaningful titles, e.g. Clarify docs of os.path.normpath(). See
http://docs.python.org/devguide/triaging.html for more informations.
Also if you could provide patches that would be even better.
Terry J. Reedy added the comment:
The question is what is the 'this' that may change the patch meaning. It does
not seem to me that changing '/' to '\' would do that. So I suspect 'this'
refers to 'collapsing'. If so, I suggest the entry be:
Normalize a pathname (but not the case -- use
Ezio Melotti added the comment:
Your suggestion -- assuming it's accurate -- looks good to me, however I would
get rid of the exclamation mark and move the link to normcase at the end:
Normalize a pathname by collapsing redundant separators and up-level references
so that A//B, A/B/, A/./B