From 620c1e59735a09828f2e326056a2a76474c0ac59 Mon Sep 17 00:00:00 2001 From: Matt Mower Date: Fri, 29 Jan 2016 18:24:16 -0600 Subject: repopick: Don't crash if change not found If change is not located, skip it. Resolves the following: Traceback (most recent call last): File "(omitted)/build/tools/repopick.py", line 258, in review = [x for x in reviews if x['number'] == change][0] IndexError: list index out of range Change-Id: I96423ddc809ac1c7994998db4e1929ca813f20ca --- tools/repopick.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'tools/repopick.py') diff --git a/tools/repopick.py b/tools/repopick.py index 64db7bfc5..1840a566d 100755 --- a/tools/repopick.py +++ b/tools/repopick.py @@ -255,7 +255,11 @@ if __name__ == '__main__': continue change = int(change) - review = [x for x in reviews if x['number'] == change][0] + review = next((x for x in reviews if x['number'] == change), None) + if review is None: + print('Change %d not found, skipping' % change) + continue + mergables.append({ 'subject': review['subject'], 'project': review['project'], -- cgit v1.2.3