[PATCH] generate-cmdlist.sh: avoid selecting synopsis at wrong place

[PATCH] generate-cmdlist.sh: avoid selecting synopsis at wrong place

[Nguyễn Thái Ngọc Duy]

In "common" man pages there is luckily no "NAME" anywhere except at
beginning of documents. If there is another "NAME", sed could
mis-select it and lead to common-cmds.h corruption. So better nail it
at beginning of line, which would reduce corruption chance.

Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
---
 generate-cmdlist.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/generate-cmdlist.sh b/generate-cmdlist.sh
index a2913c2..75c68d9 100755
--- a/generate-cmdlist.sh
+++ b/generate-cmdlist.sh
@@ -14,7 +14,7 @@ sort |
 while read cmd
 do
      sed -n '
-     /NAME/,/git-'"$cmd"'/H
+     /^NAME/,/git-'"$cmd"'/H
      ${
             x
             s/.*git-'"$cmd"' - \(.*\)/  {"'"$cmd"'", "\1"},/
-- 
1.6.0.96.g2fad1.dirty

Re: [PATCH] generate-cmdlist.sh: avoid selecting synopsis at wrong place

[Santi Béjar]

On Thu, Sep 18, 2008 at 6:01 PM, Nguyễn Thái Ngọc Duy <pclouds@gmail.com> wrote:
> In "common" man pages there is luckily no "NAME" anywhere except at
> beginning of documents. If there is another "NAME", sed could
> mis-select it and lead to common-cmds.h corruption. So better nail it
> at beginning of line, which would reduce corruption chance.
>

You could narrow it even more with /^NAME$/

Santi

Re: [PATCH] generate-cmdlist.sh: avoid selecting synopsis at wrong place

[Nguyen Thai Ngoc Duy]

On 9/18/08, Santi Béjar <santi@agolina.net> wrote:
> On Thu, Sep 18, 2008 at 6:01 PM, Nguyễn Thái Ngọc Duy <pclouds@gmail.com> wrote:
>  > In "common" man pages there is luckily no "NAME" anywhere except at
>  > beginning of documents. If there is another "NAME", sed could
>  > mis-select it and lead to common-cmds.h corruption. So better nail it
>  > at beginning of line, which would reduce corruption chance.
>  >
>
>
> You could narrow it even more with /^NAME$/

That may miss "^NAME $", which is hard to recognize by eyes, although
I'm pretty sure there is no trailing spaces in git. It's up to Junio
to decide how strict he wants.
-- 
Duy