Skip to content

Commit f89814f

Browse files
committed
macos: Fix duplicate entries in checkMacPermissions
Replace the deniedPermissions list with a set to remove duplicates.
1 parent 411364d commit f89814f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

checks/macos.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ def checkRosettaTranslationStatus(lines):
7373

7474
def checkMacPermissions(lines):
7575
macPerms = search('[macOS] Permission for', lines)
76-
deniedPermissions = []
76+
deniedPermissions = set()
7777

7878
for line in macPerms:
7979
if 'denied' in line:
@@ -88,7 +88,7 @@ def checkMacPermissions(lines):
8888
}.get(permissionName)
8989

9090
if permissionDescription:
91-
deniedPermissions.append(permissionDescription)
91+
deniedPermissions.add(permissionDescription)
9292

9393
if deniedPermissions:
9494
deniedPermissionsString = "<br>\n<ul>\n<li>" + "</li>\n<li>".join(deniedPermissions) + "</li>\n</ul>"

0 commit comments

Comments
 (0)