Skip Menu |

This queue is for tickets about the Schedule-Cron CPAN distribution.

Report information
The Basics
Id: 75176
Status: open
Priority: 0/
Queue: Schedule-Cron

People
Owner: Nobody in particular
Requestors: STOCKS [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in:
  • 0.99
  • 0.99_1
  • 0.99_2
  • 0.99_3
  • 1.00
  • 1.00_1
  • 1.01
  • 1.01_1
  • 1.01_2
  • 1.01_3
Fixed in: (no value)



Subject: Ignore crontab lines consisting entirely of whitespace
The load_crontab() method should probably ignore lines consisting entirely of whitespace, but it isn't currently doing so. A line in the crontab file consisting entirely of whitespace causes a fatal error: Exactly 5 or 6 columns has to be specified for a crontab entry ! (not 0) at /usr/lib/perl5/site_perl/5.8.8/Schedule/Cron.pm line 1091. It looks like the intention may have been to handle it this way: Schedule/Cron.pm: 448: next if /^$/; 449: next if /^$/; Replacing the two above lines with this should fix it: next if /^\s*$/; Please let me know if you need any additional information from me. Thanks! -Bob
On Mon Feb 20 16:08:32 2012, STOCKS wrote:
Show quoted text
> The load_crontab() method should probably ignore lines consisting
> entirely of whitespace, but it
> isn't currently doing so. A line in the crontab file consisting
> entirely of whitespace causes a fatal
> error:
>
> Exactly 5 or 6 columns has to be specified for a crontab entry ! (not
> 0) at
> /usr/lib/perl5/site_perl/5.8.8/Schedule/Cron.pm line 1091.
>
> It looks like the intention may have been to handle it this way:
>
> Schedule/Cron.pm:
> 448: next if /^$/;
> 449: next if /^$/;
>
> Replacing the two above lines with this should fix it:
>
> next if /^\s*$/;
>
> Please let me know if you need any additional information from me.
>
> Thanks!
> -Bob

Thanks, this will be fixed in 1.02