[Bug 244894] Review Request: yum-cron - get yum updates via a cron job

bugzilla at redhat.com bugzilla at redhat.com
Mon Jul 30 05:50:10 UTC 2007


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: yum-cron - get yum updates via a cron job


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





------- Additional Comments From ahabig at umn.edu  2007-07-30 01:50 EST -------
Thanks for the review!  Updated package files, with your concerns addressed:

  Spec URL: http://neutrino.d.umn.edu/~habig/yum-cron.spec
  SRPM URL: http://neutrino.d.umn.edu/~habig/yum-cron-0.2-5.src.rpm

Questions/comments:

Regarding the "We Are Upstream" bit:  Added the comments.  Momentarily took out
the URL field (which points to the main yum website) since, well, We Are
Upstream.  But rpmlint didn't like not having a URL.  What's the appropriate URL
to use in this case?

Regarding the dist tag: went and read all about it, and think that it's not
appropriate here.  The same spec file will happily work for all dists which are
using the yum after which the cron files were yanked, yum-3.0.  So, I changed
the "requires" to yum >= 3.0, which should cover all the bases.

I see what you mean about the initscripts - it's moving to a whole new format,
while this script is of the old format.  I'll work on understanding and porting
to the new standards, but that won't happen for a while and I appreciate you not
blocking things while I do so.

One warning - I'm about to embark on a two-week combination of moving my family
to another town (on sabbatical this coming year) and camping/vacation trip, so
will have spotty net.access in the interim.  So, please don't be offended if I'm
not so responsive till mid-August.


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




More information about the Fedora-package-review mailing list