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

Unified Diff: tools/create_sdk.py

Issue 2826793002: [Fuchsia] Build only the parts of the SDK that are needed (Closed)
Patch Set: Back out bad fix Created 3 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 | « tools/copy_tree.py ('k') | tools/write_revision_file.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/create_sdk.py
diff --git a/tools/create_sdk.py b/tools/create_sdk.py
index 0260693f6ee08e29305370c3f2c495b927145189..b93d0e4278c3cd68483062cb7eb80bc64f609dd3 100755
--- a/tools/create_sdk.py
+++ b/tools/create_sdk.py
@@ -94,28 +94,9 @@ def GetOptions():
help='Where to output the sdk')
options.add_option("--snapshot_location",
help='Location of the snapshots.')
- options.add_option("--copy_libs",
- action="store_true", default=False,
- help='Copy dynamically linked libraries to the SDK bin directory.')
- options.add_option("--disable_stripping",
- action="store_true", default=False,
- help='Do not try to strip binaries. Use when they are already stripped')
return options.parse_args()
-def ReplaceInFiles(paths, subs):
- """Reads a series of files, applies a series of substitutions to each, and
- saves them back out. subs should by a list of (pattern, replace) tuples."""
- for path in paths:
- contents = open(path).read()
- for pattern, replace in subs:
- contents = re.sub(pattern, replace, contents)
-
- dest = open(path, 'w')
- dest.write(contents)
- dest.close()
-
-
def Copy(src, dest):
copyfile(src, dest)
copymode(src, dest)
@@ -138,20 +119,6 @@ def CopyShellScript(src_file, dest_dir):
Copy(src, dest)
-def CopyLibs(out_dir, bin_dir):
- for library in ['libcrypto', 'libssl']:
- ext = '.so'
- if HOST_OS == 'macos':
- ext = '.dylib'
- elif HOST_OS == 'win32':
- ext = '.dll'
- src = os.path.join(out_dir, library + ext)
- dst = os.path.join(bin_dir, library + ext)
- if os.path.isfile(src):
- copyfile(src, dst)
- copymode(src, dst)
-
-
def CopyDartScripts(home, sdk_root):
for executable in ['dart2js_sdk', 'dartanalyzer_sdk', 'dartfmt_sdk',
'pub_sdk', 'dartdoc', 'dartdevc_sdk']:
@@ -243,9 +210,9 @@ def Main():
copyfile(dart_src_binary, dart_dest_binary)
copymode(dart_src_binary, dart_dest_binary)
# Strip the binaries on platforms where that is supported.
- if HOST_OS == 'linux' and not options.disable_stripping:
+ if HOST_OS == 'linux':
subprocess.call(['strip', dart_dest_binary])
- elif HOST_OS == 'macos' and not options.disable_stripping:
+ elif HOST_OS == 'macos':
subprocess.call(['strip', '-x', dart_dest_binary])
#
@@ -336,9 +303,6 @@ def Main():
CopyAnalysisSummaries(SNAPSHOT, LIB)
CopyDevCompilerSdk(HOME, LIB)
- if options.copy_libs:
- CopyLibs(build_dir, BIN)
-
# Write the 'version' file
version = utils.GetVersion()
versionFile = open(os.path.join(SDK_tmp, 'version'), 'w')
« no previous file with comments | « tools/copy_tree.py ('k') | tools/write_revision_file.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698