Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224684
Summary: Review Request: cronolog - Web log rotation program for Apache Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: jafo-redhat@tummy.com QAContact: fedora-package-review@redhat.com
Spec URL: ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/cronolog.spec SRPM URL: ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/cronolog-1.6.2-3.src.rpm Description: cronolog is a simple filter program that reads log file entries from standard input and writes each entry to the output file specified by a filename template and the current date and time. When the expanded filename changes, the current file is closed and a new one opened. cronolog is intended to be used in conjunction with a Web server, such as Apache, to split the access log into daily or monthly logs.
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: cronolog - Web log rotation program for Apache
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224684
ville.skytta@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |ville.skytta@iki.fi OtherBugsDependingO|163776 |163778 nThis| |
------- Additional Comments From ville.skytta@iki.fi 2007-01-27 02:56 EST ------- - "strip src/cronolog" needs to go, let rpmbuild take care of it and produce a usable debuginfo package.
- Use "make install DESTDIR=..." instead of %makeinstall http://fedoraproject.org/wiki/Packaging/Guidelines#head-fcaf3e6fcbd51194a5d0...
- Why is the info file removed instead of being included and properly installed? http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#info
- The man page refers to /www/sbin/cronolog which could be replaced by the actual installation path, /usr/sbin/cronolog
- Would /usr/bin be a more appropriate install path for cronosplit than /usr/sbin?
- INSTALL is useless in %doc, and should be dropped.
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: cronolog - Web log rotation program for Apache
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224684
------- Additional Comments From jafo-redhat@tummy.com 2007-01-27 13:43 EST ------- All these items addressed, thanks.
ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/cronolog-1.6.2-4.src.rpm ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/cronolog.spec
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: cronolog - Web log rotation program for Apache
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224684
ville.skytta@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 |163779 nThis| |
------- Additional Comments From ville.skytta@iki.fi 2007-01-27 15:21 EST ------- Looks good, approved.
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: cronolog - Web log rotation program for Apache
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224684
jafo-redhat@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
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: cronolog - Web log rotation program for Apache
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224684
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora
jafo@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From jafo@tummy.com 2007-07-05 21:52 EST ------- New Package CVS Request ======================= Package Name: cronolog Short Description: Web log rotation program for Apache Owners: jafo@tummy.com Branches: EL-5 InitialCC:
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: cronolog - Web log rotation program for Apache
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224684
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From jkeating@redhat.com 2007-07-05 22:53 EST ------- Branch 'EL-5' created for module 'cronolog' from branch 'FC-6'
I assume this was a branch request since the module already existed. Please use the proper request form next time.
package-review@lists.fedoraproject.org