fix: check capturing stderr of cargo, mixing it with stdout

When ./check runs a command, in the runcmd_unchecked method, it
collects by default stdout and stderr into on string. The two outputs
get mixed. So far, this has been OK, but I'm about to make change
where it won't be OK. The get_template method needs to be able to
distinguish between stdout and stderr, when I change Subplot to write
warnings into stderr.

Sponsored-by: author

1 files changed, 5 insertions, 2 deletions

diff --git a/check b/check
index 533ac50..9dac5d8 100755
--- a/check
+++ b/check
@@ -58,8 +58,10 @@ class Runcmd:
         self.msg(f"RUN: {argv} {kwargs}")
 
         if not self._verbose:
-            kwargs["stdout"] = PIPE
-            kwargs["stderr"] = STDOUT
+            if "stdout" not in kwargs:
+                kwargs["stdout"] = PIPE
+            if "stderr" not in kwargs:
+                kwargs["stderr"] = STDOUT
 
         assert "key" not in kwargs
         env = dict(os.environ)
@@ -179,6 +181,7 @@ class Runcmd:
                 filename,
             ],
             stdout=PIPE,
+            stderr=PIPE,
         ).stdout.decode("UTF-8")
         metadata = json.loads(metadata)
         impls = metadata.get("impls", {})