Update mig.py to take an explicit sdk argument.

third_party/crashpad/crashpad/util/mach/mig.py

 #!/usr/bin/env python
 # coding: utf-8
 
 # Copyright 2014 The Crashpad Authors. All rights reserved.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #
 #     http://www.apache.org/licenses/LICENSE-2.0
 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+import argparse
 import os
 import re
 import subprocess
 import sys
 
 def FixUserImplementation(implementation):
   """Rewrites a MIG-generated user implementation (.c) file.
 
   Rewrites the file at |implementation| by adding “__attribute__((unused))” to
   the definition of any structure typedefed as “__Reply” by searching for the
@@ skipping 72 matching lines @@
 #ifdef __cplusplus
 }
 #endif
 ''' % (contents, declarations_text)
   file.seek(0)
   file.truncate()
   file.write(contents)
   file.close()
 
 def main(args):
-  if len(args) == 5:
-    (defs_file, user_c, server_c, user_h, server_h) = args
-  elif len(args) == 6:
-    (defs_file, user_c, server_c, user_h, server_h, dev_dir) = args
-    os.environ['DEVELOPER_DIR'] = dev_dir
-  else:
-    assert False, "Wrong number of arguments"
-  subprocess.check_call(['mig',
-                         '-user', user_c,
-                         '-server', server_c,
-                         '-header', user_h,
-                         '-sheader', server_h,
-                         defs_file])
-  FixUserImplementation(user_c)
-  server_declarations = FixServerImplementation(server_c)
-  FixHeader(user_h)
-  FixHeader(server_h, server_declarations)
+  parser = argparse.ArgumentParser()
+  parser.add_argument('--developer-dir', help='Path to Xcode')
+  parser.add_argument('--sdk', help='Path to SDK')
+  parser.add_argument('defs')
+  parser.add_argument('user_c')
+  parser.add_argument('server_c')
+  parser.add_argument('user_h')
+  parser.add_argument('server_h')
+  parsed = parser.parse_args()

[Mark Mentovai, 2017/02/10 02:11:39]

OK.

Either put “args” here OR get rid of the “args” argument to main(), then. My
preference is the former (and I bet you’ll agree after you read the rest), but
if you really are opposed to it, it’s not the worst thing. You should at least
clean up after yourself and get rid of the unused argument if it is in fact
unused.

The reason that I prefer the “args” argument is so that you can use this as a
library. That way, some other code can import mig and call mig.main(arguments).
If you just use parse_args(), it always pulls from sys.argv, which makes this
impossible. Incidentally, this is also why we always put main into its own
function and call it wrapped in the __name__ check.

[erikchen, 2017/02/10 03:24:18]

Done

+
+  command = ['mig',
+             '-user', parsed.user_c,
+             '-server', parsed.server_c,
+             '-header', parsed.user_h,
+             '-sheader', parsed.server_h,
+            ]
+  if parsed.sdk is not None:
+    command.extend(['-isysroot', parsed.sdk])
+  if parsed.developer_dir is not None:
+    os.environ['DEVELOPER_DIR'] = parsed.developer_dir
+  command.append(parsed.defs)
+  subprocess.check_call(command)
+  FixUserImplementation(parsed.user_c)
+  server_declarations = FixServerImplementation(parsed.server_c)
+  FixHeader(parsed.user_h)
+  FixHeader(parsed.server_h, server_declarations)
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv[1:]))