[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
Sun May 29 15:15:11 PDT 2016


On Wed, Jan 13, 2016 at 08:55:33PM +0100, Dimitry Andric via llvm-commits wrote:
> On 13 Jan 2016, at 20:48, Dimitry Andric via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> > 
> > Author: dim
> > Date: Wed Jan 13 13:48:50 2016
> > New Revision: 257663
> > 
> > URL: http://llvm.org/viewvc/llvm-project?rev=257663&view=rev
> > Log:
> > Remove bashism from merge.sh: POSIX sh does not have the `function`
> > reserved word, and it is even superfluous in bash, for this particular
> > instance.
> > 
> > Modified:
> >    llvm/trunk/utils/release/merge.sh
> 
> Hi Hans,
> 
> Can I merge this to the 3.8 branch? (Using the fixed merge.sh script of course ;-)

Hi Tom,
this is still open and creating some pain for those where /bin/sh is not
bash. Can we merge this?

Joerg


More information about the llvm-commits mailing list