From f787f827ffa9d16438104c5bfed7ae54a1c55fab Mon Sep 17 00:00:00 2001 From: Benoit Sigoure Date: Tue, 6 Nov 2007 20:02:16 +0100 Subject: [PATCH] Do not screw up titles ending with `!' or `?'. Also mention in the README section that svn-wrapper is not expected to work on brain-damaged b0rken foobared Solaris /bin/sh. Signed-off-by: Benoit Sigoure --- svn-wrapper.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/svn-wrapper.sh b/svn-wrapper.sh index 09b46f5..fa8a695 100755 --- a/svn-wrapper.sh +++ b/svn-wrapper.sh @@ -31,7 +31,8 @@ # also works with Git and tries to detect whether the current repository is a # Git or SVN repository. # This script has been written to be as much portable as possible and cover as -# many use-case as reasonably possible. +# many use-case as reasonably possible. It won't work with Solaris' +# brain-damaged stock /bin/sh. # # HOWEVER, there will be bugs, there will be cases in which the script doesn't # wrap properly the svn-cli, etc. In this case, you can try to mail me at @@ -1096,7 +1097,7 @@ vi: ft=diff:noet:tw=76:" >>"$tmp_log" mail_comment=`echo "$full_log" | sed '/^Comment:$/,$!d; //d'` full_log=`echo "$full_log" | sed '/^Comment:$/,$d'` # Add a period at the end of the title. - sed_tmp='/^Title: */!d;s///;/ *\.$/!s/ *$/./' + sed_tmp='/^Title: */!d;s///;/ *[.!?]$/!s/ *$/./' mail_title=`echo "$full_log" | sed "$sed_tmp"` ensure_not_empty 'commit title' "$mail_title" mail_title=`echo "$mail_title" | sed "$sed_eval_tags; $sed_escape"` -- 2.11.4.GIT