Re: [PATCH] Remove "bashism" from contrib/thunderbird-patch-inline/appp.sh

From: Victor Engmark <victor.engmark@terreactive.ch>
Date: Mar 29, 2011, 14:03 UTC
Message-ID: <4D91E6AE.9040208@terreactive.ch>
In-Reply-To: <8721039.4955.1301382568626.JavaMail.trustmail@mail1.terreactive.ch>

On 03/29/2011 09:09 AM, Junio C Hamano wrote:

> Ángel González <ingenit@zoho.com> writes:

Show 8 quoted lines

>>>  if [ "$?" != "0" ] ; then
> 
> While I personally do not like this style (I am old fashioned) and would
> probably write:
> 
> 	if test $? != 0
>         then
>         	...

Nitpicking I suppose, but since `$?` is always an integer we should use `-ne` (positive/negative integers) instead of `!=` (string comparison).

Show 8 quoted lines

> or make it even more readable by writing it together with the previous
> statement, i.e.
> 
> 	PATCH=$(zenity --file-selection) ||
>         ...
> 
> myself, it is definitely not bash-ism to use [] for conditionals.  Some
> people seem to find it more readable than traditional "test" (not me).

Alternatively:

if ! PATCH=$(zenity --file-selection) then ...

Yep, that works in dash - Both variable assignment and exit code checking.

-- 
Victor Engmark