From: Brian Gesiak <modocache@gmail.com>
Date: Fri, 28 Feb 2014 11:16:13 GMT
Subject: Re: [PATCH 1/2] t3200-branch: test setting branch as own upstream
Message-ID: <CAN7MxmVxYhF1VXWZNAXyfF8iBNQaqqwoyjmDAP_9_bc5NUUU6A@mail.gmail.com>
In-Reply-To: <20140228105929.GA18960@sigill.intra.peff.net>

> I just don't want to regress somebody else's workflow due
> to my lack of imagination.

This makes a lot of sense to me, although as-is the function emits a
warning and returns immediately (although with a successful status
code), so I'm also stumped as to what kind of workflow this would be
included in.

In any case, if the jury's out on this one, I suppose the two patches
I submitted are good to merge? Part of me thinks the bump from warning
to error belongs in its own patch anyway.

- Brian Gesiak