[PATCH] test: change "#!/bin/bash" to "#!/usr/bin/env bash" enhances portability

Joel Borggrén-Franck joel.borggren.franck at gmail.com
Thu Dec 2 08:33:04 PST 2010


On Thu, Dec 2, 2010 at 4:13 PM, Ruben Pollan <meskio at sindominio.net> wrote:
> On 21:27, Wed 01 Dec 10, Joel Borggrén-Franck wrote:
>> From: Joel Borggrén-Franck <jbf at codehouse.se>
>>
>> Change #!/bin/bash at start of tests to "#!/usr/bin/env bash". That way
>> systems running on bash < 4 can prepend bash >= 4 to path before
>> running the tests.
>
> Is there any reason to use bash? Is the test system dependent to any key feature
> of bash?
>
> Not every system has bash installed. To have a dependency on bash just for
> development I think is not a big issue. But if it don't requires any big effort
> I think will be better to have it more generic.
>

The test harness explicitly requires bash >= 4 . The docs say it needs
support for associative arrays.

cheers
/Joel


More information about the notmuch mailing list