commit 87e452739e5d0210fa0fd6f29475e39789654a17
author Kelvin Zhang <zhangkelvin@google.com> Wed Jul 27 11:14:12 2022 -0700
committer Kelvin Zhang <zhangkelvin@google.com> Wed Jul 27 11:15:58 2022 -0700
tree eda19f2b3a68ce411b033f63c6311219540f0560
parent d024518e1e69e798112ed452a294606f47cf0350

Fix typo in recently changed extra_apex_payload_key flag

The intention was to make extra_apex_payload_key take a comma separted
list of keys. Iterating over a string will only iterate over each
characters individually. Must split by "," to iterate over keys.

Test: Run signing test locally using tradefed.sh
Bug: 239991438
Change-Id: I51bfed53a43b1b0fc1e33e0dc3d418a727e6ff15

tools/releasetools/sign_target_files_apks.py

diff --git a/tools/releasetools/sign_target_files_apks.py b/tools/releasetools/sign_target_files_apks.py
index dbe79ab..4f2b1bf 100755
--- a/tools/releasetools/sign_target_files_apks.py
+++ b/tools/releasetools/sign_target_files_apks.py
@@ -1381,7 +1381,7 @@
         OPTIONS.extra_apks[n] = key
     elif o == "--extra_apex_payload_key":
       apex_names, key = a.split("=")
-      for name in apex_names:
+      for name in apex_names.split(","):
         OPTIONS.extra_apex_payload_keys[name] = key
     elif o == "--skip_apks_with_path_prefix":
       # Check the prefix, which must be in all upper case.