Re: [PATCH 3/5] Replace $((...)) with expr invocations.

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Ralf Wildenhues
Date: Tuesday, November 6, 2007 - 1:26 pm

* Ralf Wildenhues wrote on Tue, Nov 06, 2007 at 09:18:09PM CET:

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
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/5] some shell portability fixes, Ralf Wildenhues, (Tue Nov 6, 1:15 pm)
[PATCH 2/5] Fix sed script to work with AIX sed., Ralf Wildenhues, (Tue Nov 6, 1:17 pm)
Re: [PATCH 3/5] Replace $((...)) with expr invocations., Ralf Wildenhues, (Tue Nov 6, 1:18 pm)
[PATCH 4/5] Fix sed string regex escaping in module_name., Ralf Wildenhues, (Tue Nov 6, 1:20 pm)
Re: [PATCH 3/5] Replace $((...)) with expr invocations., Ralf Wildenhues, (Tue Nov 6, 1:26 pm)
Re: [PATCH 0/5] some shell portability fixes, Junio C Hamano, (Tue Nov 6, 1:46 pm)
Re: [PATCH 0/5] some shell portability fixes, Mike Hommey, (Tue Nov 6, 2:02 pm)
Re: [PATCH 3/5] Replace $((...)) with expr invocations., Junio C Hamano, (Tue Nov 6, 2:06 pm)
Re: [PATCH 0/5] some shell portability fixes, Ralf Wildenhues, (Tue Nov 6, 2:09 pm)
[PATCH] Add Documentation/CodingStyle, Johannes Schindelin, (Tue Nov 6, 4:17 pm)
Re: [PATCH 0/5] some shell portability fixes, Johannes Schindelin, (Tue Nov 6, 4:25 pm)
Re: [PATCH] Add Documentation/CodingStyle, Andreas Ericsson, (Tue Nov 6, 5:04 pm)
Re: [PATCH] Add Documentation/CodingStyle, Junio C Hamano, (Tue Nov 6, 5:40 pm)
Re: [PATCH] Add Documentation/CodingStyle, Wincent Colaiuta, (Wed Nov 7, 12:53 am)
Re: [PATCH] Add Documentation/CodingStyle, Andreas Ericsson, (Wed Nov 7, 1:52 am)
Re: [PATCH] Add Documentation/CodingStyle, Andreas Ericsson, (Wed Nov 7, 1:53 am)
Re: [PATCH 0/5] some shell portability fixes, Mike Ralphson, (Wed Nov 7, 7:17 am)
Re: [PATCH 0/5] some shell portability fixes, Johannes Schindelin, (Wed Nov 7, 7:47 am)
Re: [PATCH] Add Documentation/CodingStyle, Johannes Schindelin, (Wed Nov 7, 7:54 am)
[PATCH v2] Add Documentation/CodingStyle, Johannes Schindelin, (Wed Nov 7, 7:59 am)
Re: [PATCH 0/5] some shell portability fixes, Mike Ralphson, (Wed Nov 7, 8:30 am)
Re: [PATCH 0/5] some shell portability fixes, Johannes Schindelin, (Wed Nov 7, 8:37 am)
Re: [PATCH 0/5] some shell portability fixes, Nguyen Thai Ngoc Duy, (Wed Nov 7, 8:58 am)
Re: [PATCH 0/5] some shell portability fixes, Nguyen Thai Ngoc Duy, (Wed Nov 7, 9:05 am)
Re: [PATCH] Add Documentation/CodingStyle, Jon Loeliger, (Wed Nov 7, 12:40 pm)
Re: [PATCH] Add Documentation/CodingStyle, Johannes Schindelin, (Wed Nov 7, 1:13 pm)
Re: [PATCH v2] Add Documentation/CodingStyle, Robin Rosenberg, (Wed Nov 7, 2:43 pm)
[PATCH v3] Add Documentation/CodingGuidelines, Johannes Schindelin, (Wed Nov 7, 3:35 pm)
Re: [PATCH v3] Add Documentation/CodingGuidelines, Junio C Hamano, (Wed Nov 7, 4:14 pm)
[PATCH v4] Add Documentation/CodingGuidelines, Johannes Schindelin, (Wed Nov 7, 5:33 pm)
Re: [PATCH v4] Add Documentation/CodingGuidelines, Junio C Hamano, (Wed Nov 7, 5:38 pm)
Re: [PATCH] Add Documentation/CodingStyle, Mike Ralphson, (Thu Nov 8, 4:29 am)
Re: [PATCH 0/5] some shell portability fixes, Miles Bader, (Sat Nov 10, 3:30 pm)