From e951ad06f92337cf04b6c27d2655fd7c28f85209 Mon Sep 17 00:00:00 2001 From: Greg Magolan Date: Fri, 29 Mar 2024 08:52:34 -0400 Subject: [PATCH] fix: check JS_BINARY__RUNFILES env variable in @bazel/runfiles helper --- packages/runfiles/runfiles.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/runfiles/runfiles.ts b/packages/runfiles/runfiles.ts index 9b47b762a8..274d98ac58 100644 --- a/packages/runfiles/runfiles.ts +++ b/packages/runfiles/runfiles.ts @@ -29,11 +29,13 @@ export class Runfiles { this.manifest = this.loadRunfilesManifest(_env['RUNFILES_MANIFEST_FILE']!); } else if (!!_env['RUNFILES_DIR']) { this.runfilesDir = path.resolve(_env['RUNFILES_DIR']!); + } else if (!!_env['JS_BINARY__RUNFILES']) { + this.runfilesDir = path.resolve(_env['JS_BINARY__RUNFILES']!); } else if (!!_env['RUNFILES']) { this.runfilesDir = path.resolve(_env['RUNFILES']!); } else { throw new Error( - 'Every node program run under Bazel must have a $RUNFILES_DIR, $RUNFILES or $RUNFILES_MANIFEST_FILE environment variable'); + 'Every node program run under Bazel must have a $RUNFILES_MANIFEST_FILE, $RUNFILES_DIR, $JS_BINARY__RUNFILES or $RUNFILES environment variable'); } // Under --noenable_runfiles (in particular on Windows) // Bazel sets RUNFILES_MANIFEST_ONLY=1.