Build PPAPI NaCl Glibc tests with GN

ppapi/BUILD.gn

 # Copyright 2014 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.
 
 copy("copy_test_files") {
   visibility = [ ":*" ]
   sources = [
     # Keep "test_case.html.mock-http-headers" with "test_case.html".
     "tests/ppapi_nacl_tests_newlib.nmf",
     "tests/test_case.html",
@@ skipping 127 matching lines @@
     "tools/pepper_hash_for_uma.cc",
   ]
 
   deps = [
     "//base",
     "//build/config/sanitizers:deps",
   ]
 }
 
 source_set("ppapi_cpp_lib") {
+  if (is_posix) {
+    configs -= [ "//build/config/gcc:symbol_visibility_hidden" ]
+  }
   sources = ppapi_sources.cpp_source_files
   sources += [
     "cpp/module_embedder.h",
     "cpp/ppp_entrypoints.cc",
   ]
+  if (is_nacl) {
+    cflags = [ "-fPIC" ]
+  }
+  deps = [
+    "//build/config/nacl:nacl_base",

[brettw, 2015/09/10 05:02:50]

I think you mean to put this inside the is_nacl block.

[Petr Hosek, 2015/09/11 22:10:49]

Done.

+  ]
 }
 
 source_set("ppapi_gles2_lib") {
   include_dirs = [ "lib/gl/include" ]
   sources = [
     "lib/gl/gles2/gl2ext_ppapi.c",
     "lib/gl/gles2/gl2ext_ppapi.h",
     "lib/gl/gles2/gles2.c",
   ]
   deps = [
     "//ppapi/cpp",
   ]
 }
 
 if (enable_nacl) {
   if (is_nacl) {
+    shared_library("ppapi_cpp_lib_shared") {
+      deps = [
+        ":ppapi_cpp_lib",
+      ]
+    }
+
     executable("ppapi_nacl_tests_newlib") {
-      include_dirs = [
-        "lib/gl/include",
-        "..",
-      ]
+      include_dirs = [ "lib/gl/include" ]
 
       sources = ppapi_sources.test_common_source_files +
                 ppapi_sources.test_nacl_source_files
 
       defines = [ "GL_GLEXT_PROTOTYPES" ]
 
       deps = [
         ":ppapi_cpp_lib",
         "//ppapi/native_client:ppapi_lib",
       ]
     }
 
+    executable("ppapi_nacl_tests_glibc") {
+      configs -= [ "//build/config/gcc:symbol_visibility_hidden" ]
+
+      include_dirs = [ "lib/gl/include" ]
+
+      sources = ppapi_sources.test_common_source_files +
+                ppapi_sources.test_nacl_source_files
+
+      defines = [ "GL_GLEXT_PROTOTYPES" ]
+
+      ldflags = [ "-pthread" ]
+
+      deps = [
+        ":ppapi_cpp_lib_shared",
+        "//ppapi/native_client/src/untrusted/irt_stub:ppapi_stub_lib",
+        "//build/config/nacl:nacl_base",
+      ]
+    }
+
     copy("nacl_tests_copy") {
       sources = [
         "${root_out_dir}/ppapi_nacl_tests_newlib.nexe",
       ]
       outputs = [
         "${root_build_dir}/{{source_name_part}}_${target_cpu}.nexe",
       ]
       deps = [
         ":ppapi_nacl_tests_newlib",
       ]
     }
+
+    copy("nacl_glibc_tests_copy") {
+      sources = [
+        "${root_out_dir}/ppapi_nacl_tests_glibc.nexe",
+      ]
+      outputs = [
+        "${root_build_dir}/{{source_name_part}}_${target_cpu}.nexe",

[brettw, 2015/09/10 05:02:50]

Actually, I just realized how this should really work. Sorry I didn't notice
this in the first pass.

In the nacl build, you'll have different toolchains for the glibc and newlib
builds. So you would never express a "glibc" target or have different variants
of the nmf action. You would have only one, and inside conditionally check flags
like CPU or the glibc/newlib variant and change things accordingly.

We'll need to add new builds args so it's easy for code to check the
glibc/newlib-ness of the current toolchain.

If you feel like this patch is a good first step, we can land as-is and fix in a
followup. But basically any target name with "glibc" in it is wrong.

[Petr Hosek, 2015/09/10 06:25:31]

I'm fine to fix it even in this change. Shall I introduce new args: is_glibc,
is_newlib; and set these accordingly inside the toolchain?

[Dirk Pranke, 2015/09/10 22:27:21]

You'll still need the target outputs to have "glibs" and "newlib" in the names,
though, right? So you can distinguish the two toolchains?

[Petr Hosek, 2015/09/11 22:10:49]

Done.

+      ]
+      deps = [
+        ":ppapi_nacl_tests_glibc",
+      ]
+    }
+
+    nacl_toolchain_dir = rebase_path("//native_client/toolchain")
+    os_toolchain_dir = "${nacl_toolchain_dir}/${host_os}_x86"
+    glibc_toolchain_dir = "${os_toolchain_dir}/nacl_x86_glibc"
+
+    action("generate_glibc_nmf") {
+      glibc_nmf = "${root_build_dir}/ppapi_nacl_tests_glibc.nmf"
+      script = "//native_client_sdk/src/tools/create_nmf.py"
+      sources = get_target_outputs(":nacl_glibc_tests_copy")
+      outputs = [
+        glibc_nmf,
+      ]
+      glibc_nmf_flags = [ "--library-path=" + rebase_path(root_out_dir) ]
+      if (current_cpu == "x86") {
+        libdir_glibc32 = "${glibc_toolchain_dir}/x86_64-nacl/lib32"
+        glibc_nmf_flags +=
+            [ "--library-path=" + rebase_path(libdir_glibc32, root_build_dir) ]
+      }
+      if (target_cpu == "x64" || (target_cpu == "x86" && is_win)) {
+        libdir_glibc64 = "${glibc_toolchain_dir}/x86_64-nacl/lib"
+        glibc_nmf_flags +=
+            [ "--library-path=" + rebase_path(libdir_glibc64, root_build_dir) ]
+      }
+      args = [
+               "--no-default-libpath",
+               "--objdump=${glibc_toolchain_dir}/bin/x86_64-nacl-objdump",
+               "--output=" + rebase_path(glibc_nmf, root_build_dir),
+               "--stage-dependencies=" + rebase_path(root_build_dir),
+             ] + glibc_nmf_flags + rebase_path(sources, root_build_dir)
+      deps = [
+        ":nacl_glibc_tests_copy",
+      ]
+    }
   }
 
   group("ppapi_nacl_tests") {
     deps = []
     if (target_cpu == "x86" || target_cpu == "x64") {
-      deps += [ ":nacl_tests_copy(//build/toolchain/nacl:clang_newlib_${target_cpu})" ]
+      deps += [
+        ":nacl_tests_copy(//build/toolchain/nacl:clang_newlib_${target_cpu})",
+        ":generate_glibc_nmf(//build/toolchain/nacl:glibc_${target_cpu})",
+      ]
     }
   }
 }