Code

Fix fetch/clone --quiet when stdout is connected
authorTuncer Ayaz <tuncer.ayaz@gmail.com>
Sun, 5 Oct 2008 13:53:00 +0000 (15:53 +0200)
committerShawn O. Pearce <spearce@spearce.org>
Mon, 6 Oct 2008 07:01:07 +0000 (00:01 -0700)
Fixes the `git clone --quiet` issue raised by Dave Jones in
http://marc.info/?l=git&m=121529226023180&w=2

With this simple patch applied we no longer see the following remote
messages as no-progress is correctly sent to the remote site:

  remote: Counting objects: 84102, done.
  remote: Compressing objects: 100% (24720/24720), done.
  remote: Total 84102 (delta 60949), reused 80810 (delta 57900)

Signed-off-by: Tuncer Ayaz <tuncer.ayaz@gmail.com>
Acked-by: Daniel Barkalow <barkalow@iabervon.org>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
transport.c

index 71433d99978ddcc53c80fd6de4aee98871c6d8cf..35cac441f88737c3b3d50e824a9fc1b9d25aec23 100644 (file)
@@ -643,8 +643,8 @@ static int fetch_refs_via_pack(struct transport *transport,
        args.use_thin_pack = data->thin;
        args.include_tag = data->followtags;
        args.verbose = (transport->verbose > 0);
-       args.quiet = args.no_progress = (transport->verbose < 0);
-       args.no_progress = !isatty(1);
+       args.quiet = (transport->verbose < 0);
+       args.no_progress = args.quiet || !isatty(1);
        args.depth = data->depth;
 
        for (i = 0; i < nr_heads; i++)