qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 0/5] ui: keymap fixes for 2.10


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH 0/5] ui: keymap fixes for 2.10
Date: Tue, 8 Aug 2017 10:04:48 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1

On 08/08/2017 04:20 AM, Markus Armbruster wrote:
> Just back from the signed-vs-unsigned swamp, still catching up, sorry
> for the delay...
> 

>>>   File "/var/tmp/patchew-tester-tmp-xs4657r8/src/scripts/qapi2texi.py", 
>>> line 160, in texi_members
>>>     items += member_func(section.member) + desc + '\n'
>>>   File "/var/tmp/patchew-tester-tmp-xs4657r8/src/scripts/qapi2texi.py", 
>>> line 133, in texi_enum_value
>>>     return '@item @code{%s}\n' % value.name
>>> AttributeError: 'NoneType' object has no attribute 'name'
>>
>> Eww, we have a bug in our qapi generator (a stack dump instead of a nice
>> error message).  I suspect this is caused by your use of '@ac_*' instead
>> of spelling out all the ac_ members added in qapi-schema.json in 4/5 -
> 
> Yes, we require doc comments to match the schema.  Wildcards aren't
> supported.  They could be, if we want them badly enough.
> 
>> but it IS something that needs to be fixed on your end to quit tripping
>> up the doc generator.
> 
> Has anyone worked on a fix for qapi2texi.py?

We worked around the generator bug in the meantime, so the qapi2texi.py
bug is still latent (sadly, it's not my highest priority)

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]