From: Junio C Hamano Date: Tue, 22 Nov 2016 17:43:42 GMT Subject: Re: [PATCH v2 2/2] push: fix --dry-run to not push submodules Message-ID: In-Reply-To: <20161117190255.GK66382@google.com> Brandon Williams writes: > On 11/17, Stefan Beller wrote: >> On Thu, Nov 17, 2016 at 10:46 AM, Brandon Williams wrote: >> >> > sha1_array_clear(&commits); >> > - die("Failed to push all needed submodules!"); >> > + die ("Failed to push all needed submodules!"); >> >> huh? Is this a whitespace change? > > That's odd...I didn't mean to add that lone space. Is that the only glitch in this round? IOW, is the series OK to be picked up as long as I treak this out while queuing?