Bug 11144 [QA Followup] - Let each script run in sequence even if one fails
authorKyle M Hall <kyle@bywatersolutions.com>
Fri, 24 Jun 2016 15:18:04 +0000 (15:18 +0000)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 2 Sep 2016 13:51:32 +0000 (13:51 +0000)
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Tested the A;B;C variant here. If A fails, B will run. Since we can safely
assume that A (or B) will not fail on a daily basis, this seems to be better
than running them in the wrong order every day.

As the comments on Bugzilla show, several people support this improved
(reordered) scheme and look forward to improved error handling on another
report (obviously not that simple).

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

misc/cronjobs/crontab.example

index 66d075c..1765f7e 100644 (file)
@@ -48,7 +48,7 @@ KOHA_CRON_PATH = /usr/share/koha/bin/cronjobs
 
 # AUTOMATIC RENEWAL + FINES + OVERDUE NOTICES
 # Sequence is important - renew what can be renewed, charge fines for the rest, print fine amount into overdue notices
-5    1 * * *  __KOHA_USER__ $KOHA_CRON_PATH/automatic_renewals.pl && $KOHA_CRON_PATH/fines.pl && $KOHA_CRON_PATH/overdue_notices.pl -t
+5    1 * * *  __KOHA_USER__ $KOHA_CRON_PATH/automatic_renewals.pl; $KOHA_CRON_PATH/fines.pl; $KOHA_CRON_PATH/overdue_notices.pl -t
 
 # ADVANCE NOTICES
 10   1 * * *  __KOHA_USER__  $KOHA_CRON_PATH/advance_notices.pl -c