[Bug 168705] Review Request: newscache - Free cache server for USENET News

[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 report.

Summary: Review Request: newscache - Free cache server for USENET News


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=168705


jpmahowald at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|gdk at redhat.com              |jpmahowald at gmail.com




------- Additional Comments From jpmahowald at gmail.com  2005-10-29 22:11 EST -------
Needs work.

Missing many BuildRequires: libtool, automake, autoconf, texinfo, pam-devel

Release is for the build of the rpm, not the upstream release version. So
Version is 1.2rc6 and Release is 1 (first build). And I don't see the point of
defining a macro for %{name}-%{release}


socket++ is not strictly a part of newscache. I recommend it be packaged seperately.

With the dependencies installed (see BuildRequires comments) it does build on
FC4. rpmlint shows

E: newscache executable-marked-as-config-file /etc/rc.d/init.d/newscache
E: newscache executable-marked-as-config-file /etc/cron.daily/newscache
E: newscache non-standard-dir-perm /var/cache/newscache 02755

%{_initdir} does not really contain config files and as such those scripts
shouldn't be marked %config.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.