Started adding the engine hash to frameworks' Info.plist. (#9847)

Started adding the engine hash to Flutter.framework's Info.plist.

build/copy_info_plist.py

+#!/usr/bin/env python
+#
+# Copyright 2013 The Flutter Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+"""
+Copies the Info.plist and adds extra fields to it like the git hash of the
+engine.
+
+Precondition: $CWD/../../flutter is the path to the flutter engine repo.
+
+usage: copy_info_plist.py <src_path> <dest_path>
+"""
+
+from __future__ import absolute_import
+from __future__ import division
+from __future__ import print_function
+
+import sys
+import git_revision
+import os
+
+def main():
+  text = open(sys.argv[1]).read()
+  engine_path = os.path.join(os.getcwd(), "..", "..", "flutter")
+  revision = git_revision.GetRepositoryVersion(engine_path)
+  text = text.format(revision)
+  with open(sys.argv[2], "w") as outfile:
+    outfile.write(text)
+
+if __name__ == "__main__":
+  main()

build/git_revision.py

@@ -5,26 +5,19 @@
 # found in the LICENSE file.
 
 """Get the Git HEAD revision of a specified Git repository."""
+from __future__ import absolute_import
+from __future__ import division
+from __future__ import print_function
 
 import sys
 import subprocess
 import os
 import argparse
 
-def main():
-  parser = argparse.ArgumentParser();
-
-  parser.add_argument('--repository',
-                      action='store',
-                      help='Path to the Git repository.',
-                      required=True)
-
-  args = parser.parse_args()
-
-  repository = os.path.abspath(args.repository)
-
+def GetRepositoryVersion(repository):
+  "Returns the Git HEAD for the supplied repository path as a string."
   if not os.path.exists(repository):
-    exit -1
+    raise IOError("path doesn't exist")
 
   version = subprocess.check_output([
     'git',
@@ -34,7 +27,20 @@ def main():
     'HEAD',
   ])
 
-  print (version.strip())
+  return version.strip()
+
+def main():
+  parser = argparse.ArgumentParser()
+
+  parser.add_argument('--repository',
+                      action='store',
+                      help='Path to the Git repository.',
+                      required=True)
+
+  args = parser.parse_args()
+  repository = os.path.abspath(args.repository)
+  version = GetRepositoryVersion(repository)
+  print(version.strip())
 
   return 0
 

shell/platform/darwin/ios/BUILD.gn

@@ -177,7 +177,8 @@ action("copy_dylib_and_update_framework_install_name") {
   ]
 }
 
-copy("copy_framework_info_plist") {
+action("copy_framework_info_plist") {
+  script = "$flutter_root/build/copy_info_plist.py"
   visibility = [ ":*" ]
   sources = [
     "framework/Info.plist",
@@ -185,6 +186,10 @@ copy("copy_framework_info_plist") {
   outputs = [
     "$_flutter_framework_dir/Info.plist",
   ]
+  args = [
+    rebase_path(sources[0]),
+    rebase_path(outputs[0]),
+  ]
 }
 
 copy("copy_framework_module_map") {

shell/platform/darwin/ios/framework/Info.plist

@@ -22,5 +22,7 @@
   <string>1.0</string>
   <key>MinimumOSVersion</key>
   <string>8.0</string>
+  <key>FlutterEngine<key>
+  <string>{0}<string>
 </dict>
 </plist>

shell/platform/embedder/BUILD.gn

@@ -153,7 +153,8 @@ if (is_mac && !embedder_for_target) {
     ]
   }
 
-  copy("copy_info_plist") {
+  action("copy_info_plist") {
+    script = "$flutter_root/build/copy_info_plist.py"
     visibility = [ ":*" ]
     sources = [
       "assets/EmbedderInfo.plist",
@@ -161,6 +162,10 @@ if (is_mac && !embedder_for_target) {
     outputs = [
       "$_flutter_embedder_framework_dir/Versions/A/Resources/Info.plist",
     ]
+    args = [
+      rebase_path(sources[0]),
+      rebase_path(outputs[0]),
+    ]
   }
 
   copy("copy_module_map") {

shell/platform/embedder/assets/EmbedderInfo.plist

@@ -24,5 +24,7 @@
   <string>1</string>
   <key>NSHumanReadableCopyright</key>
   <string>Copyright 2013 The Flutter Authors. All rights reserved.</string>
+  <key>FlutterEngine<key>
+  <string>{0}<string>
 </dict>
 </plist>