parser doesn't work with diffs from git merge commits
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 42/100
Hướng nghiên cứu
Bắt đầu bằng cách xác định mã parser xử lý các header của unified diff và regex hoặc đường dẫn parsing cho header hunk @@@. So sánh hành vi của nó với ví dụ diff --cc đã nêu và với một diff thông thường, sau đó bổ sung coverage nếu repository có các bài test cho parser. Hoàn tất khi các diff của merge commit được nhận diện mà không làm hỏng việc parsing patch thông thường.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
@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.
- Ngôn ngữ chính
- Python
- Star
- 317
- Fork
- 91
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của getpatchwork/patchwork
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
getpatchwork/patchwork#546 · 2 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 52/100
getpatchwork/patchwork#664 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
getpatchwork/patchwork#663 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
getpatchwork/patchwork#645 ·
Tất cả issue của getpatchwork/patchwork
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
learningequality/ricecooker#747 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
BSData/horus-heresy-3rd-edition#3171 ·
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
run-llama/llama_index#23199 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
KhronosGroup/glTF-Blender-IO#2769 ·