unknown upper case commands throw an exception #1561

Closed
qwcode opened this Issue Feb 11, 2014 · 1 comment

1 participant

@qwcode

pip Yoyo

  File "/home/qwcode/.qwdev/cm/bin/pip", line 11, in <module>
    sys.exit(main())
  File "/home/qwcode/.qwdev/cm/local/lib/python2.7/site-packages/pip/__init__.py", line 177, in main
    cmd_name, cmd_args = parseopts(initial_args)
  File "/home/qwcode/.qwdev/cm/local/lib/python2.7/site-packages/pip/__init__.py", line 156, in parseopts
    cmd_args.remove(args_else[0].lower())
ValueError: list.remove(x): x not in list

this should fail like lower case unknown commands: "ERROR: unknown command"

@qwcode qwcode added the bug label Feb 11, 2014
@ddasilva ddasilva added a commit that referenced this issue Mar 8, 2014
@ddasilva ddasilva Fix exception in commands that contain capital letters.
Fixes #1561. Removes an erroneous call to .lower() and adds a regression test.
d79cf2f
@ddasilva ddasilva added a commit that referenced this issue Mar 8, 2014
@ddasilva ddasilva Fix exception in commands that contain capital letters.
Fixes #1561. Removes an erroneous call to .lower() and adds regression tests.
24eebc1
@ddasilva ddasilva added a commit that referenced this issue Mar 8, 2014
@ddasilva ddasilva Fix exception in unknown commands that contain capital letters.
Fixes #1561. Removes an erroneous call to .lower() and adds regression tests.
0c0e9ab
@qwcode

fixed in develop. closing.

@qwcode qwcode closed this Oct 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment