PNaCl. Enables Unsandboxed nonsfi ARM32.

toolchain_build/toolchain_build_pnacl.py

Index: toolchain_build/toolchain_build_pnacl.py
diff --git a/toolchain_build/toolchain_build_pnacl.py b/toolchain_build/toolchain_build_pnacl.py
index bebb5760a562899435c9be47b2bd41a9f0f55e0b..4b4e2220d901653ee6b2cb94c323ef5e12889baa 100755
--- a/toolchain_build/toolchain_build_pnacl.py
+++ b/toolchain_build/toolchain_build_pnacl.py
@@ -1147,6 +1147,9 @@ def GetUploadPackageTargets():
   for os_name in ('linux', 'mac'):
     legal_triple = pynacl.gsd_storage.LegalizeName('x86-32-' + os_name)
     host_packages[os_name].append('unsandboxed_runtime_%s' % legal_triple)
+  for os_name in ['linux']:

[Jim Stichnoth, 2015/10/15 23:31:41]

Use ('linux') like syntax above?

[John, 2015/10/31 17:22:23]

Because python, ('linux') means something else:

    os_name in ('l', 'i', 'n', 'u', 'x')

[Jim Stichnoth, 2015/10/31 17:41:31]

How about making them both of the form:

  for os_name in ['linux', 'mac']:

That seems to work for singletons as well as multiples.

[John, 2015/10/31 19:06:42]

Done.

+    legal_triple = pynacl.gsd_storage.LegalizeName('arm-' + os_name)
+    host_packages[os_name].append('unsandboxed_runtime_%s' % legal_triple)
 
   for os_name, os_packages in host_packages.iteritems():
     package_target = '%s_x86' % pynacl.platform.GetOS(os_name)
@@ -1279,6 +1282,8 @@ def main():
     unsandboxed_runtime_canonical = is_canonical or pynacl.platform.IsMac()
     packages.update(pnacl_targetlibs.UnsandboxedRuntime(
         'x86-32-%s' % pynacl.platform.GetOS(), unsandboxed_runtime_canonical))
+    packages.update(pnacl_targetlibs.UnsandboxedRuntime(
+        'arm-%s' % pynacl.platform.GetOS(), unsandboxed_runtime_canonical))
 
   if args.build_sbtc and not args.pnacl_in_pnacl:
     packages.update(pnacl_sandboxed_translator.SandboxedTranslators(