[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