From 3568f1cd786ae25595542c625ef576f55f93a980 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=A9r=C3=A9mie=20Astori?= Date: Thu, 20 Aug 2015 22:40:15 +0000 Subject: [PATCH] Make sure permission-less entries are accepted when state=absent Also, remove that try condition as, at that stage, no permissions with other than 2 or 3 fields are sent to the function. --- lib/ansible/modules/files/acl.py | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/lib/ansible/modules/files/acl.py b/lib/ansible/modules/files/acl.py index 2bd27f621f3..58550c19124 100644 --- a/lib/ansible/modules/files/acl.py +++ b/lib/ansible/modules/files/acl.py @@ -127,12 +127,10 @@ def split_entry(entry): ''' splits entry and ensures normalized return''' a = entry.split(':') - a.reverse() - try: - p, e, t = a - except ValueError, e: - print "wtf?? %s => %s" % (entry, a) - raise e + if len(a) == 2: + a.append(None) + + t, e, p = a if t.startswith("u"): t = "user"