[PATCH] set info.category for all laptop_panels

Doug Goldstein cardoe at gentoo.org
Wed Apr 25 14:13:53 PDT 2007


Doug Goldstein wrote:
> Danny Kukawka wrote:
>   
>> On Dienstag, 24. April 2007, Danny Kukawka wrote:
>>   
>>     
>>> Due to this change
>>> 10-laptop-panel-mgmt-policy.fdi set now for all laptop panels the interface
>>> information correct. Currently for example on MacBooks this don't work
>>> correct (complete interface info is missing on the related device).
>>>
>>> With this patch I would like to rename 10-laptop-panel-mgmt-policy.fdi to
>>> 20-laptop-panel-mgmt-policy.fdi to be sure that the rules in this file get
>>> processed after the laptop panel stuff in the 10-*.fdi files.
>>>     
>>>       
>> Forget the part about 10-laptop-panel-mgmt-policy.fdi - it's not needed 
>> because you can get the info about the interfaces via introspection from 
>> D-Bus. Overseen this.
>>
>> See new version attached.
>>
>> Danny
>>
>>  10-laptop-panel-mgmt-policy.fdi |   23 +++++++++++++----------
>>  10-macbook-backlight.fdi        |    1 +
>>  10-macbookpro-utils.fdi         |    1 +
>>  10-omap-backlight.fdi           |    1 +
>>  4 files changed, 16 insertions(+), 10 deletions(-)
>>
>>
>>   
>> ------------------------------------------------------------------------
>>
>> _______________________________________________
>> hal mailing list
>> hal at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/hal
>>     
> Using this patch I get the following issue...
>
> 12:54:59.902 [E] create_cache.c:479: /usr/share/hal/fdi/policy/10osvendor/10-laptop-panel-mgmt-policy.fdi:22: XML parse error: mismatched tag
> 12:54:59.903 [E] create_cache.c:544: error processing fdi file '/usr/share/hal/fdi/policy/10osvendor/10-laptop-panel-mgmt-policy.fdi'
> 12:54:59.903 [I] create_cache.c:551: skipped fdi file '/usr/share/hal/fdi/policy/10osvendor/10-laptop-panel-mgmt-policy.fdi'
>
>
>
> _______________________________________________
> hal mailing list
> hal at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/hal
>
>   
Turns out it was an easier fix then I thought. Just a missing </match> tag.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 18_hal_fix_info.category_for_laptop_panel_v2.patch
Type: text/x-patch
Size: 4979 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20070425/68b841a5/18_hal_fix_info.category_for_laptop_panel_v2.bin


More information about the hal mailing list