refactor: Always set explicitly run check parameter

The default is False, to prevent accidently not checking the return
code, always set it explicitly

2 files changed, 4 insertions, 3 deletions

diff --git a/mac/makebundle.py b/mac/makebundle.py
index 299e5b2f5..5505da290 100755
--- a/mac/makebundle.py
+++ b/mac/makebundle.py
@@ -31,7 +31,7 @@ if __name__ == '__main__':
     run(['cp', 'mac/gajim.spec', 'gajim.spec'], check=True)
     run(['pyinstaller', 'gajim.spec'], check=True)
     # we only want Gajim.app in the dmg
-    run(['rm', '-rf', 'dist/launch'])
+    run(['rm', '-rf', 'dist/launch'], check=True)
     run(['hdiutil',
          'create',
          '-volname',
diff --git a/mac/makeicns.py b/mac/makeicns.py
index 20cafe1a6..562b6bd34 100755
--- a/mac/makeicns.py
+++ b/mac/makeicns.py
@@ -21,8 +21,9 @@ def create_icns(icon_path: str) -> None:
             png_path = os.path.join(tmpdir, png_fn)
             run(['inkscape', '-z', '-e', png_path,
                  '-w', str(size_px), '-h', str(size_px), '-y', '0',
-                 ICON_SVG])
-    run(['iconutil', '-c', 'icns', '-o', icon_path, tmpdir])
+                 ICON_SVG],
+                 check=True)
+    run(['iconutil', '-c', 'icns', '-o', icon_path, tmpdir], check=True)
 
     shutil.rmtree(tmpdir)