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

Reply via email to