From: Dirk Koopman <djk@tobit.co.uk>
Date: Sun, 26 Apr 2020 23:52:33 +0000 (+0100)
Subject: add line more debugging to DXCron::cread
X-Git-Url: http://www.dxcluster.org/gitweb/gitweb.cgi?p=spider.git;a=commitdiff_plain;h=2b89a309f1c5658fbadac1a5d3e5e03d541e1005

add line more debugging to DXCron::cread
---

diff --git a/perl/DXCron.pm b/perl/DXCron.pm
index 13379197..0c388e9d 100644
--- a/perl/DXCron.pm
+++ b/perl/DXCron.pm
@@ -72,20 +72,28 @@ sub cread
 		next unless defined $min;
 		my $ref = bless {};
 		my $err;
-		
-		$err .= parse($ref, 'min', $min, 0, 60);
-		$err .= parse($ref, 'hour', $hour, 0, 23);
-		$err .= parse($ref, 'mday', $mday, 1, 31);
-		$err .= parse($ref, 'month', $month, 1, 12, "jan", "feb", "mar", "apr", "may", "jun", "jul", "aug", "sep", "oct", "nov", "dec");
-		$err .= parse($ref, 'wday', $wday, 0, 6, "sun", "mon", "tue", "wed", "thu", "fri", "sat");
-		if (!$err) {
-			$ref->{cmd} = $cmd;
-			push @out, $ref;
-			dbg("DXCron::cread: adding $_\n") if isdbg('cron');
+
+		if (defined $min && defined $hour && defined $cmd) { # it isn't all of them, but should be enough to tell if this is a real line
+			$err .= parse($ref, 'min', $min, 0, 60);
+			$err .= parse($ref, 'hour', $hour, 0, 23);
+			$err .= parse($ref, 'mday', $mday, 1, 31);
+			$err .= parse($ref, 'month', $month, 1, 12, "jan", "feb", "mar", "apr", "may", "jun", "jul", "aug", "sep", "oct", "nov", "dec");
+			$err .= parse($ref, 'wday', $wday, 0, 6, "sun", "mon", "tue", "wed", "thu", "fri", "sat");
+			if (!$err) {
+				$ref->{cmd} = $cmd;
+				push @out, $ref;
+				dbg("DXCron::cread: adding $_\n") if isdbg('cron');
+			} else {
+				$err =~ s/^, //;
+				LogDbg('cron', "DXCron::cread: error $err on line $line '$_'");
+			}
 		} else {
-			$err =~ s/^, //;
-			dbg("DXCron::cread: error $err on line $line '$_'\n") if isdbg('cron');
+			LogDbg('cron', "DXCron::cread error on line $line '$_'");
+			my @s = ($min, $hour, $mday, $month, $wday, $cmd);
+			my $s = "line $line splits as " . join(', ', (map {defined $_ ? qq{$_} : q{'undef'}} @s));
+			LogDbg('cron', $s);
 		}
+		
 	}
 	close($fh);
 	return @out;