Move logic of gyp_webrtc into gyp_webrtc.py
For historical reasons gyp_webrtc.py was launcher script
for gyp_webrtc and the python logic lived in the
gyp_webrtc. This change moves python code into the
.py file makes the extension-free gyp_webrtc a launcher
for gyp_webrtc.py.
Other changes:
* Move the code into a main() function.
* Add call to disable GC to save some processing time.
* Set executable permission on gyp_webrtc.py and remove it from
gyp_webrtc.
Similar Chromium CL: https://codereview.chromium.org/1216863010
Motivation for this change:
* Gets checked with PyLint
* Easy to add unit tests if we add our own functionality.
R=phoglund@webrtc.org
TBR=tkchin@webrtc.org
Review URL: https://codereview.webrtc.org/1895713002 .
Cr-Commit-Position: refs/heads/master@{#12410}
diff --git a/webrtc/build/gyp_webrtc.py b/webrtc/build/gyp_webrtc.py
old mode 100644
new mode 100755
index 87d8a57..2d44971
--- a/webrtc/build/gyp_webrtc.py
+++ b/webrtc/build/gyp_webrtc.py
@@ -8,17 +8,122 @@
# in the file PATENTS. All contributing project authors may
# be found in the AUTHORS file in the root of the source tree.
-# This file is (possibly, depending on python version) imported by
-# gyp_webrtc when GYP_PARALLEL=1 and it creates sub-processes
-# through the multiprocessing library.
+# This script is used to run GYP for WebRTC. It contains selected parts of the
+# main function from the src/build/gyp_chromium.py file while other parts are
+# reused to minimize code duplication.
-# Importing in Python 2.6 (fixed in 2.7) on Windows doesn't search for
-# imports that don't end in .py (and aren't directories with an
-# __init__.py). This wrapper makes "import gyp_webrtc" work with
-# those old versions and makes it possible to execute gyp_webrtc.py
-# directly on Windows where the extension is useful.
-
+import gc
+import glob
import os
+import sys
-path = os.path.abspath(os.path.split(__file__)[0])
-execfile(os.path.join(path, 'gyp_webrtc'))
+script_dir = os.path.dirname(os.path.realpath(__file__))
+checkout_root = os.path.abspath(os.path.join(script_dir, os.pardir, os.pardir))
+
+sys.path.insert(0, os.path.join(checkout_root, 'build'))
+import gyp_chromium
+import gyp_helper
+import vs_toolchain
+
+sys.path.insert(0, os.path.join(checkout_root, 'tools', 'gyp', 'pylib'))
+import gyp
+
+
+def GetSupplementalFiles():
+ """Returns a list of the supplemental files.
+
+ A supplemental file is included in all GYP sources. Such files can be used to
+ override default values.
+ """
+ # Can't use the one in gyp_chromium since the directory location of the root
+ # is different.
+ return glob.glob(os.path.join(checkout_root, '*', 'supplement.gypi'))
+
+
+def main():
+ # Disabling garbage collection saves about 5% processing time. Since this is a
+ # short-lived process it's not a problem.
+ gc.disable()
+
+ args = sys.argv[1:]
+
+ if int(os.environ.get('GYP_CHROMIUM_NO_ACTION', 0)):
+ print 'Skipping gyp_webrtc.py due to GYP_CHROMIUM_NO_ACTION env var.'
+ sys.exit(0)
+
+ if 'SKIP_WEBRTC_GYP_ENV' not in os.environ:
+ # Update the environment based on webrtc.gyp_env.
+ gyp_env_path = os.path.join(os.path.dirname(checkout_root),
+ 'webrtc.gyp_env')
+ gyp_helper.apply_gyp_environment_from_file(gyp_env_path)
+
+ # This could give false positives since it doesn't actually do real option
+ # parsing. Oh well.
+ gyp_file_specified = False
+ for arg in args:
+ if arg.endswith('.gyp'):
+ gyp_file_specified = True
+ break
+
+ # If we didn't get a file, assume 'all.gyp' in the root of the checkout.
+ if not gyp_file_specified:
+ # Because of a bug in gyp, simply adding the abspath to all.gyp doesn't
+ # work, but chdir'ing and adding the relative path does. Spooky :/
+ os.chdir(checkout_root)
+ args.append('all.gyp')
+
+ # There shouldn't be a circular dependency relationship between .gyp files,
+ args.append('--no-circular-check')
+
+ # Default to ninja unless GYP_GENERATORS is set.
+ if not os.environ.get('GYP_GENERATORS'):
+ os.environ['GYP_GENERATORS'] = 'ninja'
+
+ # Enable check for missing sources in GYP files on Windows.
+ if sys.platform.startswith('win'):
+ gyp_generator_flags = os.getenv('GYP_GENERATOR_FLAGS', '')
+ if not 'msvs_error_on_missing_sources' in gyp_generator_flags:
+ os.environ['GYP_GENERATOR_FLAGS'] = (
+ gyp_generator_flags + ' msvs_error_on_missing_sources=1')
+
+ vs2013_runtime_dll_dirs = None, None
+ if int(os.environ.get('DEPOT_TOOLS_WIN_TOOLCHAIN', '1')):
+ vs2013_runtime_dll_dirs = vs_toolchain.SetEnvironmentAndGetRuntimeDllDirs()
+
+ # Enforce gyp syntax checking. This adds about 20% execution time.
+ args.append('--check')
+
+ supplemental_includes = GetSupplementalFiles()
+ gyp_vars = gyp_chromium.GetGypVars(supplemental_includes)
+
+ # Automatically turn on crosscompile support for platforms that need it.
+ if all(('ninja' in os.environ.get('GYP_GENERATORS', ''),
+ gyp_vars.get('OS') in ['android', 'ios'],
+ 'GYP_CROSSCOMPILE' not in os.environ)):
+ os.environ['GYP_CROSSCOMPILE'] = '1'
+
+ args.extend(['-I' + i for i in
+ gyp_chromium.additional_include_files(supplemental_includes,
+ args)])
+
+ # Set the gyp depth variable to the root of the checkout.
+ args.append('--depth=' + os.path.relpath(checkout_root))
+
+ print 'Updating projects from gyp files...'
+ sys.stdout.flush()
+
+ # Off we go...
+ gyp_rc = gyp.main(args)
+
+ if vs2013_runtime_dll_dirs:
+ x64_runtime, x86_runtime = vs2013_runtime_dll_dirs
+ vs_toolchain.CopyVsRuntimeDlls(
+ os.path.join(checkout_root, gyp_chromium.GetOutputDirectory()),
+ (x86_runtime, x64_runtime))
+
+ sys.exit(gyp_rc)
+
+
+if __name__ == '__main__':
+ sys.exit(main())
+