Clean up the gyp build of standalone dart executable on Android.

runtime/bin/io_impl_sources.gypi

Index: runtime/bin/io_impl_sources.gypi
diff --git a/runtime/bin/io_impl_sources.gypi b/runtime/bin/io_impl_sources.gypi
index 0eeaf5dcc50f7974e2ea1be5ba0bc23997b12f34..3a60185285943c0d34d919ba7c74c58ef50b031c 100644
--- a/runtime/bin/io_impl_sources.gypi
+++ b/runtime/bin/io_impl_sources.gypi
@@ -23,6 +23,7 @@
     'eventhandler_win.h',
     'filter.cc',
     'filter.h',
+    'filter_unsupported.cc',
     'net/nss_memio.cc',
     'net/nss_memio.h',
     'platform.cc',
@@ -45,5 +46,24 @@
     'socket_win.cc',
     'secure_socket.cc',
     'secure_socket.h',
+    'secure_socket_unsupported.cc',
+  ],
+  'conditions': [
+    [ 'dart_io_support==1', {
+      'sources!' : [
+        'filter_unsupported.cc',
+        'secure_socket_unsupported.cc',
+      ],
+    },
+    {
+      'sources!' : [

[Ivan Posva, 2013/06/03 07:32:09]

To me this reads as if the below sources are always excluded, which I know they
are not. This has to do with the indentation.

How about:

'conditions' : [
..[ 'dart_io_support==1', {
....'sources!' : [
......'filter_unsupported.cc',
......'secure_socket_unsupported.cc',
....],
..},{
....'sources!': [
......'filter.cc',
....],
..}],
}

Maybe even add a comment marking the start of the "else" from the condition.

[Bill Hesse, 2013/06/03 09:20:41]

Done.

+        'filter.cc',
+        'filter.h',
+        'net/nss_memio.cc',
+        'net/nss_memio.h',
+        'secure_socket.cc',
+        'secure_socket.h',
+      ],
+    }],
   ],
 }