From c10a97c41dfb039515f798ab14a07fb664abc97b Mon Sep 17 00:00:00 2001
From: Antoine Pitrou <antoine@python.org>
Date: Mon, 28 Mar 2022 17:54:11 +0200
Subject: [PATCH] MINOR: [CI] Improve conda-clean output on error

When a subprocess errors, display its stderr instead of just complaining about the exit status.
---
 dev/tasks/conda-recipes/clean.py | 24 +++++++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)

diff --git a/dev/tasks/conda-recipes/clean.py b/dev/tasks/conda-recipes/clean.py
index d5ed5429e0ff6..6a101e7c6bb2f 100644
--- a/dev/tasks/conda-recipes/clean.py
+++ b/dev/tasks/conda-recipes/clean.py
@@ -1,4 +1,4 @@
-from subprocess import check_output, check_call
+import subprocess
 from typing import Set
 
 import json
@@ -20,8 +20,26 @@
 ]
 
 
+def run_command(cmdline, **kwargs):
+    kwargs.setdefault('capture_output', True)
+    p = subprocess.run(cmdline, **kwargs)
+    if p.returncode != 0:
+        print(f"Command {cmdline} returned non-zero exit status "
+              f"{p.returncode}", file=sys.stderr)
+        if p.stdout:
+            print("Stdout was:\n" + "-" * 70, file=sys.stderr)
+            print(p.stdout.decode().rstrip())
+            print("-" * 70, file=sys.stderr)
+        if p.stderr:
+            print("Stderr was:\n" + "-" * 70, file=sys.stderr)
+            print(p.stderr.decode().rstrip())
+            print("-" * 70, file=sys.stderr)
+        sys.exit(1)
+    return p.stdout
+
+
 def builds_to_delete(platform: str, to_delete: Set[str]) -> int:
-    pkgs_json = check_output(
+    pkgs_json = run_command(
         [
             "conda",
             "search",
@@ -93,4 +111,4 @@ def builds_to_delete(platform: str, to_delete: Set[str]) -> int:
 
     if "FORCE" in sys.argv and len(to_delete) > 0:
         print("Deleting ...")
-        check_call(["anaconda", "remove", "-f"] + to_delete)
+        run_command(["anaconda", "remove", "-f"] + to_delete)