[riot-notifications] [RIOT-OS/RIOT] tests/riotboot: make automatic script work with cc2538-bsl (#15446)

Alexandre Abadie notifications at github.com
Mon Nov 16 12:19:51 CET 2020


@aabadie commented on this pull request.



>  
 
 if __name__ == "__main__":
-    sys.exit(run(testfunc))
+    res = 1
+    global current_app_ver
+    global current_slot_num
+    # set current slot number and app version
+    res = run(testcurrent)

You should explain why you have to call the testrunner separately to get the slot number/app version. If possible, I would avoid that.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15446#pullrequestreview-531229779
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201116/f3ced98b/attachment.htm>


More information about the notifications mailing list