use logrotate::cron instead of a broken file resource

[zeromind]

Use logrorate::cron instead of a file resource that tries to use a file that has not been around since module version 1.4.0.

This resolves the error when including logrotate::hourly;
Error: /Stage[main]/Logrotate::Hourly/File[/etc/cron.hourly/logrotate]: Could not evaluate: Could not retrieve information from environment production source(s) puppet:///modules/logrotate/etc/cron.hourly/logrotate

[bastelfreak]

Hi @zeromind, thanks for this PR!
Two little things:

• Can you check the used email address in your commit? It is not associated with your github account
• Can you add a little spec test that ensures this now works?

[tux-o-matic]

I created issue #85 before noticing this outstanding PR to fix the problem. +1 to see this PR merged and shipped in a new release on Forge.

[zeromind]

@bastelfreak

1. I've added the email address to my Github account now.
2. I do not know how to write tests.

[osterhausm]

Any hope of getting this merged in?

[den-is]

+1 when this fix for annoyance will be published ?

[joshuaspence]

+1. The logrotate::hourly class is currently broken

[zeromind]

Seeems it got fixed via another pull request (#105), closing.

[den-is]

is anybody going to release this fix ?