Chromium Code Reviews| Index: jingle/jingle.gyp |
| =================================================================== |
| --- jingle/jingle.gyp (revision 175573) |
| +++ jingle/jingle.gyp (working copy) |
| @@ -5,6 +5,7 @@ |
| { |
| 'variables': { |
| 'chromium_code': 1, |
| + 'target_arch%': "ia32", |
| }, # variables |
| 'conditions': [ |
| ['enable_webrtc==1 or (OS != "ios" and OS != "android")', { |
| @@ -45,6 +46,8 @@ |
| '../third_party/libjingle/libjingle.gyp:libjingle', |
| '../third_party/libjingle/libjingle.gyp:libjingle_p2p', |
| ], |
| + # TODO(jschuh): crbug.com/167187 fix size_t to int truncations. |
| + 'msvs_disabled_warnings': [4267, ], |
| }, |
| # A library for sending and receiving peer-issued notifications. |
| # |
| @@ -104,8 +107,14 @@ |
| ], |
| 'defines' : [ |
| '_CRT_SECURE_NO_WARNINGS', |
| - '_USE_32BIT_TIME_T', |
| ], |
| + 'target_conditions': [ |
| + ['target_arch == "ia32"', { |
| + 'defines': [ |
| + '_USE_32BIT_TIME_T', |
|
Sergey Ulanov
2013/01/10 02:37:39
time_t is not used in the code compiled by this ta
jschuh
2013/01/10 03:12:15
Given it's current location I assumed it was used
|
| + ], |
| + }], |
| + ], |
| 'dependencies': [ |
| '../base/base.gyp:base', |
| '../build/temp_gyp/googleurl.gyp:googleurl', |
| @@ -123,6 +132,8 @@ |
| ], |
| }], |
| ], |
| + # TODO(jschuh): crbug.com/167187 fix size_t to int truncations. |
| + 'msvs_disabled_warnings': [4267, ], |
| }, |
| { |
| 'target_name': 'notifier_test_util', |
| @@ -154,6 +165,8 @@ |
| 'jingle_glue', |
| '../base/base.gyp:base', |
| ], |
| + # TODO(jschuh): crbug.com/167187 fix size_t to int truncations. |
| + 'msvs_disabled_warnings': [4267, ], |
| }, |
| { |
| 'target_name': 'jingle_unittests', |
| @@ -212,6 +225,8 @@ |
| '../testing/gtest.gyp:gtest', |
| '../third_party/libjingle/libjingle.gyp:libjingle', |
| ], |
| + # TODO(jschuh): crbug.com/167187 fix size_t to int truncations. |
| + 'msvs_disabled_warnings': [4267, ], |
| }, |
| ], |
| }, { # enable_webrtc != 1 and (OS == "ios" or OS == "android") |