Efraim Flashner <efr...@flashner.co.il> 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