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

Unified Diff: base/base.gyp

Issue 103333004: Allow Win64 build of base library even in a Win32 build. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Base Win64: Replace existing base_nacl_win64 target with the new target. Created 7 years 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 | « no previous file | chrome/chrome_exe.gypi » ('j') | third_party/modp_b64/modp_b64.gyp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/base.gyp
diff --git a/base/base.gyp b/base/base.gyp
index 0c8207f84c32e7cec29a1ed4a34483c0ed2d9349..0dd8d1f5acca4d737883cd28aab5a27887cd4b2e 100644
--- a/base/base.gyp
+++ b/base/base.gyp
@@ -1055,8 +1055,11 @@
}],
['OS == "win" and target_arch=="ia32"', {
'targets': [
+ # The base_win64 target here allows us to use base for a Win64
+ # binary that might need to be built when building a Win32 browser,
brettw 2013/12/19 21:06:05 I'd change this to something simpler like "The bas
Pete Williamson 2013/12/20 01:52:09 Done.
+ # such as NaCl and the DeskBand.
{
- 'target_name': 'base_nacl_win64',
+ 'target_name': 'base_win64',
'type': '<(component)',
'variables': {
'base_target': 1,
@@ -1064,6 +1067,7 @@
'dependencies': [
'base_static_win64',
'allocator/allocator.gyp:allocator_extension_thunks_win64',
+ '../third_party/modp_b64/modp_b64.gyp:modp_b64_win64',
'third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations_win64',
],
# TODO(gregoryd): direct_dependent_settings should be shared with the
@@ -1074,12 +1078,9 @@
],
},
'defines': [
+ 'BASE_WIN64',
'<@(nacl_win64_defines)',
],
- 'sources!': [
- # base64.cc depends on modp_b64.
- 'base64.cc',
- ],
'configurations': {
'Common_Base': {
'msvs_target_platform': 'x64',
@@ -1092,6 +1093,40 @@
],
}],
],
+ # TODO(rvargas): Bug 78117. Remove this.
+ 'msvs_disabled_warnings': [
+ 4244,
+ 4996,
+ 4267,
+ ],
+ 'sources': [
+ 'third_party/nspr/prcpucfg.h',
+ 'third_party/nspr/prcpucfg_win.h',
+ 'third_party/nspr/prtypes.h',
+ 'third_party/xdg_user_dirs/xdg_user_dir_lookup.cc',
+ 'third_party/xdg_user_dirs/xdg_user_dir_lookup.h',
+ 'async_socket_io_handler.h',
+ 'async_socket_io_handler_posix.cc',
+ 'async_socket_io_handler_win.cc',
+ 'auto_reset.h',
+ 'event_recorder.h',
+ 'event_recorder_stubs.cc',
+ 'event_recorder_win.cc',
+ 'linux_util.cc',
+ 'linux_util.h',
+ 'md5.cc',
+ 'md5.h',
+ 'message_loop/message_pump_observer.h',
+ 'message_loop/message_pump_libevent.cc',
+ 'message_loop/message_pump_libevent.h',
+ 'metrics/field_trial.cc',
+ 'metrics/field_trial.h',
+ 'posix/file_descriptor_shuffle.cc',
+ 'posix/file_descriptor_shuffle.h',
+ 'sync_socket.h',
+ 'sync_socket_win.cc',
+ 'sync_socket_posix.cc',
+ ],
},
{
'target_name': 'base_i18n_nacl_win64',
@@ -1143,7 +1178,7 @@
},
},
'defines': [
- 'NACL_WIN64',
+ '<@(nacl_win64_defines)',
],
# TODO(rvargas): Bug 78117. Remove this.
'msvs_disabled_warnings': [
« no previous file with comments | « no previous file | chrome/chrome_exe.gypi » ('j') | third_party/modp_b64/modp_b64.gyp » ('J')

Powered by Google App Engine
This is Rietveld 408576698