On Tue, Apr 09, 2019 at 04:27:47PM +0200, Andrea Bolognani wrote: > The current code is a bit awkward, and we're going to need > to share it later anyway. We can drop the call to AC_SUBST() > while we're at it, since LIBVIRT_CHECK_LIB() already marks > READLINE_CFLAGS for substitution. > > The new code goes to some extra length to avoid setting > -D_FUNCTION_DEF twice: this is mostly for cosmetic reasons, > and it's necessary because LIBVIRT_CHECK_READLINE() is called > twice: once on its own, and then once more as part of > LIBVIRT_CHECK_BASH_COMPLETION(). Technically theres a way to cache things so that the funcs are only run once, but its not worth the pain of learning more m4 > > Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx> > --- > m4/virt-readline.m4 | 12 ++++++++++-- > 1 file changed, 10 insertions(+), 2 deletions(-) Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx> Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
