Handle dayOfWeek+dayOfMonth the same as vixiecron

MichaelXavier / cron

Cron data structure and parser for Haskell

Other

53 stars 33 forks source link

Handle dayOfWeek+dayOfMonth the same as vixiecron #8

Closed joelwilliamson closed 10 years ago

joelwilliamson commented 10 years ago

crontab(5) says
> The day of a command's execution can be specified by two fields - day of
> month, and day of week. If both fields are restricted (ie, aren't *),
> the command will be run when either field matches the current time.

The current behaviour is to require both fields to match. Since there are no tests covering this case, I assume it wasn't considered in development.

MichaelXavier commented 10 years ago

Hi Joel. Thanks! I definitely want to match the behavior of cron. It looks like this breaks the build. Do you think you could look at the failure and add a test case that exercises this behavior? After that, I would be happy to merge your code.

joelwilliamson commented 10 years ago

This builds fine. The first commit in the series is a modification to the build process (I couldn't get cabal-dev to install).

MichaelXavier commented 10 years ago

Perfect. Thanks!