Created attachment 170722 [details]
ansible `cron' module does not work correctly: when writing crontab, it passes arguments to crontab(1) in wrong order (`crontab <newcrontabfile> -u <user>' instead of `crontab -u <user> <newcrontabfile>'). This leads to rewriting root's crontab instead of appending user's. The fix was already upstreamed:
but since it'll take some time to get into the release, it'd be nice to have it in the port right now. MFH candidate as well.
Created attachment 170725 [details]
fix crontab module
Please use this instead. It rebases the patch on ansible 22.214.171.124
and lists the patch and uses the EXTRA_PATCHES approach to avoid
Thanks for the fix! I had to do a similar thing with "at" module
a while ago.
In order to merge to stable, you can merge it together with r415910,
or, if you would prefer, add this one as a second line to EXTRA_PATCHES.
A commit references this bug:
Date: Sat Jun 11 00:49:53 UTC 2016
New revision: 416699
sysutils/ansible: add patch to fix cron module
Submitted by: Dmitry Marakasov <amdmi3@FreeBSD.org>
Submitted by: Nikolai Lifanov <firstname.lastname@example.org> (maintainer)
Approved by: bdrewery (implicit, mentor)
Committed, thank you for your submission!