Refactor the process of choosing validators.

src/trusted/service_runtime/service_runtime.gyp

Index: src/trusted/service_runtime/service_runtime.gyp
diff --git a/src/trusted/service_runtime/service_runtime.gyp b/src/trusted/service_runtime/service_runtime.gyp
index 4e72be06f951025626bd48dae1f41d3d545e591e..b15650f53967013f7025dfac3f8ffe0c012045d4 100644
--- a/src/trusted/service_runtime/service_runtime.gyp
+++ b/src/trusted/service_runtime/service_runtime.gyp
@@ -205,6 +205,9 @@
       'variables': {
         'target_base': 'sel',
       },
+      'sources': [
+          '<(DEPTH)/native_client/src/trusted/validator/validator_init.c',

[Nick Bray, 2012/05/22 23:46:31]

This should be added to the 'target_base=="sel"' declaration - where all the
other sources are.  Note there are 'sel' and 'sel64' targets where this source
list gets injected into.  because you put it here, the sel64 version didn't get
the source file and you papered over the problem by also adding it to the 64-bit
validator's gyp file.  You should be able to remove that change after cleaning
up this file?

Nit: _init may be the wrong file name - _interface?  I don't have a very good
suggestion, but I feel it's not quite right.

Indentation is wrong.

[pasko-google - do not use, 2012/05/23 14:28:11]

Thank you for this! Seriously, great suggestion! I tried to put this source in
different places and was completely frustrated :)

Things like this:
'target_conditions': [['target_base=="sel"',
... look just sooo cute for a GYP newbie. Freaks me out.

Done.
Same thoughts here, maybe validator_selector?
Done.

+      ],
       'dependencies': [
         'env_cleanser',
         'nacl_error_code',