I think such patch would be too harsh for IDE plugin developers. When git is run from IDE, the isatty(1) will be true. But progress information will be still useful, to display an operation progress to user. Please provide a way to force output of progress (for example using environment). Regards, Constantine On Sat, Oct 4, 2008 at 12:18 AM, Tuncer Ayaz <tuncer.ayaz@xxxxxxxxx> wrote: > On Fri, Oct 3, 2008 at 9:50 PM, Daniel Barkalow <barkalow@xxxxxxxxxxxx> wrote: >> On Fri, 3 Oct 2008, Tuncer Ayaz wrote: > > <snip> > >>> diff --git a/transport.c b/transport.c >>> index 71433d9..1f24011 100644 >>> --- a/transport.c >>> +++ b/transport.c >>> @@ -644,7 +644,7 @@ static int fetch_refs_via_pack(struct transport *transport, >>> args.include_tag = data->followtags; >>> args.verbose = (transport->verbose > 0); >>> args.quiet = args.no_progress = (transport->verbose < 0); >>> - args.no_progress = !isatty(1); >>> + args.no_progress = args.quiet || !isatty(1); >> >> If you're doing that, remove the "args.no_progress =" from the previous >> line, which was there to have that effect (but not so clearly). Aside from >> that, it looks good to me. >> > > <snip> > > Thanks Daniel, that makes a lot of sense. > > Signed-off-by: Tuncer Ayaz <tuncer.ayaz@xxxxxxxxx> > --- > transport.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/transport.c b/transport.c > index 71433d9..35cac44 100644 > --- a/transport.c > +++ b/transport.c > @@ -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++) > -- > 1.6.0.2 > -- > To unsubscribe from this list: send the line "unsubscribe git" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
