[infra] Fix copy_tree.py to avoid extra rebuilding in Fuchsia

[infra] Fix copy_tree.py to avoid extra rebuilding in Fuchsia

This CL computes the correct input and output in the copy_tree GN template by
modifying tools/copy_tree.py to perform a dry-run that lists the files to be copied.

Also, shutil.copytree uses shutil.copy2 to copy files and their metadata, but copying
the metadata appears to confuse ninja dependency tracking.

To do the dry run and to use shutil.copy instead of shutil.copy2, copy_tree.py now
uses its own CopyTree function instead of shutil.copytree.

R=whesse@google.com

Committed: https://github.com/dart-lang/sdk/commit/5ec18fc68e993167ea54d5e9949b338622de19d6

[zra, 2017-05-11 06:05:27]

Description was changed from

==========
[infra] Fix copy_tree.py to avoid extra rebuilding in Fuchsia
==========

to

==========
[infra] Fix copy_tree.py to avoid extra rebuilding in Fuchsia

This CL computes the correct input and output in the copy_tree GN template by
modifying tools/copy_tree.py to perform a dry-run that lists the files to be
copied.

Also, shutil.copytree uses shutil.copy2 to copy files and their metadata, but
copying
the metadata appears to confuse ninja dependency tracking.

To do the dry run and to use shutil.copy instead of shutil.copy2, copy_tree.py
now
uses its own CopyTree function instead of shutil.copytree.
==========

[zra, 2017-05-11 06:08:21]

zra@google.com changed reviewers:
+ asiva@google.com, whesse@google.com

[zra, 2017-05-11 06:08:22]

This is split off from a CL I sent earlier today. Please take a look at this one
first. Thanks!

[Bill Hesse, 2017-05-11 15:04:12]

lgtm

[zra, 2017-05-11 16:46:19]

Description was changed from

==========
[infra] Fix copy_tree.py to avoid extra rebuilding in Fuchsia

This CL computes the correct input and output in the copy_tree GN template by
modifying tools/copy_tree.py to perform a dry-run that lists the files to be
copied.

Also, shutil.copytree uses shutil.copy2 to copy files and their metadata, but
copying
the metadata appears to confuse ninja dependency tracking.

To do the dry run and to use shutil.copy instead of shutil.copy2, copy_tree.py
now
uses its own CopyTree function instead of shutil.copytree.
==========

to

==========
[infra] Fix copy_tree.py to avoid extra rebuilding in Fuchsia

This CL computes the correct input and output in the copy_tree GN template by
modifying tools/copy_tree.py to perform a dry-run that lists the files to be
copied.

Also, shutil.copytree uses shutil.copy2 to copy files and their metadata, but
copying
the metadata appears to confuse ninja dependency tracking.

To do the dry run and to use shutil.copy instead of shutil.copy2, copy_tree.py
now
uses its own CopyTree function instead of shutil.copytree.

R=whesse@google.com

Committed:
https://github.com/dart-lang/sdk/commit/5ec18fc68e993167ea54d5e9949b338622de19d6
==========

[zra, 2017-05-11 16:46:20]

Committed patchset #2 (id:20001) manually as
5ec18fc68e993167ea54d5e9949b338622de19d6 (presubmit successful).