diff --git a/tools/releasetools/add_img_to_target_files.py b/tools/releasetools/add_img_to_target_files.py index 7143775333..da7e11a93f 100644 --- a/tools/releasetools/add_img_to_target_files.py +++ b/tools/releasetools/add_img_to_target_files.py @@ -64,7 +64,7 @@ import verity_utils import ota_metadata_pb2 from apex_utils import GetApexInfoFromTargetFiles -from common import AddCareMapForAbOta +from common import AddCareMapForAbOta, ZipDelete if sys.hexversion < 0x02070000: print("Python 2.7 or newer is required.", file=sys.stderr) @@ -1034,9 +1034,10 @@ def OptimizeCompressedEntries(zipfile_path): if zinfo.compress_size > zinfo.file_size * 0.80 and zinfo.compress_type != zipfile.ZIP_STORED: entries_to_store.append(zinfo) zfp.extract(zinfo, tmpdir) + if len(entries_to_store) == 0: + return # Remove these entries, then re-add them as ZIP_STORED - common.RunAndCheckOutput( - ["zip", "-d", zipfile_path] + [entry.filename for entry in entries_to_store]) + ZipDelete(zipfile_path, [entry.filename for entry in entries_to_store]) with zipfile.ZipFile(zipfile_path, "a", allowZip64=True) as zfp: for entry in entries_to_store: zfp.write(os.path.join(tmpdir, entry.filename), entry.filename, compress_type=zipfile.ZIP_STORED) diff --git a/tools/releasetools/common.py b/tools/releasetools/common.py index 686102a58d..850619940b 100644 --- a/tools/releasetools/common.py +++ b/tools/releasetools/common.py @@ -2818,6 +2818,9 @@ def ZipDelete(zip_filename, entries): """ if isinstance(entries, str): entries = [entries] + # If list is empty, nothing to do + if not entries: + return cmd = ["zip", "-d", zip_filename] + entries RunAndCheckOutput(cmd) diff --git a/tools/releasetools/sign_target_files_apks.py b/tools/releasetools/sign_target_files_apks.py index 5b16d803cf..adba866371 100755 --- a/tools/releasetools/sign_target_files_apks.py +++ b/tools/releasetools/sign_target_files_apks.py @@ -1521,8 +1521,5 @@ def main(argv): if __name__ == '__main__': try: main(sys.argv[1:]) - except common.ExternalError as e: - print("\n ERROR: %s\n" % (e,)) - raise finally: common.Cleanup()