On Sun, 2019-05-05 at 22:20 +0200, Petr Vorel wrote: > Patchwork didn't recognise some patches due missing some extended header > lines (e.g. "old mode" and "new mode" for renaming file mode, see [1]). > Thus adding all modes from git doc [2]. > > Fixes: #267 > > [1] https://lists.ozlabs.org/pipermail/patchwork/2019-April/005741.html > [2] https://git-scm.com/docs/git-diff#_generating_patches_with_p > > Suggested-by: Veronika Kabatova <[email protected]> > Signed-off-by: Petr Vorel <[email protected]>
Apologies for the delay in getting to this. This is merged now, though I also had to include a follow up to ensure the additional lines were captured in the diff. https://github.com/getpatchwork/patchwork/commit/fc1d7505991474b07638a87c0834157d12e1e476 I'll backport this to stable/2.1 shortly. Thanks! Stephen > --- > patchwork/parser.py | 12 ++++++++++-- > 1 file changed, 10 insertions(+), 2 deletions(-) > > diff --git a/patchwork/parser.py b/patchwork/parser.py > index 712780a..633fc71 100644 > --- a/patchwork/parser.py > +++ b/patchwork/parser.py > @@ -752,7 +752,7 @@ def parse_patch(content): > # 3 -> 4 (@@ line) > # 4 -> 5 (patch content) > # 5 -> 1 (run out of lines from @@-specifed count) > - # 1 -> 6 (rename from / rename to / new file / index) > + # 1 -> 6 (extended header lines) > # 6 -> 2 (---) > # 6 -> 1 (other text) > # > @@ -782,7 +782,15 @@ def parse_patch(content): > if line.startswith('--- '): > state = 2 > > - if line.startswith(('rename from ', 'rename to ', > + # extended header lines > + # @see > https://git-scm.com/docs/git-diff#_generating_patches_with_p > + if line.startswith(('old mode ', 'new mode ', > + 'deleted file mode ', > + 'new file mode ', > + 'copy from ', 'copy to ', > + 'rename from ', 'rename to ', > + 'similarity index ', > + 'dissimilarity index ', > 'new file mode ', 'index ')): > state = 6 > elif state == 2: _______________________________________________ Patchwork mailing list [email protected] https://lists.ozlabs.org/listinfo/patchwork
