only add macOS if we're looking at a default pkg_group

jessebot · Dec 16, 2022 · d8856d1 · d8856d1
1 parent 956bf1d
commit d8856d1
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/onboardme/pkg_management.py b/onboardme/pkg_management.py
@@ -65,9 +65,10 @@ def run_pkg_mngrs(pkg_mngrs=[], pkg_groups=[]):
         # brew has a special flow because it works on both linux and mac
         if pkg_mngr == 'brew':
             if 'Darwin' in OS:
-                if type(pkg_groups) is tuple and 'default' in pkg_groups:
-                    pkg_groups = list(pkg_groups)
-                pkg_groups.append("macOS")
+                if 'default' in pkg_groups:
+                    if type(pkg_groups) is tuple:
+                        pkg_groups = list(pkg_groups)
+                    pkg_groups.append("macOS")
 
         log.debug(f"pkg groups for {pkg_mngr} are {available_pkg_groups}")