[Mac/iOS/GN] copy_bundle_data.py should preserve symlinks.

build/toolchain/mac/copy_bundle_data.py

 # Copyright 2016 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 argparse
 import os
 import shutil
 import sys
 
 
@@ skipping 52 matching lines @@
   Args:
     source: string, path to the source file
     dest: string, path to the destination file
   """
   if os.path.isdir(source):
     if os.path.exists(dest):
       shutil.rmtree(dest)
     # Copy tree.
     # TODO(thakis): This copies file attributes like mtime, while the
     # single-file branch below doesn't. This should probably be changed to
     # be consistent with the single-file branch.

[Nico, 2016/06/09 14:20:53]

it looks like this was copy-pasted from somewhere...why wasn't this needed where
it's from?

[Robert Sesek, 2016/06/09 14:22:35]

It was a boog from before. We never included symlinks in our bundle, but we may
start doing so in the future if we start versioning the Chrome Framework (which
I am proposing we do...).

-    shutil.copytree(source, dest)
+    shutil.copytree(source, dest, symlinks=True)
     return
 
   if os.path.exists(dest):
     os.unlink(dest)
 
   _, extension = os.path.splitext(source)
   if extension == '.strings':
     copy_strings_file(source, dest)
     return
 
   shutil.copy(source, dest)
 
 
 def main():
   parser = argparse.ArgumentParser(
       description='copy source to destination for the creation of a bundle')
   parser.add_argument('source', help='path to source file or directory')
   parser.add_argument('dest', help='path to destination')
   args = parser.parse_args()
 
   copy_file(args.source, args.dest)
 
 if __name__ == '__main__':
   main()