Reformat python files checked by pylint (part 1/2).
After recently changing .pylintrc (see [1]) we discovered that
the presubmit check always checks all the python files when just
one python file gets updated.
This CL moves all these files one step closer to what the linter
wants.
Autogenerated with:
# Added all the files under pylint control to ~/Desktop/to-reformat
cat ~/Desktop/to-reformat | xargs sed -i '1i\\'
git cl format --python --full
This is part 1 out of 2. The second part will fix function names and
will not be automated.
[1] - https://webrtc-review.googlesource.com/c/src/+/186664
No-Presubmit: True
Bug: webrtc:12114
Change-Id: Idfec4d759f209a2090440d0af2413a1ddc01b841
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/190980
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#32530}
diff --git a/tools_webrtc/ios/merge_ios_libs.py b/tools_webrtc/ios/merge_ios_libs.py
index 651024e..31ffc1d 100755
--- a/tools_webrtc/ios/merge_ios_libs.py
+++ b/tools_webrtc/ios/merge_ios_libs.py
@@ -7,7 +7,6 @@
# 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 merging generated iOS libraries."""
import sys
@@ -22,7 +21,7 @@
def MergeLibs(lib_base_dir):
- """Merges generated iOS libraries for different archs.
+ """Merges generated iOS libraries for different archs.
Uses libtool to generate FAT archive files for each generated library.
@@ -33,92 +32,96 @@
Returns:
Exit code of libtool.
"""
- output_dir_name = 'fat_libs'
- archs = [arch for arch in os.listdir(lib_base_dir)
- if arch in VALID_ARCHS]
- # For each arch, find (library name, libary path) for arch. We will merge
- # all libraries with the same name.
- libs = {}
- for lib_dir in [os.path.join(lib_base_dir, arch) for arch in VALID_ARCHS]:
- if not os.path.exists(lib_dir):
- continue
- for dirpath, _, filenames in os.walk(lib_dir):
- for filename in filenames:
- if not filename.endswith('.a'):
- continue
- entry = libs.get(filename, [])
- entry.append(os.path.join(dirpath, filename))
- libs[filename] = entry
- orphaned_libs = {}
- valid_libs = {}
- for library, paths in libs.items():
- if len(paths) < len(archs):
- orphaned_libs[library] = paths
- else:
- valid_libs[library] = paths
- for library, paths in orphaned_libs.items():
- components = library[:-2].split('_')[:-1]
- found = False
- # Find directly matching parent libs by stripping suffix.
- while components and not found:
- parent_library = '_'.join(components) + '.a'
- if parent_library in valid_libs:
- valid_libs[parent_library].extend(paths)
- found = True
- break
- components = components[:-1]
- # Find next best match by finding parent libs with the same prefix.
- if not found:
- base_prefix = library[:-2].split('_')[0]
- for valid_lib, valid_paths in valid_libs.items():
- if valid_lib[:len(base_prefix)] == base_prefix:
- valid_paths.extend(paths)
- found = True
- break
- assert found
+ output_dir_name = 'fat_libs'
+ archs = [arch for arch in os.listdir(lib_base_dir) if arch in VALID_ARCHS]
+ # For each arch, find (library name, libary path) for arch. We will merge
+ # all libraries with the same name.
+ libs = {}
+ for lib_dir in [os.path.join(lib_base_dir, arch) for arch in VALID_ARCHS]:
+ if not os.path.exists(lib_dir):
+ continue
+ for dirpath, _, filenames in os.walk(lib_dir):
+ for filename in filenames:
+ if not filename.endswith('.a'):
+ continue
+ entry = libs.get(filename, [])
+ entry.append(os.path.join(dirpath, filename))
+ libs[filename] = entry
+ orphaned_libs = {}
+ valid_libs = {}
+ for library, paths in libs.items():
+ if len(paths) < len(archs):
+ orphaned_libs[library] = paths
+ else:
+ valid_libs[library] = paths
+ for library, paths in orphaned_libs.items():
+ components = library[:-2].split('_')[:-1]
+ found = False
+ # Find directly matching parent libs by stripping suffix.
+ while components and not found:
+ parent_library = '_'.join(components) + '.a'
+ if parent_library in valid_libs:
+ valid_libs[parent_library].extend(paths)
+ found = True
+ break
+ components = components[:-1]
+ # Find next best match by finding parent libs with the same prefix.
+ if not found:
+ base_prefix = library[:-2].split('_')[0]
+ for valid_lib, valid_paths in valid_libs.items():
+ if valid_lib[:len(base_prefix)] == base_prefix:
+ valid_paths.extend(paths)
+ found = True
+ break
+ assert found
- # Create output directory.
- output_dir_path = os.path.join(lib_base_dir, output_dir_name)
- if not os.path.exists(output_dir_path):
- os.mkdir(output_dir_path)
+ # Create output directory.
+ output_dir_path = os.path.join(lib_base_dir, output_dir_name)
+ if not os.path.exists(output_dir_path):
+ os.mkdir(output_dir_path)
- # Use this so libtool merged binaries are always the same.
- env = os.environ.copy()
- env['ZERO_AR_DATE'] = '1'
+ # Use this so libtool merged binaries are always the same.
+ env = os.environ.copy()
+ env['ZERO_AR_DATE'] = '1'
- # Ignore certain errors.
- libtool_re = re.compile(r'^.*libtool:.*file: .* has no symbols$')
+ # Ignore certain errors.
+ libtool_re = re.compile(r'^.*libtool:.*file: .* has no symbols$')
- # Merge libraries using libtool.
- libtool_returncode = 0
- for library, paths in valid_libs.items():
- cmd_list = ['libtool', '-static', '-v', '-o',
- os.path.join(output_dir_path, library)] + paths
- libtoolout = subprocess.Popen(cmd_list, stderr=subprocess.PIPE, env=env)
- _, err = libtoolout.communicate()
- for line in err.splitlines():
- if not libtool_re.match(line):
- print >>sys.stderr, line
- # Unconditionally touch the output .a file on the command line if present
- # and the command succeeded. A bit hacky.
- libtool_returncode = libtoolout.returncode
- if not libtool_returncode:
- for i in range(len(cmd_list) - 1):
- if cmd_list[i] == '-o' and cmd_list[i+1].endswith('.a'):
- os.utime(cmd_list[i+1], None)
- break
- return libtool_returncode
+ # Merge libraries using libtool.
+ libtool_returncode = 0
+ for library, paths in valid_libs.items():
+ cmd_list = [
+ 'libtool', '-static', '-v', '-o',
+ os.path.join(output_dir_path, library)
+ ] + paths
+ libtoolout = subprocess.Popen(cmd_list,
+ stderr=subprocess.PIPE,
+ env=env)
+ _, err = libtoolout.communicate()
+ for line in err.splitlines():
+ if not libtool_re.match(line):
+ print >> sys.stderr, line
+ # Unconditionally touch the output .a file on the command line if present
+ # and the command succeeded. A bit hacky.
+ libtool_returncode = libtoolout.returncode
+ if not libtool_returncode:
+ for i in range(len(cmd_list) - 1):
+ if cmd_list[i] == '-o' and cmd_list[i + 1].endswith('.a'):
+ os.utime(cmd_list[i + 1], None)
+ break
+ return libtool_returncode
def Main():
- parser_description = 'Merge WebRTC libraries.'
- parser = argparse.ArgumentParser(description=parser_description)
- parser.add_argument('lib_base_dir',
- help='Directory with built libraries. ',
- type=str)
- args = parser.parse_args()
- lib_base_dir = args.lib_base_dir
- MergeLibs(lib_base_dir)
+ parser_description = 'Merge WebRTC libraries.'
+ parser = argparse.ArgumentParser(description=parser_description)
+ parser.add_argument('lib_base_dir',
+ help='Directory with built libraries. ',
+ type=str)
+ args = parser.parse_args()
+ lib_base_dir = args.lib_base_dir
+ MergeLibs(lib_base_dir)
+
if __name__ == '__main__':
- sys.exit(Main())
+ sys.exit(Main())