Merge "releasetools: Make validate_target_files.py pylint clean."
am: a4c7d59afc
Change-Id: I3b9e9c15493bcf5b7d88de7161bcc9405d31ea4d
This commit is contained in:
@@ -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):
|
||||||
|
Reference in New Issue
Block a user