build: gyp exclude libm linking on macOS

PR-URL: nodejs#56901 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Shelley Vohr <[email protected]>
acidiney · Feb 23, 2025 · 4f2c0cd · 4f2c0cd
1 parent 51fcab5
commit 4f2c0cd
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/deps/brotli/brotli.gyp b/deps/brotli/brotli.gyp
@@ -59,14 +59,15 @@
           'defines': [
             'OS_MACOSX'
           ]
+        }, {
+          'libraries': [
+            '-lm',
+          ],
         }],
       ],
       'direct_dependent_settings': {
         'include_dirs': [ 'c/include' ]
       },
-      'libraries': [
-        '-lm',
-      ],
       'sources': [
         '<@(brotli_sources)',
       ]

diff --git a/deps/uv/uv.gyp b/deps/uv/uv.gyp
@@ -220,7 +220,6 @@
             '<@(uv_sources_posix)',
           ],
           'link_settings': {
-            'libraries': [ '-lm' ],
             'conditions': [
               ['OS=="solaris"', {
                 'ldflags': [ '-pthreads' ],
@@ -231,6 +230,11 @@
               ['OS != "solaris" and OS != "android" and OS != "zos"', {
                 'ldflags': [ '-pthread' ],
               }],
+              ['OS!="mac"', {
+                'libraries': [
+                  '-lm'
+                ],
+              }],
             ],
           },
           'conditions': [