On Tue, Nov 12, 2019 at 11:36:49AM +0100, Andrea Bolognani wrote:
On Fri, 2019-11-01 at 14:23 -0600, liushuyu wrote:
> From: liushuyu <liushuyu011(a)gmail.com>
Please include a Signed-off-by: tag with your full legal name and
your email address to certify that you're in compliance with the DCO,
as explained in [1]. git authorship should also match.
We cannot merge the patch until you've signed off the DCO.
> +++ b/tools/api_tests.py
> @@ -47,10 +47,10 @@ def main():
> else:
> missing.add(el)
>
> - print "missing: %s, implemented: %s" % (len(missing),
len(implemented))
> - print "missing:"
> + print("missing: %s, implemented: %s" % (len(missing),
len(implemented)))
> + print("missing:")
Interesting: it looks like
from __future__ import print_function
is no longer necessary in Python 2.7, which is the only version of
Python 2 we still care about (though not for much longer!).
The patch looks good! Please take care of the formalities mentioned
above and repost, then I'll happily merge it :)
Hum... yes, thank you Andrea to handle this patch and thanks Liushuyu
for your contribution.
s.
[1]
https://libvirt.org/hacking.html#patches item 6
--
Andrea Bolognani / Red Hat / Virtualization