remote: disallow some nonsensical option combinations
authorJeff King <peff@peff.net>
Wed, 30 Mar 2011 19:52:52 +0000 (15:52 -0400)
committerJunio C Hamano <gitster@pobox.com>
Wed, 30 Mar 2011 20:47:51 +0000 (13:47 -0700)
commit13fc2c1877a50dde5ea43e2f37420ebfb2f7dad1
tree1058e79a0b64aef224659118fbcc88c9ff835661
parent90cff968b3757061914f83b85a8d58cb9f29e72a
remote: disallow some nonsensical option combinations

It doesn't make sense to use "-m" on a mirror, since "-m"
sets up the HEAD symref in the remotes namespace, but with
mirror, we are by definition not using a remotes namespace.

Similarly, it does not make much sense to specify refspecs
with --mirror. For a mirror you plan to push to, those
refspecs will be ignored. For a mirror you are fetching
from, there is no point in mirroring, since the refspec
specifies everything you want to grab.

There is one case where "--mirror -t <X>" would be useful.
Because <X> is used as-is in the refspec, and because we
append it to to refs/, you could mirror a subset of the
hierarchy by doing:

  git remote add --mirror -t 'tags/*'

But using anything besides a single branch as an argument to
"-t" is not documented and only happens to work, so closing
it off is not a serious regression.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/remote.c