From: liushuyu <liushuyu011(a)gmail.com&gt;

+++ 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:")

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&gt; > > 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