build: make sure this suppresses semi errors

nodesource · Oct 4, 2024 · e9cad6c · e9cad6c
1 parent 2789dd1
commit e9cad6c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/node.gyp b/node.gyp
@@ -514,15 +514,14 @@
     # Putting these explicitly here so not to depend on `common.gypi`.
     # `common.gypi` need to be more general because it is used to build userland native addons.
     # Refs: https://github.com/nodejs/node-gyp/issues/1118
-    'cflags': [ '-Wall', '-Wextra', '-Wno-unused-parameter', ],
+    'cflags': [ '-Wall', '-Wextra', '-Wno-unused-parameter', '-Wno-c++98-compat-extra-semi', ],
     'xcode_settings': {
       'WARNING_CFLAGS': [
         '-Wall',
         '-Wendif-labels',
         '-W',
         '-Wno-unused-parameter',
         '-Werror=undefined-inline',
-        '-Werror=extra-semi',
         '-Wno-c++98-compat-extra-semi',
       ],
     },
@@ -538,7 +537,7 @@
 
     'conditions': [
       ['clang==0 and OS!="win"', {
-        'cflags': [ '-Wno-restrict', ],
+        'cflags': [ '-Wno-restrict', '-Wno-c++98-compat-extra-semi', ],
       }],
       # Pointer authentication for ARM64.
       ['target_arch=="arm64"', {
@@ -561,6 +560,7 @@
         ],
       }],
       ['OS=="linux" and clang==1', {
+        'cflags': [ '-Wno-c++98-compat-extra-semi', ],
         'libraries': ['-latomic'],
       }],
     ],
@@ -631,7 +631,7 @@
 
       'conditions': [
         [ 'error_on_warn=="true"', {
-          'cflags': ['-Werror'],
+          'cflags': ['-Werror', '-Wno-c++98-compat-extra-semi'],
           'xcode_settings': {
             'WARNING_CFLAGS': [ '-Werror' ],
           },

diff --git a/node.gypi b/node.gypi
@@ -27,7 +27,7 @@
 
   'conditions': [
     [ 'clang==1', {
-      'cflags': [ '-Werror=undefined-inline', '-Werror=extra-semi']
+      'cflags': [ '-Werror=undefined-inline', '-Wno-c++98-compat-extra-semi']
     }],
     [ '"<(_type)"=="executable"', {
       'msvs_settings': {
@@ -338,6 +338,11 @@
 
     [ 'node_shared_otlp_http_exporter=="false"', {
       'dependencies': [ 'deps/opentelemetry-cpp/otlp-http-exporter.gyp:otlp-http-exporter' ],
+      'direct_dependent_settings': {
+        'cflags': [
+          '-Wno-c++98-compat-extra-semi',
+        ],
+      },
       'conditions': [
         [ 'force_load=="true"', {
           'xcode_settings': {