Fix preparsing by using getopt.getopt #24

Closed
wants to merge 11 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

oscarbenjamin commented Apr 25, 2012

Ok, so this patch keeps the tests and everything from the patch in GH-23 but it uses getopt again and actually reduces the number of lines in opster.py.

It turns out preparsing should just use getopt instead of gnu_getopt. This way options are only parsed up to the first non-option argument (the subcommand name). If only global options are allowed before the command, name then it will be able to parse those arguments without error.

Owner

piranha commented Apr 26, 2012

What do you think about removing unnecessary commits? :) I'm talking mostly about those before 4a8c370 (those which are not about tests or docs).

@piranha piranha closed this May 3, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment