Re: [bug] Git submodule command interprets switch as argument and switch

[R0b0t1 <r030t1@gmail.com>]

On Fri, Aug 18, 2017 at 3:38 PM, Jonathan Nieder <jrnieder@gmail.com> wrote:
> Hi,
>
> R0b0t1 wrote:
>
>> The issue is as follows:
>>
>> R0b0t1@host:~/devel/project$ git submodule add
>> https://github.com/user/project -f
>> Cloning into '/home/R0b0t1/devel/project/-f'...
>
> Thanks for reporting.  Confusingly, I think this is intended behavior.
> "git help submodule" explains:
>
>         add [-b <branch>] [-f|--force] [--name <name>]
>                 [--reference <repository>] [--depth <depth>] [--]
>                 <repository> [<path>]
>
>                 Add the given repository as a submodule at the given
>                 path [etc]
>
> Since the -f comes after <repository>, it is a <path>.
>

Not to comment on every response to this bug, but I understand. What
is confusing is that the command was failing without being forced, and
without thinking I added "-f" at the end. The command succeeded as if
I supplied the force flag, but per my experience with other tools, and
with other Git commands, "-f" should not be a flag. It should be a
path, as you say. However it appears to be both.

To reproduce my situation exactly, add "*" to your .gitignore. The
directory "gerrit" should then be ignored, and Git will warn you that
the submodule will not be tracked (I may have another issue to report
related to this, but I'm still trying to figure out what is going on).
However, if you name the directory something else, like "-f", it will
still be matched by the .gitignore rule and should not succeed. This
is why I think the path is also being interpreted as a flag.

Something else may be happening, but either way the behavior does not
seem to be expected nor consistent with other parts of Git.

> That said, there are a few related things wrong here.
>
> The usage string above says I can put "--" before the <repository> to
> make things extra unambiguous.  But when I try that, I get the following
> result:
>
>         $ git submodule add -- https://gerrit.googlesource.com/gerrit -f
>         Cloning into '/tmp/t/test/-f'...
> [...]
>         Resolving deltas: 100% (215796/215796), done.
>         /usr/lib/git-core/git-submodule: line 261: cd: -f: invalid option
>         cd: usage: cd [-L|[-P [-e]] [-@]] [dir]
>         Unable to checkout submodule '-f'
>
> If I try to put the "--" between <repository> and <path>, I get another
> confusing result:
>
>         $ git submodule add https://gerrit.googlesource.com/gerrit -- -f
>         '--' already exists in the index
>
> "git help cli" is supposed to give advice about this kind of thing as
> well --- e.g., it gives some sound advice about what form of flags
> scripts should use (e.g., to always use the 'stuck' form --name=<name>
> instead of --name name).  But it doesn't mention this issue of flags
> belonging before other arguments.
>
> Thoughts?
>

I have experienced issues with -- before. It has been long enough I
have forgotten. Sorry for not being able to provide anything concrete.