More robust kup publish (#87)

Author: goodlyrottenapple

CHANGELOG

@@ -1,3 +1,7 @@
+# 0.2.2
+
+* `kup publish` returns exit code 1 if `cachix push/pin` commands fail
+
 # 0.2.1
 
 * Allow arbitrary overrides of the form `kup install ... --override ... github:...`

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "kup"
-version = "0.2.1"
+version = "0.2.2"
 description = "kup is a tool for managing installations of the K framework along with the different available semantics"
 authors = [
     "Runtime Verification, Inc. <contact@runtimeverification.com>",

src/kup/__main__.py

@@ -739,11 +739,19 @@ def publish_package(cache: str, uri_or_path_with_package_name: str, keep_days: O
         rich.print('❗ [red]Could not find out path for package!')
         sys.exit(1)
 
-    subprocess.call(['cachix', 'push', cache, nix_store_path])
-    pin_args = ['cachix', 'pin', cache, cache_key, nix_store_path] + (
-        ['--keep-days', str(keep_days)] if keep_days else []
-    )
-    subprocess.call(pin_args)
+    try:
+        subprocess.run(['cachix', 'push', cache, nix_store_path], check=True)
+    except Exception:
+        rich.print('❗ [red]Could not push binaries to cachix!')
+        sys.exit(1)
+    try:
+        pin_args = ['cachix', 'pin', cache, cache_key, nix_store_path] + (
+            ['--keep-days', str(keep_days)] if keep_days else []
+        )
+        subprocess.run(pin_args, check=True)
+    except Exception:
+        rich.print('❗ [red]Could not pin package! Make sure you have cachix >=1.6')
+        sys.exit(1)
 
 
 def print_help(subcommand: str, parser: ArgumentParser) -> None: