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=226473
--- Comment #4 from Parag AN(पराग) panemade@gmail.com 2010-07-22 00:58:41 EDT --- Review: + package builds in mock (rawhide i686). koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=2337682 + rpmlint output for SRPM and for RPM. system-switch-mail.noarch: E: non-executable-script /usr/share/system-switch-mail/switchmail_tui.py 0644L /usr/bin/python system-switch-mail.noarch: W: no-manual-page-for-binary system-switch-mail-nox system-switch-mail-gnome.noarch: E: explicit-lib-dependency libglade2 system-switch-mail-gnome.noarch: W: no-documentation system-switch-mail-gnome.noarch: E: non-executable-script /usr/share/system-switch-mail/switchmail_gui.py 0644L /usr/bin/python
+ source files match upstream url (sha1sum) 02cc140a9f837de89bfd0f45e2b1975f5c172b3b system-switch-mail-1.0.1.tar.bz2 02cc140a9f837de89bfd0f45e2b1975f5c172b3b system-switch-mail-1.0.1.tar.bz2.srpm
+ package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + license is open source-compatible. + License text is included in package. + %doc is present. - BuildRequires are proper + %clean is present which is not needed now. + package installed properly. + Macro use appears rather consistent. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + Desktop file installed correctly. + GUI application
Just to Note here, It will be good if this package start using following guidelines in fedora. For F-13 and above then please follow 1) buildroot should be removed 2) %clean not needed 3) cleaning of buildroot at start of %install also not needed
Suggestions: I think I missed few points 1) BuildRequires: automake is not needed 2) non-executable-script rpmlint error message can be solved by adding ------------------------------------------------- for lib in %{buildroot}%{_datadir}/%{name}/switchmail_tui.py %{buildroot}%{_datadir}/%{name}/switchmail_gui.py; do sed '//usr/bin/python/d' $lib > $lib.new && touch -r $lib $lib.new && mv $lib.new $lib done ------------------------------------------------------------
to end of %install section.
See the successful scratch build when fixed above 2 issues at http://koji.fedoraproject.org/koji/taskinfo?taskID=2338109
I installed this new changes and looks package is working fine.
Can you please fix above 2 issues?