Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

parser doesn't work with diffs from git merge commits

Ouverte
#284 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
42/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
python
Domaine
backend

Piste de recherche

Commencez par localiser le code du parser qui gère les en-têtes de unified diff, ainsi que l’expression régulière ou le chemin de parsing de l’en-tête de hunk @@@. Comparez son comportement avec l’exemple diff --cc présenté et avec un diff normal, puis ajoutez des tests si le dépôt contient des tests du parser. Le travail est terminé lorsque les diffs de commits de merge sont reconnus sans casser le parsing normal des patches.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
Python
Étoiles
317
Forks
91
Aucune PR mergée en 30 j

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de getpatchwork/patchwork

Toutes les issues de getpatchwork/patchwork

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.