From 6c97f21f0914bdfbd576b3e439ba8f5829488f76 Mon Sep 17 00:00:00 2001 From: Fabian Meumertzheim Date: Tue, 28 Mar 2023 09:00:23 +0200 Subject: [PATCH] Use `ctx.readdir` instead of `ls` for SDK platform detection --- go/private/sdk.bzl | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/go/private/sdk.bzl b/go/private/sdk.bzl index 58f546ffe0..c83839de9f 100644 --- a/go/private/sdk.bzl +++ b/go/private/sdk.bzl @@ -495,14 +495,12 @@ def _detect_host_sdk(ctx): def _detect_sdk_platform(ctx, goroot): path = goroot + "/pkg/tool" - res = ctx.execute(["ls", path]) - if res.return_code != 0: - fail("Could not detect SDK platform: unable to list %s: %s" % (path, res.stderr)) + tool_entries = ctx.path(path).readdir() platforms = [] - for f in res.stdout.strip().split("\n"): - if f.find("_") >= 0: - platforms.append(f) + for f in tool_entries: + if f.basename.find("_") >= 0: + platforms.append(f.basename) if len(platforms) == 0: fail("Could not detect SDK platform: found no platforms in %s" % path)