<html><body>
<p><font size="2" face="sans-serif"><br>
</font><tt><font size="2">> From: Doug Hellmann <doug@doughellmann.com></font></tt><br>
<tt><font size="2">> To: openstack-dev <openstack-dev@lists.openstack.org></font></tt><br>
<tt><font size="2">> Date: 07/28/2015 07:59 PM</font></tt><br>
<tt><font size="2">> Subject: Re: [openstack-dev] [glance] Removing python-swiftclient <br>
> from requirements.txt</font></tt><br>
<tt><font size="2">> <br>
</font></tt><br>
<tt><font size="2"><snip></font></tt><br>
<tt><font size="2"><br>
> I replied on both patches, but I'll repeat it here for a broader<br>
> audience:<br>
> <br>
> Please set up an "extras" entry for each backend instead of just<br>
> removing the dependencies. That will signal to users that you know<br>
> what dependencies there are for a backend, but that they are optional,<br>
> and still allow someone to do the equivalent of "pip install<br>
> glance[vmware]" or "pip install glance[swift]" to get those<br>
> dependencies. Nova and oslo.versionedobjects have examples in their<br>
> setup.cfg if you need a template.<br>
> <br>
> I didn't mention in the reviews, but this will also make integration<br>
> tests in our gate easier, since you can put ".[vmware]" or ".[swift]" in<br>
> the tox.ini to pull in those dependencies.<br>
> <br>
> Doug<br>
> </font></tt><br>
<br>
<tt><font size="2"><snip></font></tt><br>
<br>
<tt><font size="2">I did this... but now I'm thinking I really shouldn't have. That makes perfect sense for glance_store, but shouldn't that only be in glance_store, and not also in glance?</font></tt><br>
<br>
<tt><font size="2">Matthew</font></tt><br>
</body></html>