samba.upgrade: Use list comprehension.
authorJelmer Vernooij <jelmer@samba.org>
Wed, 12 Oct 2011 18:12:26 +0000 (20:12 +0200)
committerJelmer Vernooij <jelmer@samba.org>
Wed, 12 Oct 2011 20:44:40 +0000 (22:44 +0200)
Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Wed Oct 12 22:44:40 CEST 2011 on sn-devel-104

source4/scripting/python/samba/upgrade.py

index 53ae93afcfa7ed723078c8a3ef007953c661d6c1..b41ab77eb23b9ee5c89aaf1a4c34e3fffafd36bf 100644 (file)
@@ -602,8 +602,8 @@ Please fix this account before attempting to upgrade again
     logger.info("Next rid = %d", next_rid)
 
     # Check for same username/groupname
-    group_names = set(map(lambda g: g.nt_name, grouplist))
-    user_names = set(map(lambda u: u['account_name'], userlist))
+    group_names = set([g.nt_name for g in grouplist])
+    user_names = set([u['account_name'] for u in userlist])
     common_names = group_names.intersection(user_names)
     if common_names:
         logger.error("Following names are both user names and group names:")
@@ -612,8 +612,8 @@ Please fix this account before attempting to upgrade again
         raise ProvisioningError("Please remove common user/group names before upgrade.")
 
     # Check for same user sid/group sid
-    group_sids = set(map(lambda g: str(g.sid), grouplist))
-    user_sids = set(map(lambda u: "%s-%u" % (domainsid, u['rid']), userlist))
+    group_sids = set([str(g.sid) for g in grouplist])
+    user_sids = set(["%s-%u" % (domainsid, u['rid']) for u in userlist])
     common_sids = group_sids.intersection(user_sids)
     if common_sids:
         logger.error("Following sids are both user and group sids:")