Fix WebRTC API framework build.

webrtc/build/ios/export_headers.py

 #!/usr/bin/python
 
 #  Copyright 2016 The WebRTC project authors. All Rights Reserved.
 #
 #  Use of this source code is governed by a BSD-style license
 #  that can be found in the LICENSE file in the root of the source
 #  tree. An additional intellectual property rights grant can be found
 #  in the file PATENTS.  All contributing project authors may
 #  be found in the AUTHORS file in the root of the source tree.
 
 """Script for exporting iOS header files."""
 
 import errno
 import optparse
 import os
-import re
 import shutil
 import sys
 
 LEGACY_HEADER_DIRS = ['talk/app/webrtc/objc/public', 'webrtc/base/objc/']
 HEADER_DIRS = ['webrtc/api/objc/', 'webrtc/base/objc/',
                'webrtc/modules/audio_device/ios/objc']
 # Individual header files that should also be exported.
 LEGACY_HEADER_INCLUDES = []
 HEADER_INCLUDES = []
 # Individual header files that should not be exported.
 LEGACY_HEADER_EXCLUDES = ['talk/app/webrtc/objc/public/RTCNSGLVideoView.h']
 HEADER_EXCLUDES = [
   'webrtc/api/objc/avfoundationvideocapturer.h',
   'webrtc/api/objc/RTCNSGLVideoView.h',
+  'webrtc/api/objc/RTCVideoRendererAdapter.h',
   'webrtc/base/objc/NSString+StdString.h',
   'webrtc/base/objc/RTCUIApplication.h',
   'webrtc/modules/audio_device/ios/objc/RTCAudioSessionDelegateAdapter.h',
 ]
 
 def ExportHeaders(include_base_dir, use_legacy_headers):
   """Exports iOS header files.
 
   Creates an include directory and recreates the hierarchy for the header files
   within the include directory.
 
   Args:
     include_base_dir: directory where the include directory should be created
   """
 
   include_dir_name = 'include'
   include_path = os.path.join(include_base_dir, include_dir_name)
   # Remove existing directory first in case files change.
-  if (os.path.exists(include_path)):
+  if os.path.exists(include_path):

[kjellander_webrtc, 2016/04/15 04:59:41]

Aha! With the .py extension you couldn't escape the PyLint I assume? :)

[tkchin_webrtc, 2016/04/16 00:34:18]

Indeed! I'm glad I decided to rename it :)

     shutil.rmtree(include_path)
 
   script_path = sys.path[0]
   webrtc_base_path = os.path.join(script_path, '../../..')
 
   header_dirs = HEADER_DIRS
   include_headers = HEADER_INCLUDES
   exclude_headers = HEADER_EXCLUDES
   if use_legacy_headers:
     header_dirs = LEGACY_HEADER_DIRS
@@ skipping 14 matching lines @@
     output_dir = os.path.join(include_path, os.path.dirname(header_path))
     # Create hierarchy for the header file within the include directory.
     try:
       os.makedirs(output_dir)
     except OSError as exc:
       if exc.errno != errno.EEXIST:
         raise exc
     current_path = os.path.join(webrtc_base_path, header_path)
     new_path = os.path.join(include_path, header_path)
     shutil.copy(current_path, new_path)
 

[kjellander_webrtc, 2016/04/15 04:59:41]

nit: +1 blank line before top-level definition
https://google.github.io/styleguide/pyguide.html?showone=Blank_Lines#Blank_Lines
Applies to whole file.

[tkchin_webrtc, 2016/04/16 00:34:19]

Done.

 def Main():
   parser = optparse.OptionParser()
   _, args = parser.parse_args()

[kjellander_webrtc, 2016/04/15 04:59:41]

If you're only using positional arguments you might as well use
https://docs.python.org/2/library/sys.html#sys.argv instead.

[tkchin_webrtc, 2016/04/16 00:34:18]

Done.

   if len(args) != 2:
     parser.error('Error: Exactly 2 arguments required.')
   include_base_dir = args[0]
   use_legacy_headers = False if int(args[1]) == 0 else True
   ExportHeaders(include_base_dir, use_legacy_headers)
 
 if __name__ == '__main__':
   sys.exit(Main())