Prefer the 'form' login manager in ipa setups

Signed-off-by: Simo Sorce <simo@redhat.com>
Reviewed-by: Patrick Uiterwijk
- Replaced "all(lm not in" with "not any(lm in"

diff --git a/ipsilon/helpers/ipa.py b/ipsilon/helpers/ipa.py
index 54d5199..4a4849a 100755 (executable)
--- a/ipsilon/helpers/ipa.py
+++ b/ipsilon/helpers/ipa.py
@@ -183,6 +183,6 @@ class Installer(object):
         opts['krb'] = 'yes'
         if 'krb' not in opts['lm_order']:
             opts['lm_order'].insert(0, 'krb')
-        opts['pam'] = 'yes'
-        if 'pam' not in opts['lm_order']:
-            opts['lm_order'].append('pam')
+        opts['form'] = 'yes'
+        if not any(lm in opts['lm_order'] for lm in ('form', 'pam')):
+            opts['lm_order'].append('form')