Changeset 59659

Timestamp:

01/17/2025 10:35:41 AM (5 months ago)

Author:

johnbillion

Message:

Build/Test Tools: Hide the Node.js error message when a Docker command produces a non-zero exit code.

When running a command that goes via docker.js and produces a non-zero exit code, the error message and stack trace from node an safely be hidden because the stack trace only points to the execSync() call and is of no use.

Fixes #62814

File:

• trunk/tools/local-env/scripts/docker.js (modified) (1 diff)

trunk/tools/local-env/scripts/docker.js

@@ -13 +13 @@
 }
 
-// Execute any docker compose command passed to this script.
-execSync( 'docker compose ' + composeFiles + ' ' + process.argv.slice( 2 ).join( ' ' ), { stdio: 'inherit' } );
+// This try-catch prevents the superfluous Node.js debugging information from being shown if the command fails.
+try {
+    // Execute any Docker compose command passed to this script.
+    execSync( 'docker compose ' + composeFiles + ' ' + process.argv.slice( 2 ).join( ' ' ), { stdio: 'inherit' } );
+} catch ( error ) {
+    process.exit( 1 );
+}