Reviewers: Yang,

Message:
Yang: take a quick look please.

Ben: FYI. Thanks for the patch.

Description:
Add solaris support to gyp build.

Patch by Ben Noordhuis <[email protected]>. I don't have a Solaris box around
to test this myself.


BUG=v8:1684


Please review this at http://codereview.chromium.org/7889038/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M build/common.gypi
  M tools/gyp/v8.gyp


Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 4e896e019a8579210cc544fa63e691575746c71a..34508913fedd2d8b7f3482b91c25821c311a6507 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -184,6 +184,9 @@
           }],
         ],
       }],
+      ['OS=="solaris"', {
+        'defines': [ '__C99FEATURES__=1' ],  # isinf() etc.
+      }],
     ],
     'configurations': {
       'Debug': {
Index: tools/gyp/v8.gyp
diff --git a/tools/gyp/v8.gyp b/tools/gyp/v8.gyp
index 50144172a0471df4e235c8e94b4ad64e931f40d2..18b468449a74ca0712fd1ad7166b7b9d1afd74df 100644
--- a/tools/gyp/v8.gyp
+++ b/tools/gyp/v8.gyp
@@ -641,6 +641,13 @@
                 ],
               }
             ],
+            ['OS=="solaris"', {
+                'sources': [
+                  '../../src/platform-solaris.cc',
+                  '../../src/platform-posix.cc',
+                ],
+              }
+            ],
             ['OS=="mac"', {
               'sources': [
                 '../../src/platform-macos.cc',


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to