search

debops / ansible-apt

Configure and manage APT Package Manager
GNU General Public License v3.0
10 stars 15 forks source link

Proposition for fix #62 #65

Closed azman0101 closed 7 years ago

drybjed commented 8 years ago

Looks much better :-) Although, instead of

'{{ omit if (expr1) or (expr2) ellse expr3 }}'

I would go with

'{{ omit if (expr1 or expr2) else expr3 }}'

This way you will make sure that Python evaluates all of the expressions in if statement, instead of just selecting on the first one.

le9i0nx commented 8 years ago

sorry, this patch does not solve the problem of obtaining lists

drybjed commented 7 years ago

This should be solved with the v0.4.0 version of the role, so I'm closing this PR.