[libvirt PATCH] meson: fix readline detection if there is no pkg-config file

Andrea Bolognani abologna at redhat.com
Wed Aug 5 11:16:01 UTC 2020


On Wed, 2020-08-05 at 12:51 +0200, Pavel Hrdina wrote:
> Commit <74416b1d4849ef77ef31de5344dd75f03094434b> added check for
> rl_completion_quote_character to make sure we have correct readline
> library. Commit <a9443bc9a9ef451b46306e66ed3b706756fc1414> added
> inaccurate comment that it's a function.

So it was my fault after all O:-)

> +    correct_rl = cc.has_header_symbol('readline/readline.h', 'rl_completion_quote_character', prefix: '#include <stdio.h>')
> +    if not correct_rl
> +      if get_option('readline').enabled()
> +        error('readline is missing rl_completion_quote_character')

This matches the original error message, but thinking about it again
I would suggest changing it to something like

  error('readline >= @0@ was not found'.format(readline_version))

because the specifics of the symbol are not important here, we just
happen to be using it as a witness.

Either way,

  Reviewed-by: Andrea Bolognani <abologna at redhat.com>

and thanks for taking care of this :)

-- 
Andrea Bolognani / Red Hat / Virtualization




More information about the libvir-list mailing list