git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v3 2/3] fast-import: add option command

From
Sverre Rabbelier <srabbelier@gmail.com>
Date
Aug 14, 2009, 17:43 UTC
Message-ID
<fabb9a1e0908141043l35d2070bxdadae96a7dd8a232@mail.gmail.com>
In-Reply-To
<20090814173929.GU1033@spearce.org>
Heya,
On Fri, Aug 14, 2009 at 10:39, Shawn O. Pearce<spearce@spearce.org> wrote:
> Sorry, lack of caffeine.  Rereading the docs makes sense.

Perfectly acceptable, I totally though the 'break' in a 'switch' statement would break out of the outer for loop the other day.

Show 6 quoted lines
>> > So yea, I really do think its a good idea for command line options
>> > to override stream options, despite what Dscho may think. ?:-)
>>
>> Which is what the current version does, yes?
>
> Yes.  :-)

Ok, I think perhaps we can wait 'nother day to see if anyone notices anything (possibly after more caffeine) and then ask Junio to apply?

-- 
Cheers,

Sverre Rabbelier
Previous: Shawn O. Pearce
Message 8 of 8 in “fast-import: add a new option command”
  1. 0/3 fast-import: add a new option commandSverre Rabbelier, Aug 13, 2009
  2. 1/3 fast-import: put option parsing code in seperate functionsSverre Rabbelier, Aug 13, 2009
  3. 2/3 fast-import: add option commandSverre Rabbelier, Aug 13, 2009
  4. 3/3 fast-import: test the new option commandSverre Rabbelier, Aug 13, 2009
  5. Shawn O. PearceAug 14, 2009
  6. Sverre RabbelierAug 14, 2009
  7. Shawn O. PearceAug 14, 2009
  8. Sverre RabbelierAug 14, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.