From: Junio C Hamano Date: Wed, 11 Feb 2009 18:26:29 GMT Subject: Re: [PATCH 2/4] completion: Use consistent if [...] convention. No test. Message-ID: <7v1vu4g78q.fsf@gitster.siamese.dyndns.org> In-Reply-To: <20090211180705.GL30949@spearce.org> "Shawn O. Pearce" writes: >> - elif test -f "$g/rebase-apply/applying" >> - then >> + elif [ -f "$g/rebase-apply/applying" ]; then > > There is some sort of whitespace damage right here, the elif doesn't > seem to line up correctly. If that is the only gripe and otherwise if you are Ok with the patch, I'll queue the entire series with a fix-up here myself.