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

Side by Side Diff: content/content_shell.gypi

Issue 211503005: Fixup mojo dylibs to support mojo in content (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Copy explicitly instead of copies command Created 6 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 unified diff | Download patch
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'variables': { 6 'variables': {
7 'content_shell_product_name': 'Content Shell', 7 'content_shell_product_name': 'Content Shell',
8 # The "19" is so that sites that sniff for version think that this is 8 # The "19" is so that sites that sniff for version think that this is
9 # something reasonably current; the "77.34.5" is a hint that this isn't a 9 # something reasonably current; the "77.34.5" is a hint that this isn't a
10 # standard Chrome. 10 # standard Chrome.
(...skipping 835 matching lines...) Expand 10 before | Expand all | Expand 10 after
846 # doesn't currently use the SCM keys for anything, 846 # doesn't currently use the SCM keys for anything,
847 # but this seems like a really good place to store them. 847 # but this seems like a really good place to store them.
848 'postbuild_name': 'Tweak Info.plist', 848 'postbuild_name': 'Tweak Info.plist',
849 'action': ['../build/mac/tweak_info_plist.py', 849 'action': ['../build/mac/tweak_info_plist.py',
850 '--breakpad=1', 850 '--breakpad=1',
851 '--keystone=0', 851 '--keystone=0',
852 '--scm=1', 852 '--scm=1',
853 '--version=<(content_shell_version)', 853 '--version=<(content_shell_version)',
854 '--branding=<(content_shell_product_name)'], 854 '--branding=<(content_shell_product_name)'],
855 }, 855 },
856 {
857 # Copy libmojo_system.dylib explicitly. The copies command hard
Mark Mentovai 2014/03/26 16:56:55 Same comments apply here.
858 # links, and since we modify it we would end up with the wrong
859 # load command in things linked afterward.
860 'postbuild_name': 'Copy libmojo_system.dylib',
861 'action': ['cp',
862 '-p',
863 '${BUILT_PRODUCTS_DIR}/libmojo_system.dylib',
864 '${BUILT_PRODUCTS_DIR}/${CONTENTS_FOLDER_PATH}/Librarie s'],
865 },
866 {
867 # Changes the load command of libmojo_system.dylib to come from
868 # the Libraries folder.
869 'postbuild_name': 'Update load of libmojo_system.dylib',
870 'action': ['install_name_tool',
871 '-change',
872 '@loader_path/libmojo_system.dylib',
873 '@loader_path/Libraries/libmojo_system.dylib',
874 '${BUILT_PRODUCTS_DIR}/${EXECUTABLE_PATH}'],
875 },
876 {
877 # Changes the id of libmojo_system.dylib to come from the
878 # Libraries folder to be consistent with the load command.
879 'postbuild_name': 'Update id of libmojo_system.dylib',
880 'action': ['install_name_tool',
881 '-id',
882 '@loader_path/Libraries/libmojo_system.dylib',
883 '${BUILT_PRODUCTS_DIR}/${CONTENTS_FOLDER_PATH}/Librarie s/libmojo_system.dylib'],
884 },
856 ], 885 ],
857 'copies': [ 886 'copies': [
858 { 887 {
859 # Copy FFmpeg binaries for audio/video support. 888 # Copy FFmpeg for audio/video support and mojo.
860 'destination': '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Libraries', 889 'destination': '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Libraries',
861 'files': [ 890 'files': [
862 '<(PRODUCT_DIR)/ffmpegsumo.so', 891 '<(PRODUCT_DIR)/ffmpegsumo.so',
863 ], 892 ],
864 }, 893 },
865 { 894 {
866 'destination': '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Resources', 895 'destination': '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Resources',
867 'files': [ 896 'files': [
868 '<(PRODUCT_DIR)/crash_inspector', 897 '<(PRODUCT_DIR)/crash_inspector',
869 '<(PRODUCT_DIR)/crash_report_sender.app' 898 '<(PRODUCT_DIR)/crash_report_sender.app'
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
1137 '--input_symbol', '<(PRODUCT_DIR)/content_shell.exe.pdb', 1166 '--input_symbol', '<(PRODUCT_DIR)/content_shell.exe.pdb',
1138 '--destination_dir', '<(dest_dir)', 1167 '--destination_dir', '<(dest_dir)',
1139 ], 1168 ],
1140 }, 1169 },
1141 ], 1170 ],
1142 }, 1171 },
1143 ], 1172 ],
1144 }], # OS=="win" and fastbuild==0 and target_arch=="ia32" 1173 }], # OS=="win" and fastbuild==0 and target_arch=="ia32"
1145 ] 1174 ]
1146 } 1175 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698