build: gyp exclude libm linking on macOS

PR-URL: #56901 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Shelley Vohr <shelley.vohr@gmail.com>
nodejs · Feb 7, 2025 · b7ce941 · b7ce941
1 parent 756a242
commit b7ce941
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': [