[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 04/10] gnu: Add python-keystoneclient.
From: |
Ricardo Wurmus |
Subject: |
Re: [PATCH 04/10] gnu: Add python-keystoneclient. |
Date: |
Mon, 4 Jan 2016 17:33:02 +0100 |
Efraim Flashner <address@hidden> writes:
>> > + (build-system python-build-system)
>> > + (native-inputs
>> > + `(("python-setuptools" ,python-setuptools)
>> > + ;; and some packages for the tests
>> > + ("openssl" ,openssl)
>> > + ("python-coverage" ,python-coverage)
>> > + ("python-discover" ,python-discover)
>> > + ("python-fixtures" ,python-fixtures)
>> > + ("python-hacking" ,python-hacking)
>> > + ("python-keyring" ,python-keyring)
>> > + ("python-lxml" ,python-lxml)
>> > + ("python-mock" ,python-mock)
>> > + ("python-mox3" ,python-mox3)
>> > + ("python-oauthlib" ,python-oauthlib)
>> > + ("python-oslosphinx" ,python-oslosphinx)
>> > + ("python-oslotest" ,python-oslotest)
>> > + ("python-pycrypto" ,python-pycrypto)
>> > + ("python-requests-mock" ,python-requests-mock)
>> > + ("python-sphinx" ,python-sphinx)
>>
>> This doesn’t look like it’s needed for the tests. Maybe you could move
>> it above the comment (or remove the comment)?
>
> Do you mean python-sphinx or all of the above? (Just read the next one about
> python-sphinx being for documentation, moved that one.) Openssl isn't a
> declared test dependency, but testing failed without it.
I just meant “python-sphinx”.
~~ Ricardo