parser doesn't work with diffs from git merge commits
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 42/100
Línea de trabajo
Comienza localizando el código del parser que gestiona las cabeceras de unified diff y la expresión regular o ruta de parsing para la cabecera de hunk @@@. Compara su comportamiento con el ejemplo mostrado de diff --cc y con un diff normal, y añade cobertura si el repositorio tiene pruebas del parser. Se considera terminado cuando se reconocen los diffs de commits de merge sin romper el parsing normal de parches.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
@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.
- Lenguaje dominante
- Python
- Estrellas
- 317
- Forks
- 91
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de getpatchwork/patchwork
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
getpatchwork/patchwork#546 · 2 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
getpatchwork/patchwork#664 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
getpatchwork/patchwork#663 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
getpatchwork/patchwork#645 ·
Todos los issues de getpatchwork/patchwork
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
learningequality/ricecooker#747 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
BSData/horus-heresy-3rd-edition#3171 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
run-llama/llama_index#23199 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
KhronosGroup/glTF-Blender-IO#2769 ·