Fix the docs build (#30058)

Type inference was incorrectly inferring that our `check_output()`
helper was returning `str` when in reality, it returns `bytes`. This
fixes the caller that was no longer decoding those bytes and fixes the
type annotation on the function.

1 files changed, 5 insertions, 5 deletions

diff --git a/python/servo/command_base.py b/python/servo/command_base.py
index ce8c1ddb237..43d2e97d89c 100644
--- a/python/servo/command_base.py
+++ b/python/servo/command_base.py
@@ -141,7 +141,7 @@ def call(*args, **kwargs):
     return subprocess.call(*args, shell=sys.platform == 'win32', **kwargs)
 
 
-def check_output(*args, **kwargs):
+def check_output(*args, **kwargs) -> bytes:
     """Wrap `subprocess.call`, printing the command if verbose=True."""
     verbose = kwargs.pop('verbose', False)
     if verbose:
@@ -980,10 +980,10 @@ class CommandBase(object):
 
         servo.platform.get().passive_bootstrap()
 
-        needs_toolchain_install = self.cross_compile_target \
-            and self.cross_compile_target not in check_output(
-                ["rustup", "target", "list", "--installed"], cwd=self.context.topdir
-            )
+        needs_toolchain_install = self.cross_compile_target and \
+            self.cross_compile_target not in \
+            check_output(["rustup", "target", "list", "--installed"],
+                         cwd=self.context.topdir).decode()
         if needs_toolchain_install:
             check_call(["rustup", "target", "add", self.cross_compile_target],
                        cwd=self.context.topdir)