On Tue, Sep 13, 2022 at 02:27:51PM +0200, David Disseldorp wrote:
> On Tue, 13 Sep 2022 17:33:45 +0800, Zorro Lang wrote:
>
> > Due to the change in grep project:
> > a95156247098 ("egrep, fgrep: now obsolete")
> >
> > some tests start to fail on latest Fedora system with grep-3.8-1.fc38
> > or newer version, as:
> >
> > generic/447 70s ... egrep: warning: egrep is obsolescent; using grep -E
> > ...
> > @@ -1,4 +1,5 @@
> > QA output created by 447
> > +egrep: warning: egrep is obsolescent; using grep -E
> > ...
> >
> > So use grep -E and -F to instead of egrep and fgrep commands.
> >
> > Signed-off-by: Zorro Lang <zlang@xxxxxxxxxx>
> > ---
> > common/rc | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> >
> > diff --git a/common/rc b/common/rc
> > index a25cbcd0..dee3965e 100644
> > --- a/common/rc
> > +++ b/common/rc
> > @@ -30,6 +30,17 @@ dd()
> > fi
> > }
> >
> > +# The egrep and fgrep are obsolescent
> > +egrep()
> > +{
> > + grep -E "$@"
> > +}
> > +
> > +fgrep()
> > +{
> > + grep -F "$@"
> > +}
> > +
> > # Prints the md5 checksum of a given file
> > _md5_checksum()
> > {
>
> Nit: AFAICT there's only one fgrep caller, so just change that single
> case. With that fixed:
Thanks for reviewing!
Yeah, I thought about that too. But then I thought a fgrep() might help later
patches to avoid using fgrep command directly, so ...
Thanks,
Zorro
>
> Reviewed-by: David Disseldorp <ddiss@xxxxxxx>
>
