[flac-dev] [PATCH] Change test scripts shell to bash, to avoid lack of arithmetic support in dash, which is sh on Ubuntu 10.04
Erik de Castro Lopo
mle+la at mega-nerd.com
Tue Mar 19 20:22:42 PDT 2013
Jesse Weinstein wrote:
> The subject line mostly says it all, but for reference, having #!/bin/sh causes the following error:
>
> arithmetic expression: expecting primary: " % 255 + 1"
Arithmetic support is actually not the issue, rather its that dash/sh
doesn't support $RANDOM. The solution to this is to replace $RANDOM
with $(date +%N) which is obviouslsy not all that random, but which
is close enough to random for this usage.
Since Jaren is in the process of making the scripts POSIX shell
compliant, I think swicthing to bash is a bad idea :-).
I was going to do the $RANDOM to $(date +%N) conversion after Jaren's
POSIX patches were all applied.
Cheers,
Erik
--
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/
More information about the flac-dev
mailing list