Merge pull request #111 from crazy-max/fix-buildx-build-err

crazy-max · web-flow · commit a5b683b46e1b · 2023-06-07T14:54:24.000+02:00
buildx(install): throw error on failed build
diff --git a/src/buildx/install.ts b/src/buildx/install.ts
@@ -86,7 +86,7 @@ export class Install {
         ignoreReturnCode: true
       }).then(res => {
         if (res.stderr.length > 0 && res.exitCode != 0) {
-          core.warning(res.stderr.trim());
+          throw new Error(`build failed with: ${res.stderr.match(/(.*)\s*$/)?.[0]?.trim() ?? 'unknown error'}`);
         }
         return tc.cacheFile(`${outputDir}/buildx`, os.platform() == 'win32' ? 'docker-buildx.exe' : 'docker-buildx', 'buildx', vspec, this.platform());
       });

Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ export class Install {`
`86`	`86`	`ignoreReturnCode: true`
`87`	`87`	`}).then(res => {`
`88`	`88`	`if (res.stderr.length > 0 && res.exitCode != 0) {`
`89`		`- core.warning(res.stderr.trim());`
	`89`	+ throw new Error(`build failed with: ${res.stderr.match(/(.)\s$/)?.[0]?.trim() ?? 'unknown error'}`);
`90`	`90`	`}`
`91`	`91`	return tc.cacheFile(`${outputDir}/buildx`, os.platform() == 'win32' ? 'docker-buildx.exe' : 'docker-buildx', 'buildx', vspec, this.platform());
`92`	`92`	`});`