From: Junio C Hamano Date: Thu, 31 Mar 2011 21:56:45 GMT Subject: Re: [PATCH] Remove "bashism" from contrib/thunderbird-patch-inline/appp.sh Message-ID: <7vvcyzhs2q.fsf@alter.siamese.dyndns.org> In-Reply-To: Just for the record, the patch at the bottom is what I queued. -- >8 -- From: Maxin john Subject: [PATCH] contrib/thunderbird-patch-inline: do not require bash to run the script The script does not have to be run under bash, but any POSIX compliant shell would do, as it does not use any bash-isms. It may be written under a different style than what is recommended in Documentation/CodingGuidelines, but that is a different matter. While at it, fix obvious typos in the comment. Signed-off-by: Maxin B. John Signed-off-by: Junio C Hamano --- contrib/thunderbird-patch-inline/appp.sh | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/contrib/thunderbird-patch-inline/appp.sh b/contrib/thunderbird-patch-inline/appp.sh index cc518f3..5eb4a51 100755 --- a/contrib/thunderbird-patch-inline/appp.sh +++ b/contrib/thunderbird-patch-inline/appp.sh @@ -1,8 +1,8 @@ -#!/bin/bash +#!/bin/sh # Copyright 2008 Lukas Sandström # # AppendPatch - A script to be used together with ExternalEditor -# for Mozilla Thunderbird to properly include pathes inline i e-mails. +# for Mozilla Thunderbird to properly include patches inline in e-mails. # ExternalEditor can be downloaded at http://globs.org/articles.php?lng=en&pg=2 -- 1.7.4.2.422.g537d99