Skip to content

Commit a34add7

Browse files
jhheiderCopilot
andauthored
Apply suggestions from code review
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
1 parent b1bb2ec commit a34add7

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

build/build.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ platform_cache(() => config.path.home).mkdir('p') // we’ve indeed found thing
135135
const proc = new Deno.Command(script.string, {clearEnv: true, env}).spawn()
136136
const rv = await proc.status
137137
if (!rv.success) {
138-
// if DEBUG=1 or RUNNER_DEBUG=1, we’ll see the config tool output in the logs
138+
// If verbosity is debug (DEBUG=1, or GITHUB_ACTIONS=true with RUNNER_DEBUG=1), print config logs to aid diagnosis
139139
if (verbosity >= Verbosity.debug) {
140140
const wanted = new Set(['config.log', 'CMakeError.log', 'CMakeOutput.log', 'meson-log.txt'])
141141
try {
@@ -147,7 +147,7 @@ if (!rv.success) {
147147
console.debug(`==== END ${path} ====`)
148148
}
149149
}
150-
} catch { console.warn("failed to read config logs") }
150+
} catch (err) { console.warn("failed to read config logs:", err) }
151151
}
152152
throw new Error(`UR BUILD FAILED WITH CODE ${rv.code} & SIGNAL ${rv.signal}`)
153153
}

0 commit comments

Comments
 (0)