[libvirt PATCH 1/4] meson: Print custom message when GNU grep is not installed
Andrea Bolognani
abologna at redhat.com
Thu Apr 1 09:51:15 UTC 2021
On Thu, 2021-04-01 at 11:23 +0200, Erik Skultety wrote:
> Please squash this in before merging:
>
> diff --git a/build-aux/meson.build b/build-aux/meson.build
> index 1095982397..e491bdeebc 100644
> --- a/build-aux/meson.build
> +++ b/build-aux/meson.build
> @@ -26,10 +26,6 @@ if host_machine.system() == 'freebsd'
> if not grep_prog.found()
> error('GNU grep not found')
> endif
> - grep_cmd = run_command(grep_prog, '--version')
> - if grep_cmd.stdout().startswith('grep (BSD grep')
> - error('GNU grep not found')
> - endif
Mh, so what you're saying is that we can assume that
/usr/local/bin/grep is *always* going to be GNU grep? That's probably
a fair assumption.
I disagree that the hunk should be squashed in, though, since this
patch simply changes the error message and not what's been checked.
It should go in separately, either after this series or before it.
Do you want me to post it in that form, or would you rather do that
yourself since you have already done the work? Whatever is more
convenient for you :)
--
Andrea Bolognani / Red Hat / Virtualization
More information about the libvir-list
mailing list