[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