> The patch is a preparation step towards understanding series. It will be
> handy to parse those prefixes, looking for 'x/n' to retrieve the order
> of a patch in a series.
>
> Signed-off-by: Damien Lespiau <damien.lesp...@intel.com>
One comments below.
Stephen
> diff --git a/patchwork/bin/parsemail.py b/patchwork/bin/parsemail.py
> index c15564e..ae588f0 100755
> --- a/patchwork/bin/parsemail.py
> +++ b/patchwork/bin/parsemail.py
> @@ -213,7 +213,8 @@ def find_content(project, mail):
> comment = None
>
> if pullurl or patchbuf:
> - name = clean_subject(mail.get('Subject'), [project.linkname])
> + (name, prefixes) = clean_subject(mail.get('Subject'),
> + [project.linkname])
> patch = Patch(name = name, pull_url = pullurl, content = patchbuf,
> date = mail_date(mail), headers = mail_headers(mail))
Are we using prefixes anywhere? If not, you should replace 'prefixes' with '_'
(unnamed variable). If it's used in a later patch (I haven’t read ahead) then
ignore this comment.
_______________________________________________
Patchwork mailing list
Patchwork@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/patchwork
