guix-devel
[Top][All Lists]
Advanced

[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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]