Android build target for libadblockplus

common.gypi

Index: common.gypi
===================================================================
--- a/common.gypi
+++ b/common.gypi
@@ -19,6 +19,30 @@
         'ldflags': [ '-pthread', ],
       },
     }],
+    ['OS=="android"', {
+      'target_defaults': {
+        'cflags': ['-std=gnu++0x', '-fexceptions', '-frtti'],

[Andrey Novikov, 2013/04/30 20:16:05]

May be we need other flags, I do not know, I only added those that are required
for successful builds.

+        'cflags!': [
+          '-pthread',  # Not supported by Android toolchain.
+        ],
+        'ldflags!': [
+          '-pthread',  # Not supported by Android toolchain.
+        ],
+        'include_dirs': [
+          'include',
+          'third_party/v8/include',
+          'third_party/googletest/',
+          'third_party/googletest/include',
+          'third_party/googletest/include/internal'

[Wladimir Palant, 2013/05/07 08:18:54]

Is it really necessary to list include directories here? We already list them
for the specific targets, why isn't it sufficient?

+        ],
+        'link_settings': {
+          'libraries': [
+            'android_arm.release/obj.target/tools/gyp/libv8_base.a',
+            'android_arm.release/obj.target/tools/gyp/libv8_snapshot.a',

[Wladimir Palant, 2013/05/07 08:18:54]

These should only be relevant for the libadblockplus target - they should be
added there, as part of OS=="android" condition. No point adding these libraries
for all targets.

+          ],
+        },
+      },
+    }],
   ],
 
   'target_defaults': {