From: Junio C Hamano Date: Tue, 05 Mar 2013 15:45:22 GMT Subject: Re: [PATCH v6] submodule: add 'deinit' command Message-ID: <7vppzdvoj1.fsf@alter.siamese.dyndns.org> In-Reply-To: <20130305072944.GA4677@sandbox-ub> Heiko Voigt writes: >> + if test -z "$force" >> + then >> + git rm -n "$sm_path" || >> + die "$(eval_gettext "Submodule work tree '\$sm_path' contains local modifications; use '-f' to discard them")" > > Minor nit. IMO, there is an indentation for the || missing here. Maybe > Junio can squash that in on his side? Sorry, but I do not see an indentation nit here. The format looks perfectly sane to me and in fact any other indentation would be wrong. Puzzled...