On Wed, Oct 17, 2007 at 07:24:19AM +0000, Shawn O. Pearce wrote: > Pierre Habouzit <madcoder@xxxxxxxxxx> 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. wonderful, thanks. -- ·O· Pierre Habouzit ··O madcoder@xxxxxxxxxx OOO http://www.madism.org
Attachment:
pgpJTx6JyAsJr.pgp
Description: PGP signature
