Set non-zero exit codes for custom component build and install comman…

…ds when failures occur (#9088) * Exit codes * add changeset * Fix text --------- Co-authored-by: gradio-pr-bot <[email protected]>
gradio-app · Aug 12, 2024 · 96f8ffa · 96f8ffa
1 parent 508ac84
commit 96f8ffa
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/.changeset/every-parks-count.md b/.changeset/every-parks-count.md
@@ -0,0 +1,5 @@
+---
+"gradio": patch
+---
+
+fix:Set non-zero exit codes for custom component build and install commands when failures occur
diff --git a/gradio/cli/commands/components/build.py b/gradio/cli/commands/components/build.py
@@ -162,7 +162,7 @@ def _build(
                 live.update(":red_square: Build failed!")
                 live.update(pipe.stderr)
                 live.update(pipe.stdout)
-                return
+                raise SystemExit("Frontend build failed")
             else:
                 live.update(":white_check_mark: Build succeeded!")
 
@@ -172,6 +172,7 @@ def _build(
         if pipe.returncode != 0:
             live.update(":red_square: Build failed!")
             live.update(pipe.stderr)
+            raise SystemExit("Python build failed")
         else:
             live.update(":white_check_mark: Build succeeded!")
             live.update(

diff --git a/gradio/cli/commands/components/install_component.py b/gradio/cli/commands/components/install_component.py
@@ -71,6 +71,8 @@ def _install_command(
     if pipe.returncode != 0:
         live.update(":red_square: Python installation [bold][red]failed[/][/]")
         live.update(pipe.stderr)
+        raise SystemExit("Python installation failed")
+
     else:
         live.update(":white_check_mark: Python install succeeded!")
 
@@ -85,6 +87,7 @@ def _install_command(
             live.update(":red_square: NPM install [bold][red]failed[/][/]")
             live.update(pipe.stdout)
             live.update(pipe.stderr)
+            raise SystemExit("NPM install failed")
         else:
             live.update(":white_check_mark: NPM install succeeded!")