changelog merge hook should use a 3-way merge
Bug #516060 reported by
John A Meinel
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
bzr-builddeb |
Fix Released
|
Wishlist
|
John A Meinel |
Bug Description
Currently the debian/changelog merge hook only parses THIS and OTHER to compute the merge. It should also look at BASE. This would enable a few things.
1) It should conflict if THIS and OTHER both change the same section. (Or possibly try to do a regular 3-way merge on the internal lines?)
2) It should be able to decide whether THIS or OTHER simply updated a section, and thus just use that section.
3) We may want to change the code so that for sections where THIS == OTHER == BASE we don't parse them at all. (It may be tricky to find the sections, though.)
Related branches
lp:~jameinel/bzr-builddeb/changelog-parser
Merged
into
lp:bzr-builddeb
- Robert Collins (community): Approve
-
Diff: 609 lines (+271/-269)2 files modifiedmerge_changelog.py (+104/-222)
tests/test_merge_changelog.py (+167/-47)
Changed in bzr-builddeb: | |
status: | New → Triaged |
importance: | Undecided → Wishlist |
Changed in bzr-builddeb: | |
assignee: | nobody → John A Meinel (jameinel) |
status: | Triaged → In Progress |
To post a comment you must log in.
I think this is fixed; further bug reports very welcome.