Merge "releasetools: Make validate_target_files.py pylint clean."

am: a4c7d59afc

Change-Id: I3b9e9c15493bcf5b7d88de7161bcc9405d31ea4d
This commit is contained in:
Tao Bao
2018-02-02 03:21:03 +00:00
committed by android-build-merger

View File

@@ -23,13 +23,14 @@ It performs checks to ensure the integrity of the input zip.
same check also applies to the vendor image if present. same check also applies to the vendor image if present.
""" """
import common
import logging import logging
import os.path import os.path
import re import re
import sparse_img
import sys import sys
import common
import sparse_img
def _GetImage(which, tmpdir): def _GetImage(which, tmpdir):
assert which in ('system', 'vendor') assert which in ('system', 'vendor')
@@ -64,7 +65,7 @@ def _ReadFile(file_name, unpacked_name, round_up=False):
def ValidateFileAgainstSha1(input_tmp, file_name, file_path, expected_sha1): def ValidateFileAgainstSha1(input_tmp, file_name, file_path, expected_sha1):
"""Check if the file has the expected SHA-1.""" """Check if the file has the expected SHA-1."""
logging.info('Validating the SHA-1 of {}'.format(file_name)) logging.info('Validating the SHA-1 of %s', file_name)
unpacked_name = os.path.join(input_tmp, file_path) unpacked_name = os.path.join(input_tmp, file_path)
assert os.path.exists(unpacked_name) assert os.path.exists(unpacked_name)
actual_sha1 = _ReadFile(file_name, unpacked_name, False).sha1 actual_sha1 = _ReadFile(file_name, unpacked_name, False).sha1
@@ -147,10 +148,10 @@ def ValidateInstallRecoveryScript(input_tmp, info_dict):
script_path = 'SYSTEM/bin/install-recovery.sh' script_path = 'SYSTEM/bin/install-recovery.sh'
if not os.path.exists(os.path.join(input_tmp, script_path)): if not os.path.exists(os.path.join(input_tmp, script_path)):
logging.info('{} does not exist in input_tmp'.format(script_path)) logging.info('%s does not exist in input_tmp', script_path)
return return
logging.info('Checking {}'.format(script_path)) logging.info('Checking %s', script_path)
with open(os.path.join(input_tmp, script_path), 'r') as script: with open(os.path.join(input_tmp, script_path), 'r') as script:
lines = script.read().strip().split('\n') lines = script.read().strip().split('\n')
assert len(lines) >= 6 assert len(lines) >= 6
@@ -182,7 +183,8 @@ def ValidateInstallRecoveryScript(input_tmp, info_dict):
boot_info = applypatch_argv[boot_info_index].strip().split(':') boot_info = applypatch_argv[boot_info_index].strip().split(':')
assert len(boot_info) == 4 assert len(boot_info) == 4
ValidateFileAgainstSha1(input_tmp, file_name='boot.img', ValidateFileAgainstSha1(input_tmp, file_name='boot.img',
file_path='IMAGES/boot.img', expected_sha1=boot_info[3]) file_path='IMAGES/boot.img',
expected_sha1=boot_info[3])
recovery_sha1_index = boot_info_index + 2 recovery_sha1_index = boot_info_index + 2
expected_recovery_sha1 = applypatch_argv[recovery_sha1_index] expected_recovery_sha1 = applypatch_argv[recovery_sha1_index]
@@ -191,7 +193,7 @@ def ValidateInstallRecoveryScript(input_tmp, info_dict):
file_path='IMAGES/recovery.img', file_path='IMAGES/recovery.img',
expected_sha1=expected_recovery_sha1) expected_sha1=expected_recovery_sha1)
logging.info('Done checking {}'.format(script_path)) logging.info('Done checking %s', script_path)
def main(argv): def main(argv):