Avoid printing unnecessary warnings during fetch and push
authorShawn O. Pearce <spearce@spearce.org>
Tue, 18 Sep 2007 08:54:57 +0000 (04:54 -0400)
committerJunio C Hamano <gitster@pobox.com>
Wed, 19 Sep 2007 10:22:31 +0000 (03:22 -0700)
commitab865e6eec1f40938604b1c28a32525c1fdc7227
tree9fc7487ddf87381861d97f81c5c9cad94ec6d7dc
parent85682c1903a4ae776b0bf2d30d9ecd1e19689131
Avoid printing unnecessary warnings during fetch and push

If a transport doesn't support an option we already are telling
the higher level application (fetch or push) that the option is not
valid by sending back a >0 return value from transport_set_option
so there's not a strong motivation to have the function perform the
output itself.  Instead we should let the higher level application
do the output if it is necessary.  This avoids always telling the
user that depth isn't supported on HTTP urls even when they did
not pass a --depth option to git-fetch.

If the user passes an option and the option value is invalid we now
properly die in git-fetch instead of just spitting out a message
and running anyway.  This mimics prior behavior better where
incorrect/malformed options are not accepted by the process.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
builtin-fetch.c
transport.c