laforge submitted this change.

View Change



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: Jenkins Builder: Verified dexter: Looks good to me, approved fixeria: Looks good to me, but someone else must approve
saip-tool: Fix TAR display for implicit TAR

Until Change-Id Ifba1048e3000829d54769b0420f5134e2f9b04e1 the TAR
output was working for implicit tar. With said commit we fixed it
for explicit tar but broke implicit tar.

With this commit it works for both implicit and explicit TAR.

Change-Id: I76133b0e02996a138257f3fba5ceb0d2fc6fad80
---
M contrib/saip-tool.py
1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/contrib/saip-tool.py b/contrib/saip-tool.py
index 8208602..d9d33c0 100755
--- a/contrib/saip-tool.py
+++ b/contrib/saip-tool.py
@@ -202,7 +202,8 @@
adf = rfm.decoded.get('adfRFMAccess', None)
if adf:
print("\tADF AID: %s" % b2h(adf['adfAID']))
- for tar in rfm.decoded['tarList']:
+ tar_list = rfm.decoded.get('tarList', [inst_aid[-3:]])
+ for tar in tar_list:
print("\tTAR: %s" % b2h(tar))

def do_extract_apps(pes:ProfileElementSequence, opts):

To view, visit change 37541. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I76133b0e02996a138257f3fba5ceb0d2fc6fad80
Gerrit-Change-Number: 37541
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged