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

Unified Diff: chrome/chrome_dll_bundle.gypi

Issue 211503005: Fixup mojo dylibs to support mojo in content (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add isolate support Created 6 years, 9 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 | « chrome/chrome.isolate ('k') | chrome/tools/build/mac/dump_product_syms » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_dll_bundle.gypi
diff --git a/chrome/chrome_dll_bundle.gypi b/chrome/chrome_dll_bundle.gypi
index a4b6d6cffc2f5559238b550761b2eb3fcc6ba7fb..dad94c1f8354e743ae89bff8eb1fd93c7b24b8e4 100644
--- a/chrome/chrome_dll_bundle.gypi
+++ b/chrome/chrome_dll_bundle.gypi
@@ -128,6 +128,35 @@
'--branding=<(branding)'],
},
{
+ # Copy libmojo_system.dylib explicitly. The copies section does a
+ # hard link, and since we modify it we would end up with the wrong
+ # load command in things linked afterward.
+ 'postbuild_name': 'Copy libmojo_system.dylib',
+ 'action': ['cp',
+ '-p',
+ '${BUILT_PRODUCTS_DIR}/libmojo_system.dylib',
+ '${BUILT_PRODUCTS_DIR}/${CONTENTS_FOLDER_PATH}/Libraries'],
+ },
+ {
+ # Changes the id of libmojo_system.dylib to come from the Libraries folder
+ # to be consistent with the load command.
+ 'postbuild_name': 'Update id of libmojo_system.dylib',
+ 'action': ['install_name_tool',
+ '-id',
+ '@loader_path/Libraries/libmojo_system.dylib',
+ '${BUILT_PRODUCTS_DIR}/${CONTENTS_FOLDER_PATH}/Libraries/libmojo_system.dylib'],
+ },
+ {
+ # Changes the load command of libmojo_system.dylib to come from
+ # the Libraries folder.
+ 'postbuild_name': 'Update load of libmojo_system.dylib',
+ 'action': ['install_name_tool',
+ '-change',
+ '@loader_path/libmojo_system.dylib',
+ '@loader_path/Libraries/libmojo_system.dylib',
+ '${BUILT_PRODUCTS_DIR}/${EXECUTABLE_PATH}'],
+ },
+ {
'postbuild_name': 'Symlink Libraries',
'action': [
'ln',
@@ -139,7 +168,7 @@
],
'copies': [
{
- # Copy FFmpeg binaries for audio/video support.
+ # Copy FFmpeg binaries for audio/video support and mojo.
'destination': '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Libraries',
'files': [
'<(PRODUCT_DIR)/ffmpegsumo.so',
« no previous file with comments | « chrome/chrome.isolate ('k') | chrome/tools/build/mac/dump_product_syms » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698