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

Unified Diff: chrome/tools/build/win/create_installer_archive.py

Issue 53793002: Initial implementation of Chrome Early Loading Framework (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Assemblies Created 7 years, 1 month 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
« chrome/chrome_exe.gypi ('K') | « chrome/installer/mini_installer/chrome.release ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/tools/build/win/create_installer_archive.py
diff --git a/chrome/tools/build/win/create_installer_archive.py b/chrome/tools/build/win/create_installer_archive.py
index f5a99987adb5a204e19d12051e124a058845edf9..7cb5cc2d023609b37c3d7e2cc6fa75faa8f6af7c 100755
--- a/chrome/tools/build/win/create_installer_archive.py
+++ b/chrome/tools/build/win/create_installer_archive.py
@@ -348,6 +348,26 @@ def CopyAndAugmentManifest(build_dir, output_dir, manifest_name,
modified_manifest_file.write(''.join(manifest_lines))
modified_manifest_file.close()
+def AddVersionAssemblyManifest(staging_dir, current_version):
grt (UTC plus 2) 2013/11/05 03:08:36 nit: add a doc string
Cait (Slow) 2013/11/05 05:39:06 Done.
+ # Get the required directories for the upcoming operations.
+ chrome_dir = os.path.join(staging_dir, CHROME_DIR)
+ version_dir = os.path.join(chrome_dir, current_version)
+
+ version_manifest = (
+ "<assembly\n"
+ " xmlns='urn:schemas-microsoft-com:asm.v1' manifestVersion='1.0'>\n"
+ " <assemblyIdentity name='{version}' version='{version}'\n"
+ " type='win32'/>\n"
+ " <file name='chrome_elf.dll'/>\n"
+ "</assembly>".format(version=current_version))
+
+ manifest_name = ("{version}.manifest".format(version=current_version))
robertshield 2013/11/05 02:23:55 nit: don't need the outer ()
grt (UTC plus 2) 2013/11/05 03:08:36 feel free to ignore this nit: "%s.manifest" % cu
Cait (Slow) 2013/11/05 05:39:06 Done.
+
+ version_manifest_file = open(
grt (UTC plus 2) 2013/11/05 03:08:36 with open(os.path.join(version_dir, manifest_name)
Cait (Slow) 2013/11/05 05:39:06 Done.
+ os.path.join(version_dir, manifest_name), 'w')
+ version_manifest_file.write(version_manifest)
+ version_manifest_file.close()
+
def CopyIfChanged(src, target_dir):
"""Copy specified |src| file to |target_dir|, but only write to target if
@@ -557,6 +577,8 @@ def main(options):
DoComponentBuildTasks(staging_dir, options.build_dir,
options.target_arch, current_version)
+ AddVersionAssemblyManifest(staging_dir, current_version)
+
version_numbers = current_version.split('.')
current_build_number = version_numbers[2] + '.' + version_numbers[3]
prev_build_number = ''
« chrome/chrome_exe.gypi ('K') | « chrome/installer/mini_installer/chrome.release ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698