build: fix cctest compilation

Currently the cctest target compiles sources files even though they are
compiled for the node target. This is my fault as when I worked on the
task of getting the cctest to use the object files from the node target
I missed a few sources that were being included from node.gypi. This
also effects the build time as these sources are compiled twice.

This commit moves the conditions in question into the node target in
node.gyp. With this commit there should be no object files in
out/Release/obj.target/cctest/src/ (the path will vary depending on the
operating system being used).

PR-URL:https://github.com/nodejs/node/pull/16887
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

1 files changed, 0 insertions, 141 deletions

diff --git a/node.gypi b/node.gypi
index 1bb6581a00..718ef0c9e7 100644
--- a/node.gypi
+++ b/node.gypi
@@ -70,128 +70,6 @@
         'deps/v8/src/third_party/vtune/v8vtune.gyp:v8_vtune'
       ],
     }],
-    [ 'v8_enable_inspector==1', {
-      'defines': [
-        'HAVE_INSPECTOR=1',
-      ],
-      'sources': [
-        'src/inspector_agent.cc',
-        'src/inspector_io.cc',
-        'src/inspector_js_api.cc',
-        'src/inspector_socket.cc',
-        'src/inspector_socket_server.cc',
-        'src/inspector_agent.h',
-        'src/inspector_io.h',
-        'src/inspector_socket.h',
-        'src/inspector_socket_server.h',
-      ],
-      'dependencies': [
-        'v8_inspector_compress_protocol_json#host',
-      ],
-      'include_dirs': [
-        '<(SHARED_INTERMEDIATE_DIR)/include', # for inspector
-        '<(SHARED_INTERMEDIATE_DIR)',
-      ],
-    }, {
-      'defines': [ 'HAVE_INSPECTOR=0' ]
-    }],
-    [ 'node_use_openssl=="true"', {
-      'defines': [ 'HAVE_OPENSSL=1' ],
-      'sources': [
-        'src/node_crypto.cc',
-        'src/node_crypto_bio.cc',
-        'src/node_crypto_clienthello.cc',
-        'src/node_crypto.h',
-        'src/node_crypto_bio.h',
-        'src/node_crypto_clienthello.h',
-        'src/tls_wrap.cc',
-        'src/tls_wrap.h'
-      ],
-      'conditions': [
-        ['openssl_fips != ""', {
-          'defines': [ 'NODE_FIPS_MODE' ],
-        }],
-        [ 'node_shared_openssl=="false"', {
-          'dependencies': [
-            './deps/openssl/openssl.gyp:openssl',
-
-            # For tests
-            './deps/openssl/openssl.gyp:openssl-cli',
-          ],
-          # Do not let unused OpenSSL symbols to slip away
-          'conditions': [
-            # -force_load or --whole-archive are not applicable for
-            # the static library
-            [ 'node_target_type!="static_library"', {
-              'xcode_settings': {
-                'OTHER_LDFLAGS': [
-                  '-Wl,-force_load,<(PRODUCT_DIR)/<(OPENSSL_PRODUCT)',
-                ],
-              },
-              'conditions': [
-                ['OS in "linux freebsd" and node_shared=="false"', {
-                  'ldflags': [
-                    '-Wl,--whole-archive,'
-                        '<(OBJ_DIR)/deps/openssl/'
-                        '<(OPENSSL_PRODUCT)',
-                    '-Wl,--no-whole-archive',
-                  ],
-                }],
-                # openssl.def is based on zlib.def, zlib symbols
-                # are always exported.
-                ['use_openssl_def==1', {
-                  'sources': ['<(SHARED_INTERMEDIATE_DIR)/openssl.def'],
-                }],
-                ['OS=="win" and use_openssl_def==0', {
-                  'sources': ['deps/zlib/win32/zlib.def'],
-                }],
-              ],
-            }],
-          ],
-        }]]
-    }, {
-      'defines': [ 'HAVE_OPENSSL=0' ]
-    }],
-    [ 'node_use_dtrace=="true"', {
-      'defines': [ 'HAVE_DTRACE=1' ],
-      'dependencies': [
-        'node_dtrace_header',
-        'specialize_node_d',
-      ],
-      'include_dirs': [ '<(SHARED_INTERMEDIATE_DIR)' ],
-
-      #
-      # DTrace is supported on linux, solaris, mac, and bsd.  There are
-      # three object files associated with DTrace support, but they're
-      # not all used all the time:
-      #
-      #   node_dtrace.o           all configurations
-      #   node_dtrace_ustack.o    not supported on mac and linux
-      #   node_dtrace_provider.o  All except OS X.  "dtrace -G" is not
-      #                           used on OS X.
-      #
-      # Note that node_dtrace_provider.cc and node_dtrace_ustack.cc do not
-      # actually exist.  They're listed here to trick GYP into linking the
-      # corresponding object files into the final "node" executable.  These
-      # object files are generated by "dtrace -G" using custom actions
-      # below, and the GYP-generated Makefiles will properly build them when
-      # needed.
-      #
-      'sources': [ 'src/node_dtrace.cc' ],
-      'conditions': [
-        [ 'OS=="linux"', {
-          'sources': [
-            '<(SHARED_INTERMEDIATE_DIR)/node_dtrace_provider.o'
-          ],
-        }],
-        [ 'OS!="mac" and OS!="linux"', {
-          'sources': [
-            'src/node_dtrace_ustack.cc',
-            'src/node_dtrace_provider.cc',
-          ]
-        }
-      ] ]
-    } ],
     [ 'node_use_lttng=="true"', {
       'defines': [ 'HAVE_LTTNG=1' ],
       'include_dirs': [ '<(SHARED_INTERMEDIATE_DIR)' ],
@@ -255,25 +133,6 @@
       'dependencies': [ 'deps/uv/uv.gyp:libuv' ],
     }],
 
-    [ 'OS=="win"', {
-      'sources': [
-        'src/backtrace_win32.cc',
-        'src/res/node.rc',
-      ],
-      'defines!': [
-        'NODE_PLATFORM="win"',
-      ],
-      'defines': [
-        'FD_SETSIZE=1024',
-        # we need to use node's preferred "win32" rather than gyp's preferred "win"
-        'NODE_PLATFORM="win32"',
-        '_UNICODE=1',
-      ],
-      'libraries': [ '-lpsapi.lib' ]
-    }, { # POSIX
-      'defines': [ '__POSIX__' ],
-      'sources': [ 'src/backtrace_posix.cc' ],
-    }],
     [ 'OS=="mac"', {
       # linking Corefoundation is needed since certain OSX debugging tools
       # like Instruments require it for some features