Changeset 59249

Timestamp:

10/18/2024 12:35:37 PM (7 months ago)

Author:

desrosj

Message:

Build/Test Tools: Change commands used for the copying .env.example file.

This switches from using the test/cp commands when copying the .env.example file to using node:fs. test and cp are not available on Windows machines.

This also adds the .env file to the svn:ignore list to prevent it from being committed accidentally.

Follow up to [59038].

Props afercia, Clorith, poena.
Fixes #52668.

Location:

trunk

Files:

• . (modified) (1 prop)
• tools/local-env/scripts/start.js (modified) (1 diff)

trunk

@@ -18 +18 @@
 # Files for local environment config
 /docker-compose.override.yml
+.env

@@ -18 +18 @@
 # Files for local environment config
 /docker-compose.override.yml
+.env

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

@@ -2 +2 @@
 const dotenvExpand = require( 'dotenv-expand' );
 const { execSync } = require( 'child_process' );
+const { constants, copyFile } = require( 'node:fs' );
 
-try {
-    execSync( 'test -f .env', { stdio: 'inherit' } );
-} catch ( e ) {
-    // test exits with a status code of 1 if the test fails.
-    // Alert the user on any other failure.
-    if ( e.status !== 1 ) {
-        throw e;
-    }
-
-    // The file does not exist, copy over the default example file.
-    execSync( 'cp .env.example .env', { stdio: 'inherit' } );
-}
-
+// Copy the default .env file when one is not present.
+copyFile( '.env.example', '.env', constants.COPYFILE_EXCL, (e) => {
+    console.log( '.env file already exists. .env.example was not copied.' );
+});
 
 dotenvExpand.expand( dotenv.config() );

@@ -18 +18 @@
 # Files for local environment config
 /docker-compose.override.yml
+.env