From 870526374cd13759fba0bb055408d86663a4fbcc Mon Sep 17 00:00:00 2001
From: himself65 <himself65@mask.io>
Date: Thu, 8 Jul 2021 00:19:00 +0800
Subject: [PATCH] build: add `library_files` to gyp variables
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

GYP uses the system path when parsing node.gyp;
However, if system python is different from our
gyp runtime python, like '2.7', gyp would crash.

Co-authored-by: Michaël Zasso <targos@protonmail.com>

PR-URL: https://github.com/nodejs/node/pull/39293
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
---
 configure.py          |  4 ++++
 node.gyp              |  2 +-
 tools/search_files.py | 22 ----------------------
 3 files changed, 5 insertions(+), 23 deletions(-)
 delete mode 100644 tools/search_files.py

diff --git a/configure.py b/configure.py
index 9dc5d12cbdbd7c..3be42d496f2453 100755
--- a/configure.py
+++ b/configure.py
@@ -44,6 +44,7 @@
 import getmoduleversion
 import getnapibuildversion
 from gyp_node import run_gyp
+from utils import SearchFiles
 
 # imports in deps/v8/tools/node
 sys.path.insert(0, os.path.join('deps', 'v8', 'tools', 'node'))
@@ -1062,6 +1063,8 @@ def gcc_version_ge(version_checked):
       return False
   return True
 
+def configure_node_lib_files(o):
+  o['variables']['node_library_files'] = SearchFiles('lib', 'js')
 
 def configure_node(o):
   if options.dest_os == 'android':
@@ -1805,6 +1808,7 @@ def make_bin_override():
 flavor = GetFlavor(flavor_params)
 
 configure_node(output)
+configure_node_lib_files(output)
 configure_napi(output)
 configure_library('zlib', output)
 configure_library('http_parser', output)
diff --git a/node.gyp b/node.gyp
index cc3c87b3e29819..ab71ab17879e1c 100644
--- a/node.gyp
+++ b/node.gyp
@@ -33,7 +33,7 @@
     # Windows command length limit or there would be an error.
     # See https://docs.microsoft.com/en-us/troubleshoot/windows-client/shell-experience/command-line-string-limitation
     'library_files': [
-      '<!@(python tools/search_files.py --ext js lib)',
+      '<@(node_library_files)',
     ],
     'deps_files': [
       'deps/v8/tools/splaytree.js',
diff --git a/tools/search_files.py b/tools/search_files.py
deleted file mode 100644
index e236eee47dc852..00000000000000
--- a/tools/search_files.py
+++ /dev/null
@@ -1,22 +0,0 @@
-#!/usr/bin/env python
-
-"""
-This is a utility for recursively searching files under
-a specified directory
-"""
-
-import argparse
-import utils
-
-def main():
-  parser = argparse.ArgumentParser(
-    description='Search files with a specific extension under a directory',
-    fromfile_prefix_chars='@'
-  )
-  parser.add_argument('--ext', required=True, help='extension to search for')
-  parser.add_argument('directory', help='input directory')
-  options = parser.parse_args()
-  print('\n'.join(utils.SearchFiles(options.directory, options.ext)))
-
-if __name__ == "__main__":
-  main()