Build nacl_helper_nonsfi with GN

components/nacl/BUILD.gn

 # Copyright (c) 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
+import("//build/config/nacl/config.gni")
 import("//build/config/ui.gni")
 import("//testing/test.gni")
 
 if (enable_nacl) {
   source_set("nacl") {
     sources = [
       "loader/nacl_ipc_adapter.cc",
       "loader/nacl_ipc_adapter.h",
       "loader/nacl_listener.cc",
       "loader/nacl_listener.h",
@@ skipping 210 matching lines @@
 
       if (use_glib) {
         configs += [ "//build/config/linux:glib" ]
       }
 
       if (use_seccomp_bpf) {
         defines += [ "USE_SECCOMP_BPF" ]
       }
     }
   }
-} else {
-  group("nacl") {
+
+  if (is_nacl_nonsfi) {
+    executable("nacl_helper_nonsfi_nexe") {
+      output_name = "nacl_helper_nonsfi"
+      set_sources_assignment_filter([])
+      sources = [
+        "common/nacl_messages.cc",
+        "common/nacl_messages.h",
+        "common/nacl_types.cc",
+        "common/nacl_types.h",
+        "common/nacl_types_param_traits.cc",
+        "common/nacl_types_param_traits.h",
+        "loader/nacl_helper_linux.cc",
+        "loader/nacl_helper_linux.h",
+        "loader/nacl_trusted_listener.cc",
+        "loader/nacl_trusted_listener.h",
+        "loader/nonsfi/nonsfi_listener.cc",
+        "loader/nonsfi/nonsfi_listener.h",
+        "loader/nonsfi/nonsfi_main.cc",
+        "loader/nonsfi/nonsfi_main.h",
+      ]
+      deps = [
+        ":nacl_switches",
+        ":nacl_helper_nonsfi_sandbox",
+        "//base",
+        "//components/tracing",
+        "//content",
+        "//ipc",
+        "//native_client/src/nonsfi/irt:nacl_sys_private",
+        "//native_client/src/nonsfi/loader:elf_loader",
+        "//native_client/src/untrusted/nacl:nacl",
+        "//ppapi/proxy",
+        "//sandbox/linux:sandbox",
+      ]
+    }
+
+    source_set("nacl_helper_nonsfi_sandbox") {
+      set_sources_assignment_filter([])
+      sources = [
+        "loader/nonsfi/nonsfi_sandbox.cc",
+        "loader/nonsfi/nonsfi_sandbox.h",
+        "loader/sandbox_linux/nacl_sandbox_linux.cc",
+        "loader/sandbox_linux/nacl_sandbox_linux.h",
+      ]
+      deps = [
+        "//base",
+        "//content",
+        "//sandbox/linux:sandbox",
+      ]
+    }
+
+    copy("nacl_helper_nonsfi_copy") {
+      sources = [
+        "${root_out_dir}/nacl_helper_nonsfi",
+      ]
+      outputs = [
+        "${root_build_dir}/{{source_file_part}}",
+      ]
+      deps = [
+        ":nacl_helper_nonsfi_nexe",
+      ]
+    }
   }
-  group("nacl_helper") {
+
+  group("nacl_helper_nonsfi") {

[brettw, 2015/10/26 00:49:47]

This *should* work so I filed
https://code.google.com/p/chromium/issues/detail?id=547610
for what I think is the problem.

For now, maybe rename this group? Actually, GN style would be to name this
"helper_nonsfi" anyway since it avoids duplicating the name of the directory. I
believe this will fix the collision.

[Petr Hosek, 2015/10/26 01:55:11]

Done.

+    deps = [
+      ":nacl_helper_nonsfi_copy(//build/toolchain/nacl:newlib_pnacl_nonsfi)",
+    ]
   }
 }
 
 source_set("nacl_switches") {
   sources = [
     "common/nacl_switches.cc",
     "common/nacl_switches.h",
   ]
 }
 
@@ skipping 31 matching lines @@
 
   if (is_linux) {
     sources += [
       "common/nacl_paths.cc",
       "common/nacl_paths.h",
     ]
 
     defines = [ "__STDC_LIMIT_MACROS=1" ]
   }
 }