If we fail to decode a message payload, we'll fail with the following: Traceback (most recent call last): File "./patchwork/bin/parsemail.py", line 563, in <module> sys.exit(main(sys.argv)) File "./patchwork/bin/parsemail.py", line 553, in main return parse_mail(mail, args['list_id']) File "./patchwork/bin/parsemail.py", line 464, in parse_mail (patch, comment, filenames) = find_content(project, mail) ValueError: need more than 2 values to unpack
- as the error condition for find_content only returns a 2-tuple. This change fixes the error case to the 3-tuple return type. Signed-off-by: Jeremy Kerr <j...@ozlabs.org> --- patchwork/bin/parsemail.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/patchwork/bin/parsemail.py b/patchwork/bin/parsemail.py index f369fe4..3147431 100755 --- a/patchwork/bin/parsemail.py +++ b/patchwork/bin/parsemail.py @@ -231,7 +231,7 @@ def find_content(project, mail): # Could not find a valid decoded payload. Fail. if payload is None: - return (None, None) + return (None, None, None) if subtype in ['x-patch', 'x-diff']: patchbuf = payload -- 2.5.0 _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork