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:
Reviewed-by: David Disseldorp <ddiss@xxxxxxx>
