factor out common build settings

nodejs · Dec 5, 2019 · f94adf5 · f94adf5
1 parent ce91e14
commit f94adf5
Show file tree

Hide file tree

Showing 4 changed files with 60 additions and 48 deletions.
diff --git a/common.gypi b/common.gypi
@@ -0,0 +1,24 @@
+{
+  'variables': {
+    'NAPI_VERSION%': "<!(node -p \"process.versions.napi\")",
+    'NODE_MAJOR_VERSION%': "<!(node -p \"process.versions.node.match(/\\d+/)[0]\")",
+    'disable_deprecated': "<!(node -p \"process.env['npm_config_disable_deprecated']\")"
+  },
+  'conditions': [
+    ['NAPI_VERSION!=""', { 'defines': ['NAPI_VERSION=<@(NAPI_VERSION)'] } ],
+    ['disable_deprecated=="true"', {
+      'defines': ['NODE_ADDON_API_DISABLE_DEPRECATED']
+    }],
+    ['OS=="mac"', {
+      'cflags+': ['-fvisibility=hidden'],
+      'xcode_settings': {
+        'OTHER_CFLAGS': ['-fvisibility=hidden']
+      }
+    }]
+  ],
+  'defines': ['NODE_MAJOR_VERSION=<@(NODE_MAJOR_VERSION)'],
+  'include_dirs': ["<!@(node -p \"require('../').include\")"],
+  'dependencies': ["<!(node -p \"require('../').gyp\")"],
+  'cflags': [ '-Werror', '-Wall', '-Wextra', '-Wpedantic', '-Wunused-parameter' ],
+  'cflags_cc': [ '-Werror', '-Wall', '-Wextra', '-Wpedantic', '-Wunused-parameter' ]
+}
diff --git a/except.gypi b/except.gypi
@@ -0,0 +1,16 @@
+{
+  'defines': [ 'NAPI_CPP_EXCEPTIONS' ],
+  'cflags!': [ '-fno-exceptions' ],
+  'cflags_cc!': [ '-fno-exceptions' ],
+  'msvs_settings': {
+    'VCCLCompilerTool': {
+      'ExceptionHandling': 1,
+      'EnablePREfast': 'true',
+    },
+  },
+  'xcode_settings': {
+    'CLANG_CXX_LIBRARY': 'libc++',
+    'MACOSX_DEPLOYMENT_TARGET': '10.7',
+    'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',
+  },
+}
diff --git a/noexcept.gypi b/noexcept.gypi
@@ -0,0 +1,16 @@
+{
+  'defines': [ 'NAPI_DISABLE_CPP_EXCEPTIONS' ],
+  'cflags': [ '-fno-exceptions' ],
+  'cflags_cc': [ '-fno-exceptions' ],
+  'msvs_settings': {
+    'VCCLCompilerTool': {
+      'ExceptionHandling': 0,
+      'EnablePREfast': 'true',
+    },
+  },
+  'xcode_settings': {
+    'CLANG_CXX_LIBRARY': 'libc++',
+    'MACOSX_DEPLOYMENT_TARGET': '10.7',
+    'GCC_ENABLE_CPP_EXCEPTIONS': 'NO',
+  },
+}
diff --git a/test/binding.gyp b/test/binding.gyp
@@ -1,10 +1,6 @@
 {
-  'variables': {
-    'NAPI_VERSION%': "<!(node -p \"process.versions.napi\")",
-    'NODE_MAJOR_VERSION%': "<!(node -p \"process.versions.node.match(/\\d+/)[0]\")",
-    'disable_deprecated': "<!(node -p \"process.env['npm_config_disable_deprecated']\")"
-  },
   'target_defaults': {
+    'includes': ['../common.gypi'],
     'sources': [
         'arraybuffer.cc',
         'asynccontext.cc',
@@ -49,59 +45,19 @@
         'thunking_manual.cc',
       ],
       'conditions': [
-        ['NAPI_VERSION!=""', { 'defines': ['NAPI_VERSION=<@(NAPI_VERSION)'] } ],
-        ['disable_deprecated=="true"', {
-          'defines': ['NODE_ADDON_API_DISABLE_DEPRECATED']
-        }, {
+        ['disable_deprecated!="true"', {
           'sources': ['object/object_deprecated.cc']
-        }],
-        ['OS=="mac"', {
-          'cflags+': ['-fvisibility=hidden'],
-          'xcode_settings': {
-            'OTHER_CFLAGS': ['-fvisibility=hidden']
-          }
         }]
       ],
-      'defines': ['NODE_MAJOR_VERSION=<@(NODE_MAJOR_VERSION)'],
-      'include_dirs': ["<!@(node -p \"require('../').include\")"],
-      'dependencies': ["<!(node -p \"require('../').gyp\")"],
-      'cflags': [ '-Werror', '-Wall', '-Wextra', '-Wpedantic', '-Wunused-parameter' ],
-      'cflags_cc': [ '-Werror', '-Wall', '-Wextra', '-Wpedantic', '-Wunused-parameter' ]
   },
   'targets': [
     {
       'target_name': 'binding',
-      'defines': [ 'NAPI_CPP_EXCEPTIONS' ],
-      'cflags!': [ '-fno-exceptions' ],
-      'cflags_cc!': [ '-fno-exceptions' ],
-      'msvs_settings': {
-        'VCCLCompilerTool': {
-          'ExceptionHandling': 1,
-          'EnablePREfast': 'true',
-        },
-      },
-      'xcode_settings': {
-        'CLANG_CXX_LIBRARY': 'libc++',
-        'MACOSX_DEPLOYMENT_TARGET': '10.7',
-        'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',
-      },
+      'includes': ['../except.gypi']
     },
     {
       'target_name': 'binding_noexcept',
-      'defines': [ 'NAPI_DISABLE_CPP_EXCEPTIONS' ],
-      'cflags': [ '-fno-exceptions' ],
-      'cflags_cc': [ '-fno-exceptions' ],
-      'msvs_settings': {
-        'VCCLCompilerTool': {
-          'ExceptionHandling': 0,
-          'EnablePREfast': 'true',
-        },
-      },
-      'xcode_settings': {
-        'CLANG_CXX_LIBRARY': 'libc++',
-        'MACOSX_DEPLOYMENT_TARGET': '10.7',
-        'GCC_ENABLE_CPP_EXCEPTIONS': 'NO',
-      },
+      'includes': ['../noexcept.gypi']
     },
   ],
 }