Merge pull request #4797 from weaselkeeper/devel

fix for https://github.com/ansible/ansible/issues/4795 (incorrect changed result in cron module)
reviewable/pr18780/r1
jctanner 11 years ago
commit 6bbb4d20ef

@ -266,9 +266,10 @@ class CronTab(object):
def remove_job_file(self): def remove_job_file(self):
try: try:
os.unlink(self.cron_file) os.unlink(self.cron_file)
return True
except OSError, e: except OSError, e:
# cron file does not exist # cron file does not exist
return return False
except: except:
raise CronTabError("Unexpected error:", sys.exc_info()[0]) raise CronTabError("Unexpected error:", sys.exc_info()[0])
@ -464,8 +465,7 @@ def main():
crontab.write(backup_file) crontab.write(backup_file)
if crontab.cron_file and not do_install: if crontab.cron_file and not do_install:
crontab.remove_job_file() changed = crontab.remove_job_file()
changed = True
module.exit_json(changed=changed,cron_file=cron_file,state=state) module.exit_json(changed=changed,cron_file=cron_file,state=state)
job = crontab.get_cron_job(minute, hour, day, month, weekday, job, special_time) job = crontab.get_cron_job(minute, hour, day, month, weekday, job, special_time)

Loading…
Cancel
Save