parser doesn't work with diffs from git merge commits
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 42/100
Rechercherichtung
Beginne damit, den Parser-Code zu finden, der Unified-Diff-Header verarbeitet, sowie den regulären Ausdruck oder Parsing-Pfad für den @@@-Hunk-Header. Vergleiche sein Verhalten beim gezeigten diff --cc-Beispiel und bei einem normalen Diff und füge Tests hinzu, falls das Repository Parser-Tests enthält. Als abgeschlossen gilt die Aufgabe, wenn Diffs von Merge-Commits erkannt werden, ohne das Parsen normaler Patches zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
@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.
- Vorherrschende Sprache
- Python
- Sterne
- 317
- Forks
- 91
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus getpatchwork/patchwork
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
getpatchwork/patchwork#546 · 2 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 52/100
getpatchwork/patchwork#664 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
getpatchwork/patchwork#663 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
getpatchwork/patchwork#645 ·
Alle Issues in getpatchwork/patchwork
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 90/100
learningequality/ricecooker#747 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
BSData/horus-heresy-3rd-edition#3171 ·
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
run-llama/llama_index#23199 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
KhronosGroup/glTF-Blender-IO#2769 ·