Re: [PATCH] if ! is not portable

On Sat, 2002-02-09 at 13:28, Zack Weinberg wrote:
> The only other portable way to do this is
>
> cmp -s ./ac-helpers/find_apr.m4 ./apr/build/find_apr.m4
> if [ $? -ne 0 ]; then

Well... there is one other way:

cmp -s ./ac-helpers/find_apr.m4 ./apr/build/find_apr.m4 || {
  ...
}

but while it is more elegant, it is less clear (to my mind, anyway). So
the if-with-empty-then-branch is probably better.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:37:06 2006