Re: Passing password to script for rpmsign of list of .rpm files

[Nicolas Thierry-Mieg <Nicolas.Thierry-Mieg@xxxxxxx>]

John R. Dennison wrote:
> On Sat, May 21, 2011 at 09:30:06AM +0200, Nicolas Thierry-Mieg wrote:
>> Ljubomir Ljubojevic wrote:
>>> for i in $(find . -type f | grep .rpm); do
>>
>>> rpmsign --addsign `find . -type f | grep .rpm | grep -v .zzz`
>>
>> just a small comment, grep uses regexps so this doesn't do what you want
>> (eg the . is a wildcard char). Your script can break, silently (won't
>> sign rpms whose name contains "any char followed by zzz") or not (will
>> attempt to rpmsign eg myrpms.pl), with some particular file names.
>>
>> what you really want is files ending with .rpm, so:
>> grep '\.rpm$'
>
> Why are people passing this off to grep?
>
> rpmsign --addsign $(find . -type f -name \*.rpm ! -name \*.zzz)

agreed, using find alone is "another way to do it", although as stated 
by John Pierce the second -name is useless here.
I was pointing out a flaw in the code and offering a correction using 
"the same way to do it".
_______________________________________________
CentOS mailing list
CentOS@xxxxxxxxxx
http://lists.centos.org/mailman/listinfo/centos