On Tue, 23 Aug 2016 14:40:08 -0700 Stefan Beller <sbeller@xxxxxxxxxx> wrote: > The surrounding advice is printed to stderr, but the list of > submodules is not. Make the report consistent by reporting everything > to stderr. > > Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx> > --- > > This fixes one of the bugs mentioned in > https://public-inbox.org/git/CAGZ79kbkyupBJfvyX3Hj_R5ZW36+3ufOnnLC-Dpic40nPJAxDA@xxxxxxxxxxxxxx/T/#t > > How to fix the other was not as obvious to me as I do not > understand the philosophy on verbosity in the transport code. I had a look and I would say just enclose all the fprintf() inside a: if (transport->verbose > 0) But then this is the first time I look at the code. I was about to send a patch too but it will conflict with this one :) Anyway, thanks for the quick fix to the inconsistent printing with --quiet. -- Leandro Lucarella Technical Development Lead Sociomantic Labs GmbH <http://www.sociomantic.com> -- 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