Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

parser doesn't work with diffs from git merge commits

Aberta
#284 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
42/100
Tipo de issue
Bug
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
python
Domínio
backend

Direção de pesquisa

Comece localizando o código do parser que trata cabeçalhos de unified diff e a expressão regular ou o caminho de parsing para o cabeçalho de hunk @@@. Compare o comportamento dele com o exemplo mostrado de diff --cc e com um diff normal, depois adicione cobertura se o repositório tiver testes do parser. O trabalho estará concluído quando diffs de commits de merge forem reconhecidos sem quebrar o parsing normal de patches.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

bug

@mpe reported that https://patchwork.ozlabs.org/patch/1129056/ is missing a patch diff.

It appears this is because it's a diff of a git merge commit:

diff --cc arch/powerpc/include/asm/pgtable.h
index bf7d771f342e,64145751b2fd..000000000000
--- a/arch/powerpc/include/asm/pgtable.h
+++ b/arch/powerpc/include/asm/pgtable.h
@@@ -140,30 -140,20 +140,44 @@@ static inline void pte_frag_set(mm_cont
  }
  #endif

as opposed to a normal diff

diff --git a/arch/powerpc/kernel/module_64.c b/arch/powerpc/kernel/module_64.c
index 80ceb458e79c..ef75f590ad3e 100644
--- a/arch/powerpc/kernel/module_64.c
+++ b/arch/powerpc/kernel/module_64.c
@@ -133,20 +133,27 @@ struct ppc64_stub_entry
  * the stub, but it's significantly shorter to put these values at the
  * end of the stub code, and patch the stub address (32-bits relative

The part that the parser trips on is the unusual format of the @@@ header line because of diff --cc mode.

This isn't too high priority since sending a merge diff to a mailing list is not the normal case, but we might still want to consider fixing the regex to cope with this.

Linguagem predominante
Python
Estrelas
317
Forks
91
Nenhum PR com merge em 30d

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de getpatchwork/patchwork

Todas as issues de getpatchwork/patchwork

Issues semelhantes

Mais issues de Python

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.