Re: [PATCH] Add missing semicolon to sed command.

From: Junio C Hamano <junkio@cox.net>
Date: Mar 20, 2006, 02:15 UTC
Message-ID: <7v1wwxn9nn.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <20060320001808.GB20765@spearce.org>

Shawn Pearce <spearce@spearce.org> writes:

> generate-cmdlist.sh is giving errors messages from sed on Mac OS
> 10.4 due to a missing semicolon.

I'll take this patch for now, but I've heard a horror story that some other vintage of sed apparently does not like multiple commands separated by semicolons at all, and I've been training myself to do this kind of thing always as multi-line sed script, like this:

     sed -n '
     /NAME/,/git-'"$cmd"'/H
     ${
            x
            s/.*git-'"$cmd"' - \(.*\)/  {"'"$cmd"'", "\1"},/
	    p
     }' "Documentation/git-$cmd.txt"