Revision: 12806
Author:   [email protected]
Date:     Wed Oct 24 05:11:15 2012
Log:      Enable shared library on android

This change fixed link errors in building shared library
for android. crtbegin_so.o is added to resolve dso_handle
and exclude-libs option is removed for shared library.

Review URL: https://chromiumcodereview.appspot.com/11262003
http://code.google.com/p/v8/source/detail?r=12806

Modified:
 /branches/bleeding_edge/AUTHORS
 /branches/bleeding_edge/build/android.gypi

=======================================
--- /branches/bleeding_edge/AUTHORS     Thu Jul 26 03:03:30 2012
+++ /branches/bleeding_edge/AUTHORS     Wed Oct 24 05:11:15 2012
@@ -53,6 +53,7 @@
 Subrato K De <[email protected]>
 Tobias Burnus <[email protected]>
 Vlad Burlik <[email protected]>
+Xi Qian <[email protected]>
 Yuqiang Xian <[email protected]>
 Zaheer Ahmad <[email protected]>
 Zhongping Wang <[email protected]>
=======================================
--- /branches/bleeding_edge/build/android.gypi  Mon Sep  3 04:27:09 2012
+++ /branches/bleeding_edge/build/android.gypi  Wed Oct 24 05:11:15 2012
@@ -122,8 +122,6 @@
         'ldflags': [
           '-nostdlib',
           '-Wl,--no-undefined',
-          # Don't export symbols from statically linked libraries.
-          '-Wl,--exclude-libs=ALL',
         ],
         'libraries!': [
             '-lrt',  # librt is built into Bionic.
@@ -219,6 +217,13 @@
           ['_type=="shared_library"', {
             'ldflags': [
               '-Wl,-shared,-Bsymbolic',
+              '<(android_lib)/crtbegin_so.o',
+            ],
+          }],
+          ['_type=="static_library"', {
+            'ldflags': [
+              # Don't export symbols from statically linked libraries.
+              '-Wl,--exclude-libs=ALL',
             ],
           }],
         ],

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

Reply via email to