https://bugzilla.redhat.com/show_bug.cgi?id=1532541

Miro Hrončok <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]



--- Comment #1 from Miro Hrončok <[email protected]> ---
> python-memcached, python-sphinx

There's python2-memcached [1] and python2-sphinx [2].


> Requires:       python2-pytz

This seems like part of the test requires, I suggest to move it down a bit by
an extra new line before it.


> # build documentation
> ...
> cp -ar docs ..

What is the purpose of this cp?


> # remove .po files
> find $RPM_BUILD_ROOT -name "*.po" | xargs rm -f

Please use %{buildroot}, as you do in the rest of the file.


> %{__python} ./runtests.py --settings=test_sqlite --verbosity=2 --parallel 1

Use %{__python2} explicitly.


[1] https://koji.fedoraproject.org/koji/buildinfo?buildID=1011376
[2] https://koji.fedoraproject.org/koji/buildinfo?buildID=1015148

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to