enable android x86 build

base/base.gypi

Index: base/base.gypi
diff --git a/base/base.gypi b/base/base.gypi
index 74899b2b962f53846d18279be546efed37f67cbe..c47823e003cc815220053d057617e0e6e180d3be 100644
--- a/base/base.gypi
+++ b/base/base.gypi
@@ -677,6 +677,13 @@
           ],
         }],
         [ 'OS == "android" and _toolset == "target"', {
+          'conditions': [
+            ['target_arch == "ia32"', {
+              'sources/': [

[Paweł Hajdan Jr., 2012/04/27 06:56:19]

nit: Why don't you just use 'sources!' instead?

[Wei James, 2012/04/27 08:59:05]

this code is just copied from above line#658. do you mean "sources"? because I
would like to include the files for target build just like linux host build. I
tested with both "sources" and "sources!", but it seems the files are not
included so there is undefined error when linking. I am not a gyp expert and
cannot find the root cause yet. 

I saw most of the sources in this file use "sources/" style. is it ok to keep it
to be consistent with others? thanks 

[Paweł Hajdan Jr., 2012/04/27 09:40:06]

Ah right, it's "include" so just "sources" please.
It's simpler to use "sources" to include files and "sources!" to exclude them.
I'd rather convert the surrounding code (possibly in a separate CL) instead of
keeping the weird style (maybe someone just didn't know gyp well).

+                ['include', '^atomicops_internals_x86_gcc\\.cc$'],
+              ],
+            }],
+          ],
           'dependencies': [
             'symbolize',
             '../third_party/ashmem/ashmem.gyp:ashmem',