[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