Re: [PATCH 02/11] gnu: Add oslotest.

[Cyril Roelandt]

On 07/20/2015 06:14 PM, Thompson, David wrote:
> On Wed, Jul 15, 2015 at 7:32 PM, Cyril Roelandt <address@hidden> wrote:
>> * gnu/packages/openstack.scm: New file.
>> * gnu-system.am (GNU_SYSTEM_MODULES): Add it here.
>> * gnu/packages/python.scm (python-appdirs, python2-appdirs): New variables.
> 
> Could you break this patch so that there is one package per patch?

Hum, I'm not sure whether that really makes sense. The main "feature" I
want to add is the package for oslotest, so it seemed natural to add its
dependencies in the same patch, especially since there is only one.
Plus, this would make this patch series *huge*. Do we really want that?

> Also, all of the Python packages in openstack.scm look like they
> really belong in python.scm.  Could you move them?
> 

Well, in the end, we could have everything from PyPI in python.scm.
OpenStack is a huge project, so I thought it would make sense to have
its packages in their own file. WDYT?


Thanks for the review,
Cyril.