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

Bug: wrong documentation for git-fast-import's option command

From
TBTomi Belan <tomi.belan@gmail.com>
Date
Apr 30, 2017, 00:07 UTC
Message-ID
<CACUV5oeKoVdncK3RjtusiqrACj1xLiS--z8vC+Rsvy5TF09Lsw@mail.gmail.com>

The man page of git-fast-import says the syntax is "'option' SP <option> LF". But this is wrong, and commands like that will be silently ignored.

fast-import.c only parses commands starting with "option git ", and ignores others starting with "option ". See https://github.com/git/git/blob/027a3b943b444a3e3a76f9a89803fc10245b858f/fast-import.c#L3487-L3490 .

The correct syntax is e.g. "option git stats" or "option git big-file-threshold=123". The docs should say "'option' SP 'git' SP <option> LF" or maybe even "'option' SP 'git' SP <option> ('=' <argument>)? LF" (similarly to the docs for feature), and explain this is meant to allow other consumers of the fast import format.

Looks like this bug existed since the option command was added in 9c8398f in 2009. Am I the first one who tried to use it? ;-)

Cheers,
Tomi
Next: Andreas Schwab
Message 1 of 3 in “Bug: wrong documentation for git-fast-import's option command”
  1. Tomi BelanApr 30, 2017
  2. Andreas SchwabApr 30, 2017
  3. Tomi BelanMay 1, 2017

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.