crond: mention other cronds' behavior in comment. no code changes

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2010-07-11 23:20:15 +02:00
parent 729f39dd17
commit 69d69e2cb8

View File

@ -304,13 +304,15 @@ static void FixDayDow(CronLine *line)
//if crontab was reloaded: crond thinks that "new" job is different from "old" //if crontab was reloaded: crond thinks that "new" job is different from "old"
//even if they are in fact completely the same. Example //even if they are in fact completely the same. Example
//Crontab was: //Crontab was:
// 0-59 * * * job1 // 0-59 * * * * job1
// 0-59 * * * long_running_job2 // 0-59 * * * * long_running_job2
//User edits crontab to: //User edits crontab to:
// 0-59 * * * job1_updated // 0-59 * * * * job1_updated
// 0-59 * * * long_running_job2 // 0-59 * * * * long_running_job2
//Bug: crond can now start another long_running_job2 even if old one //Bug: crond can now start another long_running_job2 even if old one
//is still running. //is still running.
//OTOH most other versions of cron do not wait for job termination anyway,
//they end up with multiple copies of jobs if they don't terminate soon enough.
static void delete_cronfile(const char *userName) static void delete_cronfile(const char *userName)
{ {
CronFile **pfile = &G.cron_files; CronFile **pfile = &G.cron_files;