path: root/builtin/remote.c
diff options
authorJeff King <>2011-03-30 19:52:52 (GMT)
committerJunio C Hamano <>2011-03-30 20:47:51 (GMT)
commit13fc2c1877a50dde5ea43e2f37420ebfb2f7dad1 (patch)
tree1058e79a0b64aef224659118fbcc88c9ff835661 /builtin/remote.c
parent90cff968b3757061914f83b85a8d58cb9f29e72a (diff)
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 <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 'builtin/remote.c')
1 files changed, 5 insertions, 0 deletions
diff --git a/builtin/remote.c b/builtin/remote.c
index cb26080..952be2e 100644
--- a/builtin/remote.c
+++ b/builtin/remote.c
@@ -161,6 +161,11 @@ static int add(int argc, const char **argv)
if (argc < 2)
usage_with_options(builtin_remote_add_usage, options);
+ if (mirror && master)
+ die("specifying a master branch makes no sense with --mirror");
+ if (mirror &&
+ die("specifying branches to track makes no sense with --mirror");
name = argv[0];
url = argv[1];