Good point, I've made this change and will push shortly.

On 04/02/14 15:32, Chris St. Pierre wrote:
> In src/lib/Bcfg2/Client/Tools/POSIX/base.py:
>
>>                  if acl.tag_type == posix1e.ACL_USER:
>>                      qual = pwd.getpwuid(acl.qualifier)[0]
>>                  elif acl.tag_type == posix1e.ACL_GROUP:
>>                      qual = grp.getgrgid(acl.qualifier)[0]
>> -                else:
>> +                elif atype == "access":
>> +                    return
>> +                elif acl.tag_type == posix1e.ACL_MASK:
>
> Could be done more simply as:
>
> |elif atype == "access" or acl.tag_type == posix1e.ACL_MAS:
>      return
> |
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Bcfg2/bcfg2/pull/156/files#r9424185>.
>