Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(231)

Unified Diff: webrtc/build/gyp_webrtc

Issue 1895713002: Move logic of gyp_webrtc into gyp_webrtc.py (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fix PyLint warning Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « talk/app/webrtc/objc/README ('k') | webrtc/build/gyp_webrtc.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/build/gyp_webrtc
diff --git a/webrtc/build/gyp_webrtc b/webrtc/build/gyp_webrtc
old mode 100755
new mode 100644
index 2376cfbb85c6c7649c503d2ff4e2a024c9c8fb7d..c0a9ed9da8efb004074e2d4825664cfa42b9d4d6
--- a/webrtc/build/gyp_webrtc
+++ b/webrtc/build/gyp_webrtc
@@ -8,108 +8,10 @@
# in the file PATENTS. All contributing project authors may
# be found in the AUTHORS file in the root of the source tree.
-# This script is used to run GYP for WebRTC. It contains selected parts of the
-# main function from the src/build/gyp_chromium file.
+# Simple launcher script for gyp_webrtc.py.
+# TODO(kjellander): This should probably be shell script but for historical
+# reasons (all the python code used to live in this script without a
+# .py extension, and was often run as 'python gyp_webrtc') it is
+# currently still python.
-import glob
-import os
-import shlex
-import sys
-
-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 that are included in all GYP
- sources."""
- # 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'))
-
-
-if __name__ == '__main__':
- args = sys.argv[1:]
-
- if int(os.environ.get('GYP_CHROMIUM_NO_ACTION', 0)):
- print 'Skipping gyp_webrtc 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
- 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)
+execfile(__file__ + '.py')
« no previous file with comments | « talk/app/webrtc/objc/README ('k') | webrtc/build/gyp_webrtc.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698