Re: [PATCH] contrib: Replaced /bin/sh with /bin/bash to make scripts with Bash syntax work on Solaris.

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Aug 8, 2010, 03:11 UTC
Message-ID: <20100808031142.GA20077@burratino>
In-Reply-To: <1281099365-11978-1-git-send-email-asgeir@twingine.no>

Asgeir S. Nilsen wrote:

> +++ b/contrib/ciabot/ciabot.sh
> @@ -1,4 +1,4 @@
> -#!/bin/sh
> +#!/bin/bash

Maybe, relying on a sane $PATH:

	#!/usr/bin/env sh

Not sure what would happen on the less unixy platforms.

Relying on POSIX execlp:

	# shebang line omitted for portability

Simplest of all is to tweak the #! lines in whatever script is used to deploy the scripts from contrib. I would not mind seeing an install-contrib target incorporated into the Makefile that does that.