Re: [PATCH] apibuild: Turn hidden_macros into array

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

 



On 22.04.2016 11:26, Jiri Denemark wrote:
> On Fri, Apr 22, 2016 at 11:09:37 +0200, Michal Privoznik wrote:
>> This is an array, not a hash table. It makes no problem with
>> newer pythons, but with python2.6 it is a problem:
>>
>>     File "../../docs/apibuild.py", line 117
>>       "VIR_DEPRECATED", # internal macro to mark deprecated apis
>>                       ^
>>   SyntaxError: invalid syntax
>>
>> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
>> ---
>>  docs/apibuild.py | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/docs/apibuild.py b/docs/apibuild.py
>> index 9c82c4a..d065d56 100755
>> --- a/docs/apibuild.py
>> +++ b/docs/apibuild.py
>> @@ -113,10 +113,10 @@ ignored_macros = {
>>  }
>>  
>>  # macros that should be completely skipped
>> -hidden_macros = {
>> +hidden_macros = [
>>    "VIR_DEPRECATED", # internal macro to mark deprecated apis
>>    "VIR_EXPORT_VAR", # internal macro to mark exported vars
>> -}
>> +]
> 
> That's another option, but I think the fix Peter pushed for this some
> time ago is good enough.
> 

I don't think he pushed any fix. What he did in his recent commit in
this area was to introduce this array. I don't see any other movement in
that respect. What commit exactly do you have in mind?

Michal

--
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]