Makes copy_ex handle directories

build/android/gyp/copy_ex.py

Index: build/android/gyp/copy_ex.py
diff --git a/build/android/gyp/copy_ex.py b/build/android/gyp/copy_ex.py
index eee3d191747c074545ebc24f5a37cfc0ac01e6ec..76d4fe0c48b6a5cfe34bef3db450f0ac1e2f5739 100755
--- a/build/android/gyp/copy_ex.py
+++ b/build/android/gyp/copy_ex.py
@@ -7,6 +7,7 @@
 """Copies files to a directory."""
 
 import optparse
+import os
 import shutil
 import sys
 
@@ -39,7 +40,10 @@ def main(args):
     files += build_utils.ParseGypList(file_arg)
 
   for f in files:
-    shutil.copy(f, options.dest)
+    if os.path.isdir(f):
+      shutil.copytree(f, os.path.join(options.dest, os.path.basename(f)))

[cjhopman, 2015/05/08 22:58:55]

I would require that options.clear be set if any directories are copied.

[sky, 2015/05/08 23:38:31]

Why is it different than with files? The same problem exists in either case, you
may not realize there are stale files around. Seems to me you're arguing for
always clearing, which seems separate from adding support for directories.

I had started adding the warning, but then realized it starts sounding very
weird. Basically it comes down to something like "we don't trust you, use
clear=true when copying directories."

[cjhopman, 2015/05/08 23:57:05]

Yeah, exactly, I don't trust people (including myself) to do this correctly.

In the case where you are only copying files, I would expect that you will be
able to pass your actual outputs to the consumer (because you were able to tell
copy_ex your actual inputs).

The problem is that if you are copying a directory here, then you are probably
telling whoever uses the outputs to look in a directory (because if you can tell
the consumer the exact files, then why not just tell the copy step the exact
files). If you're doing that, then it's really difficult for the consumer to
detect and ignore stale files.

I would consider requiring options.stamp, too, but that's really only necessary
in gyp, not GN. The issue is that you need to be able to specify some sort of
output that is an input to the consumer so that the build knows the consumer
needs to run. In GN, just having a dependency is enough for that.

+    else:
+      shutil.copy(f, options.dest)
 
   if options.depfile:
     build_utils.WriteDepfile(