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

Igor Gnatenko <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[email protected]
           Assignee|[email protected]    |[email protected]



--- Comment #1 from Igor Gnatenko <[email protected]> ---
> %global sum HTTP REST client, simplified for Python
put to Summary tag and reuse further as %{summary}

> %autosetup -n %{name}-%{version}
jsut %autosetup

> %{python3_sitelib}/python_http_client*
too relax (actually you have 2 same lines, with glob and without). it's always
better to be more precised

* No pythonX-setuptools BR, guess it's missing
* Reuse %{url} in Source tag
* add "-v" to py.test invocation (useful when some test fails)
* do %global _description and reuse from descriptions. Also add "Python X
version." for each subpackage to easily identify without name where it belongs
to.
* Move BuildRequires under subpackages for better look&feel

-- 
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]
https://lists.fedoraproject.org/admin/lists/[email protected]

Reply via email to