Re: [PATCH 01/25] Add a simple option parser.
On Wed, Oct 17, 2007 at 07:24:19AM +0000, Shawn O. Pearce wrote:
Show 18 quoted lines
> Pierre Habouzit <madcoder@debian.org> wrote:
> > The option parser takes argc, argv, an array of struct option
> > and a usage string. ...
>
> OK, I've chewed down some version of this series. ;-)
>
> To be more specific I fetched ph/parseopt (11b83dc4da) from your
> tree at git://git.madism.org/git.git and split it apart somewhat.
> All of the patches were rebased onto my most recent master but I
> also yanked the two that impacted the builtin-fetch series out and
> layered them over a merge of db/fetch-pack and my version of your
> ph/parseopt series.
>
> Why? Well I want to keep our options open about which series
> graduates to master first. Although builtin-fetch has been cooking
> for a while there's been a number of issues with that code.
> There exists a (perhaps small) chance that ph/parseopt will be
> ready before db/fetch-pack.
Yes, this makes sense, otoh the migration of the fetch commands are
really independant so you can put those appart, even if I end up needing
to rewrite them it's not an issue.
> Currently ph/parseopt is in pu. Tomorrow I'll look at the usage
> strings in more depth and see if any improvements can be easily
> made. I already made one suggested by Dscho in builtin-add.
--
·O· Pierre Habouzit
··O madcoder@debian.org
OOO http://www.madism.org
Pierre Habouzit <madcoder@debian.org>