Improvements to GYP build

This updates the GYP build for the processor component (on windows).
- adds/removes references to files which were added or removed from the
  repository
- includes build/common.gypi in the gyp files: needed to correctly
  detect the OS (I think, the generated MSVC solutions were broken
  without it)
- conditionally compiles code platform-specific code for the given
  platform

After this minidump processor nearly compiles with VS2013: the generated
project is correct, but some files still have compilation errors.

Disclaimer: I have not tested the GYP changes on non-windows platform,
as there does not seem to be anyone using it there.

BUG=
R=mark@chromium.org

Review URL: https://codereview.chromium.org/1643633004 .
This commit is contained in:
Pavel Labath 2016-01-29 11:43:21 +00:00
parent 855ea963fb
commit 4003d20c70
3 changed files with 10 additions and 3 deletions

View File

@ -35,6 +35,11 @@
['OS=="linux"', { ['OS=="linux"', {
'defines': ['HAVE_A_OUT_H'], 'defines': ['HAVE_A_OUT_H'],
}], }],
['OS!="android"', {'sources/': [['exclude', '(^|/)android/']]}],
['OS!="linux"', {'sources/': [['exclude', '(^|/)linux/']]}],
['OS!="mac"', {'sources/': [['exclude', '(^|/)mac/']]}],
['OS!="solaris"', {'sources/': [['exclude', '(^|/)solaris/']]}],
['OS!="win"', {'sources/': [['exclude', '(^|/)windows/']]}],
], ],
}, },
'targets': [ 'targets': [

View File

@ -28,6 +28,7 @@
{ {
'includes': [ 'includes': [
'../build/common.gypi',
'processor_tools.gypi', 'processor_tools.gypi',
], ],
'targets': [ 'targets': [
@ -42,8 +43,6 @@
'basic_code_modules.h', 'basic_code_modules.h',
'basic_source_line_resolver.cc', 'basic_source_line_resolver.cc',
'basic_source_line_resolver_types.h', 'basic_source_line_resolver_types.h',
'binarystream.cc',
'binarystream.h',
'call_stack.cc', 'call_stack.cc',
'cfi_frame_info-inl.h', 'cfi_frame_info-inl.h',
'cfi_frame_info.cc', 'cfi_frame_info.cc',
@ -78,6 +77,7 @@
'pathname_stripper.h', 'pathname_stripper.h',
'postfix_evaluator-inl.h', 'postfix_evaluator-inl.h',
'postfix_evaluator.h', 'postfix_evaluator.h',
'proc_maps_linux.cc',
'process_state.cc', 'process_state.cc',
'range_map-inl.h', 'range_map-inl.h',
'range_map.h', 'range_map.h',
@ -143,7 +143,6 @@
'sources': [ 'sources': [
'address_map_unittest.cc', 'address_map_unittest.cc',
'basic_source_line_resolver_unittest.cc', 'basic_source_line_resolver_unittest.cc',
'binarystream_unittest.cc',
'cfi_frame_info_unittest.cc', 'cfi_frame_info_unittest.cc',
'contained_range_map_unittest.cc', 'contained_range_map_unittest.cc',
'disassembler_x86_unittest.cc', 'disassembler_x86_unittest.cc',

View File

@ -27,6 +27,9 @@
# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
{ {
'includes': [
'../../build/common.gypi',
],
'targets': [ 'targets': [
{ {
'target_name': 'libdisasm', 'target_name': 'libdisasm',