Fix solink and add a postsolink hook (use the hook for android strip)

build/toolchain/android/BUILD.gn

Index: build/toolchain/android/BUILD.gn
diff --git a/build/toolchain/android/BUILD.gn b/build/toolchain/android/BUILD.gn
index dc3bfeddab776e9bdfe9da5358d7e68599319f4b..0355a8c6b21dc77c048cbc9f466468709465e767 100644
--- a/build/toolchain/android/BUILD.gn
+++ b/build/toolchain/android/BUILD.gn
@@ -39,6 +39,12 @@ template("android_gcc_toolchain") {
 
     toolchain_os = "android"
     toolchain_cpu_arch = invoker.toolchain_cpu_arch
+
+    android_strip = "${tool_prefix}strip"
+    mkdir_command = "mkdir -p lib.stripped"

[cjhopman, 2014/07/07 23:00:50]

This mkdir isn't needed if ninja knows about the output file.

[brettw, 2014/07/08 22:06:54]

Can you add a comment above this line about why it's needed?

[Nico, 2014/07/08 23:02:29]

Can we not use mkdir? It's not available on Windows (well, it is, but not with a
'-p' flag I think), and I thought one of the points of gn was to make
chrome/android hostable on more platforms

[cjhopman, 2014/07/11 20:48:50]

The right way to do this is to tell ninja about the output file in that
directory.

The next most "correct" way would probably be to make a separate target that
creates this directory (or actually just a file in it), and then have these
toolchains depend on that target. That's a bit more complicated than I would
like to do just to potentially make it easier to support other platforms...

I really only need the -p so that it doesn't fail if the directory already
exists, can i do (mkdir lib.stripped || true)? How about (if [ ! -e
lib.stripped]; then mkdir lib.stripped; fi)?

Really, my preference for this postsolink hook would be that I could write it as
a normal gn action (or whatever else) that just run as part of solink and had
access to the outputs. Then it would be easy to correctly handle other platforms
(or rather, it wouldn't be a special case and could be handled like all the
other gn stuff).

+    strip_command = "$android_strip --strip-unneeded -o lib.stripped/\$soname.tmp \$lib"
+    replace_command = "if ! cmp -s lib.stripped/\${soname}.tmp lib.stripped/\${soname}; then mv lib.stripped/\${soname}.tmp lib.stripped/\${soname}; fi"
+    postsolink = "$mkdir_command && $strip_command && $replace_command"
   }
 }