process
Status: |
closed |
Resolution: |
fixed |
Dependencies: |
|
Superseder: |
|
Assigned To: |
orsenthil |
Nosy List: |
alexis, brett.cannon, docs@python, dstufft, eric.araujo, ezio.melotti, lemburg, ncoghlan, orsenthil, paul.moore, python-dev, tarek, zach.ware |
Priority: |
normal |
Keywords: |
patch |
Created on 2016-01-05 17:43 by brett.cannon, last changed 2022-04-11 14:58 by admin. This issue is now closed.
Files |
|
|
|
File name |
Uploaded |
Description |
Edit |
Issue26017.patch |
orsenthil,2016-01-08 07:47 |
|
|
Messages (7) |
|
|
msg257536 - (view) |
Author: Brett Cannon (brett.cannon) *  |
Date: 2016-01-05 17:43 |
If you look at https://docs.python.org/3/installing/index.html it lists two commands: python -m pip install SomePackage==1.0.4 # specific version python -m pip install 'SomePackage>=1.0.4' # minimum version If you notice that beyond the change from `==` to `>=`, you will notice one quotes its argument while the other one doesn't. This is a UNIX shell thing due to what `>` means. But if you don't know how the UNIX shell works this could be easily overlooked. It would be best to simply quote both examples and avoid people messing up by leaving off the quotes. |
|
|
msg257537 - (view) |
Author: Zachary Ware (zach.ware) *  |
Date: 2016-01-05 17:55 |
I'd also suggest using double quotes ("); single quotes don't work on Windows. C:\>echo 'test>=test' C:\>dir /b test* test' C:\>type "test'" 'test |
|
|
msg257740 - (view) |
Author: Senthil Kumaran (orsenthil) *  |
Date: 2016-01-08 07:47 |
Here is the patch with the changes suggested by Brett. Having created this patch, I had second thoughts on this change. Folks who use pip often, usually use it without any quotes, like `pip install requests`, `pip install CherryPy`. Pinned down versions are most often required by projects and they mention it in the requirements.txt in an unquoted manner Ref: https://pip.readthedocs.org/en/1.1/requirements.html It works fine as it is not the shell which is parsing this file. Since ">" thing parsed by shell on the command line is a special thing, why not address that scenario with more clarification, instead of quoting all the examples, which seem contrary to the common usage patterns? |
|
|
msg257769 - (view) |
Author: Ezio Melotti (ezio.melotti) *  |
Date: 2016-01-08 17:41 |
I think it's enough to add a note that says that if the argument contains > (and possibly other) special characters, then it needs to be quoted. Examples that use > should also be quoted, but not the others one. |
|
|
msg258490 - (view) |
Author: Roundup Robot (python-dev)  |
Date: 2016-01-18 02:42 |
New changeset 31debd0dc0dc by Senthil Kumaran in branch '3.5': Issue26017 - Suggest enclosing command args in double quotes when using characters which get interpreted by shell. https://hg.python.org/cpython/rev/31debd0dc0dc New changeset 18ae23b69e08 by Senthil Kumaran in branch 'default': Merge from 3.5 https://hg.python.org/cpython/rev/18ae23b69e08 |
|
|
msg258491 - (view) |
Author: Roundup Robot (python-dev)  |
Date: 2016-01-18 02:45 |
New changeset 668827be66d6 by Senthil Kumaran in branch '2.7': Issue26017 - Suggest enclosing command args in double quotes when using characters which get interpreted by shell. https://hg.python.org/cpython/rev/668827be66d6 |
|
|
msg258492 - (view) |
Author: Senthil Kumaran (orsenthil) *  |
Date: 2016-01-18 02:46 |
Thanks for the review/comment Ezio. Agree with your suggestion and made the change accordingly. |
|
|
History |
|
|
|
Date |
User |
Action |
Args |
2022-04-11 14:58:25 |
admin |
set |
github: 70205 |
2016-01-18 02:46:31 |
orsenthil |
set |
status: open -> closedversions: + Python 2.7, Python 3.5, Python 3.6messages: + assignee: docs@python -> orsenthilresolution: fixedstage: needs patch -> resolved |
2016-01-18 02:45:17 |
python-dev |
set |
messages: + |
2016-01-18 02:43:00 |
python-dev |
set |
nosy: + python-devmessages: + |
2016-01-08 17:41:36 |
ezio.melotti |
set |
nosy: + ezio.melottimessages: + |
2016-01-08 07:47:30 |
orsenthil |
set |
files: + Issue26017.patchnosy: + orsenthilmessages: + keywords: + patch |
2016-01-05 17:55:43 |
zach.ware |
set |
nosy: + zach.waremessages: + |
2016-01-05 17:43:16 |
brett.cannon |
create |
|