Re: [PATCH] transport: report missing submodule pushes consistently on stderr
- From
Stefan Beller <sbeller@google.com>
- Date
- Aug 26, 2016, 22:21 UTC
- Message-ID
- <CAGZ79kaT79WBfwbRw5bw9LOQgHThVOpn66siwr14kU7=ju2WZA@mail.gmail.com>
- In-Reply-To
- <CAGZ79kb1ur=s+FYcdCh+obKyfaozX=adAPESu9G9FQxVtLTDbg@mail.gmail.com>
On Wed, Aug 24, 2016 at 9:35 AM, Stefan Beller <sbeller@google.com> wrote:
Show 10 quoted lines
> On Wed, Aug 24, 2016 at 3:28 AM, Leandro Lucarella > <leandro.lucarella@sociomantic.com> wrote: >> On Tue, 23 Aug 2016 14:40:08 -0700 >> Stefan Beller <sbeller@google.com> 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@google.com> >>> ---
So we seem to have dropped the ball on the followup; this patch nevertheless is a good idea?