# Indenting lines starting with "die" in shell after "||"

2 messages from 2015-07-27 to 2015-07-27. Participants: Christian Couder, Johannes Schindelin.
Thread: https://gitlist.dev/t/39940

## Christian Couder, 2015-07-27 09:20

Subject: Indenting lines starting with "die" in shell after "||"
Message-ID: <CAP8UFD3UmrmREa8hN=jGNM8FMc3XHOyX9hGjfRDej_qMrNsaXw@mail.gmail.com>
URL: https://gitlist.dev/e/CAP8UFD3UmrmREa8hN%3DjGNM8FMc3XHOyX9hGjfRDej_qMrNsaXw%40mail.gmail.com

```
Hi,

It looks like we are very inconsistent in shell scripts about
indenting lines starting with "die" after a line that ends with "||",
like:

quite long command ||
die "command failed"

For example in git-rebase--interactive.sh, there is often, but not
always, an extra tab before the die.

It looks like there are no rules about that in Documentation/CodingGuidelines.

Also emacs in shell-script mode is reluctant to add a tab in front of
"die" in such a case.

I wonder if we should state a preference for no extra tab in
Documentation/CodingGuidelines.

Thanks,
Christian.

```

## Johannes Schindelin, 2015-07-27 10:25

Subject: Re: Indenting lines starting with "die" in shell after "||"
Message-ID: <847c822062707efaf2171b5568e00e45@www.dscho.org>
URL: https://gitlist.dev/e/847c822062707efaf2171b5568e00e45%40www.dscho.org
In-Reply-To: <CAP8UFD3UmrmREa8hN=jGNM8FMc3XHOyX9hGjfRDej_qMrNsaXw@mail.gmail.com>

```
Hi Christian,

On 2015-07-27 11:20, Christian Couder wrote:

> It looks like we are very inconsistent in shell scripts about
> indenting lines starting with "die" after a line that ends with "||",
> like:
> 
> quite long command ||
> die "command failed"
> 
> For example in git-rebase--interactive.sh, there is often, but not
> always, an extra tab before the die.

That is most likely my fault: I used to add that extra tab. It appears to me as if the convention in Git (and in my current coding style) is not to add an extra tab.

Ciao,
Dscho

```