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 3f736ce086
commit 442b45266d
3 changed files with 10 additions and 3 deletions

View file

@ -35,6 +35,11 @@
['OS=="linux"', {
'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': [

View file

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

View file

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