Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=754092
--- Comment #1 from Jaroslav Škarvada jskarvad@redhat.com 2011-11-21 08:08:29 EST --- MUST items: [YES] rpmplint is silent [YES] Package meets naming guidelines. [YES] Package meets packaging guidelines I am not sure about the API/ABI compatibility policy of this project. Shouldn't be there version requirement for python-restauth-common? [YES] Spec file matches base package name. [YES] License file is present, matching with spec file. [YES] Licensing Guidelines are met. [!] Spec file is legible and in American English. I would prefer summary like: "Reference implementation of RestAuth specification in Python" or similar
I would tune the description a bit. I wouldn't note the python versions there and I would probably re-word the text. Maybe you could also very briefly describe what the RestAuth is (e.g. "The RestAuth project is a system providing shared authentication, authorization and preferences."). [YES] Sources match upstream. [YES] Package builds OK. [!] BuildRequires are correct. I cannot find the python-setuptools-devel in rawhide. Is the python-setuptools really needed? It seems to build OK without it.
[YES] Package doesn't bundle copies of system libraries. [YES] Package owns all the directories it creates. [YES] Package has no duplicity in %files. [YES] Permission on files are set properly. [NO] Spec file has consistant macro usage. Please use %{optflags} instead of $RPM_OPT_FLAGS or $RPM_BUILD_ROOT instead of %{buildroot}. [YES] Package is code or permissible content. [YES] %doc files don't affect runtime. [YES] Package doesn't own files/directories that other packages own. [YES] Spec file is valid UTF-8.
Should items: [YES] Package builds in mock. [YES] Package uses sane scriptlets.
Some more comments: There is extra space in the second %doc (only cosmetic issue :)
%{!?python_sitelib: %global python_sitelib %(%{__python} -c "from distutils.sysconfig import get_python_lib; print get_python_lib()")} is probably not needed any more.
The defattr is also not needed.
AFAIK the above have only sense if it is planned to package for RHEL-5 EPEL. If so there should be also more additions (e.g. %clean section, ...).
Please consider running the included test-suite as a part of the build process.
Please consider packaging the docs.
Please consider packaging the example script.