Re: [PATCH] Add an optional <mode> argument to commit/status -u|--untracked-files option
Quoting Marius Storm-Olsen <marius@trolltech.com> writes:
Show 27 quoted lines
> Jeff King said the following on 03.06.2008 22:14:
>> On Tue, Jun 03, 2008 at 03:09:10PM +0200, Marius Storm-Olsen wrote:
>>
>>> +-u[<mode>]|--untracked-files[=<mode>]::
>>> + Show all untracked files.
>>> + The mode parameter is optional, and is used to specify
>>> + the handling of untracked files. The possible options are:
>>> + none - Show no untracked files
>>> + normal - Shows untracked files and directories
>>> + all - Also shows individual files in untracked directories.
>>> + If the mode parameter is not specified, the defaults is
>>> + 'all'.
>>
>> Hmm. Doesn't this change bundling semantics of "git commit -us"? Do we
>> care?
>
> You are right. This is the joy of the parse-options() handling of
> short-options with optional arguments.
>
> -us before meant:
> 1) commit
> 2) show all untracked files
> 3) sign-of the commit
>
> I guess it would be possible to reparse the options without the -u, if
> the argument is not one of the three (none,normal,all), but I'm not
> sure it's _that_ critical. Opinions?
Isn't the second bullet point in the description in:
http://www.kernel.org/pub/software/scm/git/docs/gitcli.html
good enough?
--
Nanako Shiraishi
http://ivory.ap.teacup.com/nanako3/
----------------------------------------------------------------------
Find out how you can get spam free email.
http://www.bluebottle.com/tag/3