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

Robert-André Mauchin <zebo...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |zebo...@gmail.com



--- Comment #1 from Robert-André Mauchin <zebo...@gmail.com> ---
 - Grab the license from upstream GIT repo:

Source1:       
https://raw.githubusercontent.com/F5Networks/f5-icontrol-rest-python/1.0/LICENSE

   Then:

%prep
%autosetup -n %{srcname}-%{version}
cp %{S:1} .

   And:

%files -n python2-%{srcname}
%doc README.rst
%license LICENSE
%{python2_sitelib}/*

%if 0%{?with_python3}
%files -n python3-%{srcname}
%doc README.rst
%license LICENSE
%{python3_sitelib}/*
%endif

 - You've made a typo here:

BuildRequires:  python3-stuptools

→ setuptools

 - The tests and docs are not included in the Pipy archive. I strongly
encourage you to consider grabbing the archive from Github instead, build the
doc with Sphinx and run the tests with py.test.

-- 
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 -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

Reply via email to