Re: [PATCH] python: Fix libvirt.py generation to include virterror info

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/14/2011 04:27 PM, Cole Robinson wrote:
> Recent generator refactoring broke libvirt.py. With this patch, libvirt.py
> is generated exactly the same as before the offending commit.
> ---
>  python/generator.py |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)
> 

Just to be clear, we definitely want this before release, since the effect of
this bug is that all python error reporting throws a traceback.

Thanks,
Cole

> diff --git a/python/generator.py b/python/generator.py
> index 327e1d5..d855d6b 100755
> --- a/python/generator.py
> +++ b/python/generator.py
> @@ -120,7 +120,8 @@ class docParser(xml.sax.handler.ContentHandler):
>                  if attrs.has_key('field'):
>                      self.function_return_field = attrs['field']
>          elif tag == 'enum':
> -            if attrs['file'] == "libvirt":
> +            if (attrs['file'] == "libvirt" or
> +                attrs['file'] == "virterror"):
>                  enum(attrs['type'],attrs['name'],attrs['value'])
>              elif attrs['file'] == "libvirt-qemu":
>                  qemu_enum(attrs['type'],attrs['name'],attrs['value'])
> @@ -130,7 +131,9 @@ class docParser(xml.sax.handler.ContentHandler):
>              print "end %s" % tag
>          if tag == 'function':
>              if self.function != None:
> -                if self.function_module == "libvirt":
> +                if (self.function_module == "libvirt" or
> +                    self.function_module == "event" or
> +                    self.function_module == "virterror"):
>                      function(self.function, self.function_descr,
>                               self.function_return, self.function_args,
>                               self.function_file, self.function_module,

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]