build: fix cctest target for --without-report

Conditionally build `test/cctest/test_report_util.cc` only when configured to include the diagnostic report feature. PR-URL: #28238 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Jun 18, 2019 · ddef3d0 · ddef3d0
1 parent b6f113b
commit ddef3d0
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/node.gyp b/node.gyp
@@ -1096,7 +1096,6 @@
         'test/cctest/test_linked_binding.cc',
         'test/cctest/test_per_process.cc',
         'test/cctest/test_platform.cc',
-        'test/cctest/test_report_util.cc',
         'test/cctest/test_traced_value.cc',
         'test/cctest/test_util.cc',
         'test/cctest/test_url.cc',
@@ -1134,6 +1133,9 @@
           },
         }],
         [ 'node_report=="true"', {
+          'sources': [
+            'test/cctest/test_report_util.cc',
+          ],
           'defines': [
             'NODE_REPORT',
             'NODE_ARCH="<(target_arch)"',