Re: [PATCH 3/5] Replace $((...)) with expr invocations.
- From
- Ralf Wildenhues <ralf.wildenhues@gmx.de>
- Date
- Nov 6, 2007, 20:26 UTC
- Message-ID
- <20071106202600.GH6361@ins.uni-bonn.de>
- In-Reply-To
- <20071106201809.GD6361@ins.uni-bonn.de>
* Ralf Wildenhues wrote on Tue, Nov 06, 2007 at 09:18:09PM CET:
Show 5 quoted lines
> --- > git-filter-branch.sh | 4 ++-- > git-rebase--interactive.sh | 8 ++++---- > git-rebase.sh | 8 ++++---- > 3 files changed, 10 insertions(+), 10 deletions(-)
Hmm, maybe this one is overkill. $((...)) is POSIX, I temporarily forgot (thanks Benoît!).
I'm unsure whether git targets non-POSIX Bourne shells like Solaris /bin/sh. That would however mean replacing stuff like $(cmd) with `cmd` as well, and from grepping the source it looks like you'd rather avoid that.
Cheers, Ralf