On 10/14/2015 07:18 AM, Akihiro Motoki wrote:
> 2015-10-14 0:14 GMT+09:00 Doug Hellmann <d...@doughellmann.com>:
>> Excerpts from Thomas Goirand's message of 2015-10-13 12:38:00 +0200:
>>> On 10/12/2015 11:09 PM, Steve Baker wrote:
>>>> On 13/10/15 02:05, Thomas Goirand wrote:
>>>>>
>>>>> BTW, the same applies for tablib which is in a even more horrible state
>>>>> that makes it impossible to package with Py3 support. But tablib could
>>>>> be removed from our (build-)dependency list, if someone cares about
>>>>> re-writing cliff-tablib, which IMO wouldn't be that much work. Doug, how
>>>>> many beers shall I offer you for that work? :)
>>>>>
>>>> Regarding tablib, cliff has had its own table formatter for some time,
>>>> and now has its own json and yaml formatters. I believe the only tablib
>>>> formatter left is the HTML one, which likely wouldn't be missed if it
>>>> was just dropped (or it could be simply reimplemented inside cliff).
>>>>
>>>> If the cliff deb depends on cliff-tablib
>>>
>>> It does.
>>
>> That dependency is backwards. cliff-tablib should depend on cliff. Cliff
>> does not need cliff-tablib, but cliff-tablib is only useful if cliff is
>> installed.
>>
>>> And also the below packages have a build-dependency on
>>> cliff-tablib:
>>>
>>> - python-neutronclient
>>> - python-openstackclient
>>>
>>> python-openstackclient also has a runtime depends on cliff-tablib.
>>
>> Now that we have a cliff with the formatters provided by tablib, we can
>> update those dependencies to remove cliff-tablib. Someone just needs to
>> follow through on that with patches to the requirements files for the
>> clients.
> 
> In neutronclient, we have cliff-tablib is test-requirements.txt,
> but it is actually unnecessary now.
> https://review.openstack.org/#/c/234334/
> 
> Akihiro

Ah, super nice! Is it also not necessary for the Liberty release of
neutronclient? Or just master?

Cheers,

Thomas Goirand (zigo)


__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to