Subject: Re: [PATCH] http: inform about alternates-as-redirects behavior

From: Eric Wong <[email-masked]>
Date: Sat, 4 Mar 2017 06:55:48 +0000

Jeff King <[email-masked]> wrote:
> The warning itself:
> 
> > +		warning("alternate disabled by http.followRedirects!=true: %s",
> 
> feels like it could use some whitespace around the "!=", but maybe
> that's just me.

Yeah, I kinda wanted to emulate the command-line syntax.

Maybe rewording it a bit and showing how to enable it will
make more sense:

		warning("alternate: %s", url);
		warning(" may be enabled by -c http.followRedirects=true");

As well as keeping individual lines shorter and hopefully
easier-to-read.