Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: libselinux https://bugzilla.redhat.com/show_bug.cgi?id=226042 adel.gadllah@xxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED ------- Additional Comments From adel.gadllah@xxxxxxxxx 2008-01-01 16:30 EST ------- OK taking this one: First the rpmlint warnings/errors: ----------------- > libselinux.x86_64: W: invalid-license Public domain > libselinux.x86_64: W: invalid-license uncopyrighted ----------------- The license tag is indeed invalid it should be "Public Domain" See: http://fedoraproject.org/wiki/Licensing ----------------- > libselinux.x86_64: W: no-url-tag ----------------- Why? Please add an URL tag pointing to the upstream homepage ----------------- > libselinux-python.x86_64: E: script-without-shebang > /usr/lib64/python2.5/site-packages/selinux.py ----------------- As this file is not intended to be executed directly chmod -x it (mode 0644 not 0755) (If it is then please add a shebang) ----------------- libselinux-python.x86_64: W: summary-not-capitalized python bindings for libselinux ----------------- OK thats an easy one ;) ----------------- libselinux-python.x86_64: E: description-line-too-long The libselinux-python package contains the python bindings for developing SELinux ----------------- Please split this into 80 char lines. ----------------- Now general comments to the spec file: >%files devel >%defattr(-,root,root) >%{_libdir}/libselinux.a Please remove this file or if you want to ship it for some reason place it in a -static subpackage. ------ >%files python >%defattr(-,root,root,0755) >%{_libdir}/python*/site-packages/_selinux.so >%{_libdir}/python*/site-packages/selinux.py* Instead of {_libdir}/python*/site-packages do this: Add %{!?python_sitelib: %define python_sitelib %(%{__python} -c "from distutils.sysconfig import get_python_lib; print get_python_lib()")} to the top of the spec file and use %{python_sitelib} instead. ------ >%build >make clean >make CFLAGS="-g %{optflags}" swigify >make CFLAGS="-g %{optflags}" all pywrap Please append %{?_smp_mflags} to the make calls (if it does not break the build). ==== If you want I can attach a patch with the changes mentioned above to help speeding up the review process. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review