[llvm] r257663 - Remove bashism from merge.sh: POSIX sh does not have the `function`

Joerg Sonnenberger via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 19 14:33:10 PST 2016


On Tue, Jan 19, 2016 at 09:12:37AM -0800, Hans Wennborg via llvm-commits wrote:
> I just worry that keeping these scripts free from bashisms is an
> uphill battle since we don't have anything preventing them slipping in
> again. Because of that, I figure using "#!/usr/bin/env bash" would be
> a better solution.

Given that some popular distributions like Debian ship with a non-bash
/bin/sh, I find requiring non-bash not that much of an uphill battle.

Joerg


More information about the llvm-commits mailing list