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

Side by Side Diff: third_party/widevine/cdm/widevine_cdm.gyp

Issue 11227048: Fix linking issue with CDM plugin wrapper and pre-built CDM. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 2 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 'widevine_cdm_version_h_file%': 'widevine_cdm_version.h', 7 'widevine_cdm_version_h_file%': 'widevine_cdm_version.h',
8 'widevine_cdm_binary_files%': [], 8 'widevine_cdm_binary_files%': [],
9 'conditions': [ 9 'conditions': [
10 [ 'branding == "Chrome"', { 10 [ 'branding == "Chrome"', {
11 'conditions': [ 11 'conditions': [
12 [ 'chromeos == 1', { 12 [ 'chromeos == 1', {
13 'widevine_cdm_version_h_file%': 13 'widevine_cdm_version_h_file%':
14 'symbols/chromeos/<(target_arch)/widevine_cdm_version.h', 14 'symbols/chromeos/<(target_arch)/widevine_cdm_version.h',
15 'widevine_cdm_binary_files%': [ 15 'widevine_cdm_binary_files%': [
16 'binaries/chromeos/<(target_arch)/libwidevinecdm.so', 16 'binaries/chromeos/<(target_arch)/libwidevinecdm.so',
17 'binaries/chromeos/<(target_arch)/manifest.json', 17 #'binaries/chromeos/<(target_arch)/manifest.json',
18 ], 18 ],
19 }], 19 }],
20 [ 'OS == "linux" and chromeos == 0', { 20 [ 'OS == "linux" and chromeos == 0', {
21 'widevine_cdm_version_h_file%': 21 'widevine_cdm_version_h_file%':
22 'symbols/linux/<(target_arch)/widevine_cdm_version.h', 22 'symbols/linux/<(target_arch)/widevine_cdm_version.h',
23 'widevine_cdm_binary_files%': [ 23 'widevine_cdm_binary_files%': [
24 'binaries/linux/<(target_arch)/libwidevinecdm.so', 24 'binaries/linux/<(target_arch)/libwidevinecdm.so',
25 'binaries/linux/<(target_arch)/manifest.json', 25 #'binaries/linux/<(target_arch)/manifest.json',
26 ], 26 ],
27 }], 27 }],
28 [ 'OS == "mac"', { 28 [ 'OS == "mac"', {
29 'widevine_cdm_version_h_file%': 29 'widevine_cdm_version_h_file%':
30 'symbols/mac/<(target_arch)/widevine_cdm_version.h', 30 'symbols/mac/<(target_arch)/widevine_cdm_version.h',
31 'widevine_cdm_binary_files%': [ 31 'widevine_cdm_binary_files%': [
32 'binaries/mac/<(target_arch)/libwidevinecdm.dylib', 32 'binaries/mac/<(target_arch)/libwidevinecdm.dylib',
33 'binaries/mac/<(target_arch)/manifest.json', 33 #'binaries/mac/<(target_arch)/manifest.json',
34 ], 34 ],
35 }], 35 }],
36 [ 'OS == "win"', { 36 [ 'OS == "win"', {
37 'widevine_cdm_version_h_file%': 37 'widevine_cdm_version_h_file%':
38 'symbols/win/<(target_arch)/widevine_cdm_version.h', 38 'symbols/win/<(target_arch)/widevine_cdm_version.h',
39 'widevine_cdm_binary_files%': [ 39 'widevine_cdm_binary_files%': [
40 'binaries/win/<(target_arch)/widevinecdm.dll', 40 'binaries/win/<(target_arch)/widevinecdm.dll',
41 'binaries/win/<(target_arch)/manifest.json', 41 #'binaries/win/<(target_arch)/manifest.json',
Ami GONE FROM CHROMIUM 2012/10/23 05:52:39 lolwat? You want to check this in this way? You
ddorwin 2012/10/23 15:48:33 Done.
42 ], 42 ],
43 }], 43 }],
44 ], 44 ],
45 }], 45 }],
46 ], 46 ],
47 }, 47 },
48 # Always provide a target, so we can put the logic about whether there's 48 # Always provide a target, so we can put the logic about whether there's
49 # anything to be done in this file (instead of a higher-level .gyp file). 49 # anything to be done in this file (instead of a higher-level .gyp file).
50 'targets': [ 50 'targets': [
51 { 51 {
52 'target_name': 'widevinecdmplugin', 52 'target_name': 'widevinecdmplugin',
53 'type': 'none', 53 'type': 'none',
54 'conditions': [ 54 'conditions': [
55 [ 'branding == "Chrome"', { 55 [ 'branding == "Chrome"', {
56 'dependencies': [ 56 'dependencies': [
57 '<(DEPTH)/ppapi/ppapi.gyp:ppapi_cpp', 57 '<(DEPTH)/ppapi/ppapi.gyp:ppapi_cpp',
58 'widevine_cdm_version_h', 58 'widevine_cdm_version_h',
59 'widevine_cdm_binaries', 59 'widevine_cdm_binaries',
60 ], 60 ],
61 'sources': [ 61 'sources': [
62 '<(DEPTH)/webkit/media/crypto/ppapi/cdm_wrapper.cc', 62 '<(DEPTH)/webkit/media/crypto/ppapi/cdm_wrapper.cc',
63 '<(DEPTH)/webkit/media/crypto/ppapi/content_decryption_module.h', 63 '<(DEPTH)/webkit/media/crypto/ppapi/content_decryption_module.h',
64 '<(DEPTH)/webkit/media/crypto/ppapi/linked_ptr.h', 64 '<(DEPTH)/webkit/media/crypto/ppapi/linked_ptr.h',
65 ], 65 ],
66 'conditions': [ 66 'conditions': [
67 [ 'os_posix == 1 and OS != "mac"', { 67 [ 'os_posix == 1 and OS != "mac"', {
68 'cflags': ['-fvisibility=hidden'], 68 'cflags': ['-fvisibility=hidden'],
69 'type': 'loadable_module', 69 'type': 'loadable_module',
70 # -gstabs, used in the official builds, causes an ICE. Simply 70 # -gstabs, used in the official builds, causes an ICE. Simply
71 # remove it. 71 # remove it.
Ami GONE FROM CHROMIUM 2012/10/23 05:52:39 Can you point to an upstream compiler (MSVC/gcc/cl
ddorwin 2012/10/23 15:48:33 This was copied from ppapi. It dates back to https
72 'cflags!': ['-gstabs'], 72 'cflags!': ['-gstabs'],
73 # Allow the plugin wrapper to find the CDM in the same directory. 73 # Allow the plugin wrapper to find the CDM in the same directory.
74 'ldflags': ['-Wl,-rpath=\$$ORIGIN'] 74 'ldflags': ['-Wl,-rpath=\$$ORIGIN'],
75 'libraries': [
76 # Copied by widevine_cdm_binaries.
77 '<(PRODUCT_DIR)/libwidevinecdm.so',
78 ],
75 }], 79 }],
76 [ 'OS == "win" and 0', { 80 [ 'OS == "win" and 0', {
77 'type': 'shared_library', 81 'type': 'shared_library',
78 }], 82 }],
79 [ 'OS == "mac" and 0', { 83 [ 'OS == "mac" and 0', {
80 'type': 'loadable_module', 84 'type': 'loadable_module',
81 'mac_bundle': 1, 85 'mac_bundle': 1,
82 'product_extension': 'plugin', 86 'product_extension': 'plugin',
83 'xcode_settings': { 87 'xcode_settings': {
84 'OTHER_LDFLAGS': [ 88 'OTHER_LDFLAGS': [
(...skipping 12 matching lines...) Expand all
97 'type': 'none', 101 'type': 'none',
98 'copies': [{ 102 'copies': [{
99 'destination': '<(SHARED_INTERMEDIATE_DIR)', 103 'destination': '<(SHARED_INTERMEDIATE_DIR)',
100 'files': [ '<(widevine_cdm_version_h_file)' ], 104 'files': [ '<(widevine_cdm_version_h_file)' ],
101 }], 105 }],
102 }, 106 },
103 { 107 {
104 'target_name': 'widevine_cdm_binaries', 108 'target_name': 'widevine_cdm_binaries',
105 'type': 'none', 109 'type': 'none',
106 'copies': [{ 110 'copies': [{
107 # TODO(ddorwin): Do we need a sub-directory? 111 # TODO(ddorwin): Do we need a sub-directory? We either need a
112 # sub-directory or to rename manifest.json before we can copy it.
108 'destination': '<(PRODUCT_DIR)', 113 'destination': '<(PRODUCT_DIR)',
109 'files': [ '<@(widevine_cdm_binary_files)' ], 114 'files': [ '<@(widevine_cdm_binary_files)' ],
110 }], 115 }],
111 }, 116 },
112 ], 117 ],
113 } 118 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698