linux-kernel - Re: [PATCH 3/4] Documentation/00-INDEX.sh: add script to help keeping the index up-to-date

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20110428124150.GG9517@debian.org>
Date:	Thu, 28 Apr 2011 14:41:50 +0200
From:	Cyril Brulebois <kibi@...ian.org>
To:	Randy Dunlap <rdunlap@...otime.net>
Cc:	lkml <linux-kernel@...r.kernel.org>, trivial@...nel.org
Subject: Re: [PATCH 3/4] Documentation/00-INDEX.sh: add script to help
 keeping the index up-to-date

Randy Dunlap <rdunlap@...otime.net> (26/04/2011):
> ad I don't see any environment variables that would affect grep either.
> 
> Anyway, using this line instead of any one of the lines above works for me:
> 
> documented=$(sed '1,/^00-INDEX$/d' 00-INDEX|grep -G -v ^[[:space:]]|sed 's:/$::')
> 
> i.e., use ^[[:space:]] instead of the (original) '^\s'.

Hm, I still can't reproduce your original issue, even with a Fedora 11
system; but anyway, using [[:space:]] looks good. In the upcoming
patch I'm also dropping -G: It's the default for GNU grep, and it's
not POSIX-documented[1] anyway.

 1. http://pubs.opengroup.org/onlinepubs/009695399/utilities/grep.html

Thanks, and sorry for the extra round-trips for such a simple patch.

KiBi.

Download attachment "signature.asc" of type "application/pgp-signature" (199 bytes)