[ccache] [PATCH] Use bash for test.sh.

Eitan Adler lists at eitanadler.com
Mon Nov 5 10:58:20 MST 2012


On 5 November 2012 12:55, Andrew Stubbs <ams at codesourcery.com> wrote:
> The test script fails with shells other than bash. At least dash
> doesn't work.
>
> Signed-off-by: Andrew Stubbs <ams at codesourcery.com>
> ---
>  test.sh |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test.sh b/test.sh
> index 4c78617..a334f62 100755
> --- a/test.sh
> +++ b/test.sh
> @@ -1,4 +1,4 @@
> -#!/bin/sh
> +#!/bin/bash


This line is buggy:
please http://blog.eitanadler.com/2012/10/binbash-considered-harmful.html
for why.

I ideally the non-portable bashims get fixed but if not the shebang
line must be "/usr/bin/env bash" not /bin/bash.

-- 
Eitan Adler


More information about the ccache mailing list