On Fri 07 Apr 2006 at 08:54PM, Stephen Potter wrote:
> Not realizing this had already been done, I started to look at this the other
> night. Surprisingly, I came up with almost exactly the same changes as Rich,
> except I only used "/var/run/cron" instead of "/var/run/CRONFIFO".
> I also came up with a couple of issues that need to be considered with this:
> 1) Man page update; since man pages have been released as part of ON yet.
> 2) When a patch is generated to install this, it will need to make sure it
> shuts down cron, deletes the /etc/cron.d/FIFO file and then installs and
> restarts cron. Otherwise, there will be an orphaned named pipe lying around
> the /etc filesystem.
Thanks Stephen; there are some deeper issues here too, which Rich
and I explored. Rich has sent me a revised patch for this, which I
haven't dealt with yet. Can we sign you up as a codereviewer?
Daniel Price - Solaris Kernel Engineering - dp at eng.sun.com - blogs.sun.com/dp