[Bug 188542] Review Request: hylafax

bugzilla at redhat.com bugzilla at redhat.com
Tue Apr 11 13:02:12 UTC 2006

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: hylafax


fedora.wickert at arcor.de changed:

           What    |Removed                     |Added
                 CC|                            |fedora.wickert at arcor.de

------- Additional Comments From fedora.wickert at arcor.de  2006-04-11 09:02 EST -------
There are too many issuse with your specfile to list them one by one. Just to
name a few:

- don't use "%define" for name and version
- don't use "%define fc_rel", use "disttag" instead, see
- don't use epoch if not necessary
- don't use "%define initdir     /etc/rc.d/init.d". If you really need it, it
should be "%{_initrddir}".
- Source0 needs an absolute URL (http://...)
- License field not valid, see
- remove "Packager:", see
- BuildRoot should be
"%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)", see
- are you sure about the BuildRequires/BuildPrereq?
- "Requires:    rpm >= 3.0.5" ist stupid and can be dropped
- try to use "%configure" instead of "./configure" and use $RPM_OPT_FLAGS
correctly and not with make, see 
- no need to pass default options to configure (like PAGESIZE)
- change "--with-AWK=/usr/bin/gawk \" to "--with-AWK=%{_bindir}/gawk \"
- same for vgetty and mgetty which will become %{_sbindir}/[v|m]getty
- doesn't use parallel make, see
- use "%defattr(-,root,root,-)" instead of "(-,root,root)"
- remove generic INSTALL from doc section. Not needed when installed from RPM.
- macro usage inconsistent: {initdir} vs. {_initdir} which should be
%{_initrddir} anyway, see http://fedoraproject.org/wiki/Extras/RPMMacros
- empty %pre section
- /sbin/ldconfig in %post and %postun is superflurious since the package doesn't
put shared libs into the linkers path.
- "chkconfig --del" belongs into %preun, see
- Requires(post)/(preun) missing for the scriptlets, see
- no changelog at all, see

I'm not sure if we need to create a system user with a fixed uid/gid and use

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

More information about the fedora-extras-list mailing list