Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Old cron dependency #55

Closed
gfrntz opened this issue Jul 14, 2017 · 10 comments
Closed

Old cron dependency #55

gfrntz opened this issue Jul 14, 2017 · 10 comments

Comments

@gfrntz
Copy link

gfrntz commented Jul 14, 2017

Hello,
Why you pin cron cookbook version to '~>1.7'? May be fix it to '> 1.7'?

Best regards

@flaccid
Copy link
Contributor

flaccid commented Jul 14, 2017

From memory this is something historic around bugs and support issues with the distros which are probably solved now. I can remove the pin, but we'll need to do some pretty thorough semi/manual testing?

@flaccid
Copy link
Contributor

flaccid commented Jul 14, 2017

@gfrntz ok thats relaxed in master now. Be great if you could help with some general manual testing across the board!

@gfrntz
Copy link
Author

gfrntz commented Jul 14, 2017

I'll try to test it. Bump version, please. Thanks.

@flaccid
Copy link
Contributor

flaccid commented Jul 14, 2017

No new version until we test.

@gfrntz
Copy link
Author

gfrntz commented Jul 14, 2017

Ok, thanks.

@olbrich
Copy link

olbrich commented Oct 4, 2017

any progress on testing this?

@gfrntz
Copy link
Author

gfrntz commented Oct 4, 2017

So, i can't found any problems with this merge in my infrastructure.

@flaccid
Copy link
Contributor

flaccid commented Oct 16, 2017

To complete, this is more about updating the dev files and making this pass. Its a bit of an uplift I didn't find time to finish. Let me resurrect it now.

@flaccid
Copy link
Contributor

flaccid commented Nov 17, 2017

Everything we need for this including the dev/style fixes is in master now. I'm going to see if there is anything else I should include before cutting a release. All the major distros are passing fine.

@flaccid
Copy link
Contributor

flaccid commented Nov 17, 2017

New release will be out shortly.

@flaccid flaccid closed this as completed Nov 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants