[Buildroot] [PATCH] package/python: enable xml libraries

[Angelo Compagnucci]

Hi Yegor,

2015-02-13 10:47 GMT+01:00 Yegor Yefremov <yegorslists at googlemail.com>:
> On Fri, Feb 13, 2015 at 10:39 AM, Angelo Compagnucci
> <angelo.compagnucci at gmail.com> wrote:
>> Hi Yegor,
>>
>> 2015-02-13 10:31 GMT+01:00 Yegor Yefremov <yegorslists at googlemail.com>:
>>
>>>>
>>>> +config BR2_PACKAGE_PYTHON_XML
>>>> +       bool "xml module"
>>>> +       select BR2_PACKAGE_PYTHON_PYEXPAT
>>>> +       help
>>>> +         Xml libraries for Python
>>>> +
>>>
>>> What about this existing option?
>>>
>>> config BR2_PACKAGE_PYTHON_PYEXPAT
>>>         select BR2_PACKAGE_EXPAT
>>>         bool "pyexpat"
>>>         help
>>>           pyexpat module for Python.
>>>
>>> Yegor
>>
>> Whithout BR2_PACKAGE_PYTHON_XML there is no way you can know exactly
>> when xml modules will be included or not without reading the configure
>> script.
>> Inedeed, python xml modules are included by the configure script only
>> if pyexpat is enabled.
>>
>> BR2_PACKAGE_PYTHON_XML makes clear that if you want xml libraries, you
>> should select pyexpat.
>
> Then the existing MACRO should be renamed. Though this implies
> renaming it in the existing packages, that have this option as
> dependency like jango etc.

Yes, probably renaming BR2_PACKAGE_PYTHON_PYEXPAT to
BR2_PACKAGE_PYTHON_XML makes sense, but it's an intrusive changing.
I'm offering to volunteer, but I will wait for more comments.

Sincerely, Angelo

>
> Yegor



-- 
Profile: http://it.linkedin.com/in/compagnucciangelo