svn commit: r241625 - head/usr.sbin/cron/cron

[Maxim Sobolev]

Guys,

The latest version of the patch against current HEAD is available here:

http://sobomax.sippysoft.com/cron.diff

Any comments/suggestions are appreciated. This version should default to 
previous behaviour of waking up every 60 seconds unless there is 
@every_second entry in the cron.

Konstantin, can you please put it for testing to your boxes? Please make 
sure to do `make clean' after applying it, as libcron is not listed as a 
dependency for cron, so it's possible that some changes may not be 
picked up.

Thanks!

-Maxim

On 10/17/2012 1:18 PM, Konstantin Belousov wrote:
> On Wed, Oct 17, 2012 at 01:07:20PM -0700, Maxim Sobolev wrote:
>> On 10/17/2012 10:48 AM, Konstantin Belousov wrote:
>>> The cron daemon in HEAD is not functional, i.e. no jobs are executed
>>> at all. Reverting r241625 r241618 r241576 gives me the expected behaviour.
>>>
>>> P.S. I do not remember a single commit from you which did not failed on
>>> tinderbox, or was not found buggy later.
>>
>> Should be fixed already in the latest HEAD. Please check and let me know
>> if you still have some issues (send crontab entry which is not getting
>> executed). The particular issue has been fixed in the r241618.
>
> Well, as I noted above, I had r241618 applied, since I needed to revert it.
> For me, neither the standard entries from /etc/crontab, nor the following
> user crontab:
> * * * * *       /usr/bin/limits -a >/tmp/1.txt
> are not executing. In particular, the /tmp/1.txt file was not created,
> and the cron.log was empty.
>
> As I said, a reversal of all three commits makes the cron(8) operate
> correctly.
>