[Bug 226473] Merge Review: system-switch-mail

[bugzilla at redhat.com]

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 at 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?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.