This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	SimonSapin	D9686 copies-rust: send PyBytes values back be dropped ino the parent thread
Closed	SimonSapin	D9685 copies-rust: introduce PyBytesWithData to reduce GIL requirement
Closed	SimonSapin	D9684 copies-rust: move CPU-heavy Rust processing into a child thread
Closed	SimonSapin	D9683 copies-rust: split up combine_changeset_copies function into a struct
Closed	SimonSapin	D9682 copies-rust: extract generic map merge logic from merge_copies_dict
Closed	marmoute	D9656 copies-rust: use imrs::OrdSet instead of imrs::HashSet
Closed	marmoute	D9655 copies-rust: use simpler overwrite when value on both side are identical
Closed	marmoute	D9654 copies-rust: make more use of the new comparison property
Closed	marmoute	D9653 copies-rust: implement PartialEqual manually
Closed	marmoute	D9652 copies-rust: record "overwritten" information from both side on delete
Closed	marmoute	D9651 copies-rust: refactor the "deletion" case
Closed	marmoute	D9650 copies-rust: process copy information of both parent at the same time
Closed	marmoute	D9649 copies-rust: yield both p1 and p2 copies in `ChangedFiles.actions()`
Closed	marmoute	D9648 copies-rust: extract the processing of a single copy information
Closed	marmoute	D9647 copies-rust: use matching to select the final copies information
Closed	marmoute	D9646 copies-rust: get the parents' copies earlier
Closed	marmoute	D9645 copies-rust: remove the ancestor Oracle logic
Closed	marmoute	D9644 copies-rust: track "overwrites" directly within CopySource
Closed	marmoute	D9643 copies-rust: add methods to build and update CopySource
Closed	marmoute	D9657 copies-rust: fix reverted argument when merging tiny minor or major
Closed	marmoute	D9642 copies-rust: rename TimeStampedPathCopy to CopySource
Closed	marmoute	D9641 copies-rust: rename TimeStampedPathCopies to InternalPathCopies
Closed	marmoute	D9613 copies: detect case when a merge decision overwrite previous data
Closed	marmoute	D9612 copies: rearrange all value comparison conditional
Closed	marmoute	D10059 test-copies: introduce merge chains test for the P/Q merges
Closed	marmoute	D10058 test-copies: add a case involving the `b` and a new `r` branch
Closed	marmoute	D10057 test-copies: introduce case combining the `p` and `q` branch
Closed	marmoute	D10056 test-copies: add a `q` branch similar to the `e` but on the new files
Closed	marmoute	D10055 test-copies: add a `p` branch similar to the `a` but on the new files
Closed	marmoute	D10054 test-copies: move the new files in the `i` branch
Closed	marmoute	D10053 test-copies: add 3 new files with their own content
Closed	marmoute	D10052 test-copies: introduce merge chaing test for the A/E + change tests
Closed	marmoute	D10051 test-copies: add a "change during merge" variant to the A+E test
Closed	marmoute	D10050 test-copies: filter out the linkrev part of `debugindex`
Closed	marmoute	D10049 test-copies: use "case-id" instead of revision number when listing sidedata
Closed	marmoute	D10048 test-copies: remove revision number from log
Closed	marmoute	D9611 test-copies: add test chaining multiple merge
Closed	marmoute	D9610 test-copies: add test chaining multiple merges
Closed	marmoute	D9609 test-copies: add test chaining multiple merges
Closed	marmoute	D10047 test-copies: add subcase titles for various "conflicting" information variant
Closed	marmoute	D10046 test-copies: improve description of the B+F case
Closed	marmoute	D10045 test-copies: improve description of the C+H case
Closed	marmoute	D10044 test-copies: improve description of the B+C "revert/restore" case
Closed	marmoute	D10043 test-copies: improve description of the G+C case
Closed	marmoute	D10042 test-copies: improve description of the G+F case
Closed	marmoute	D10041 test-copies: improve description of the D+G case
Closed	marmoute	D10040 test-copies: improve description of the A+E case
Closed	marmoute	D10039 test-copies: improve description of the B+D case
Closed	marmoute	D10038 test-copies: improve description of the B+C case
Closed	marmoute	D10037 test-copies: improve description of the A+B case
Closed	marmoute	D10036 test-copies: use intermediate variable some commit descriptions
Closed	marmoute	D10035 test-copies: don't use empty file for "same content" cases
Closed	marmoute	D9591 copies: rename value/other variable to minor/major for clarity
Closed	marmoute	D9590 copies: extract value comparison in the python copy tracing
Closed	marmoute	D9607 hghave: add some official category for known-bad and missing-good output
Closed	marmoute	D9608 copies: stop attempt to avoid extra dict copies around branching
Closed	marmoute	D9592 copies: deal with the "same revision" special case earlier
Closed	marmoute	D9589 copies-tests: update to null in test-copies-chain-merge.t
Closed	marmoute	D9588 copies-tests: add a summary of all cases created in test-copies-chain-merge.t
Closed	marmoute	D9587 test-copies: reinstall initial identical (empty) files for chained copied
Closed	marmoute	D9586 copies: explain the "arbitrary" copy source pick in case of conflict
Closed	marmoute	D9585 copies: properly match result during changeset centric copy tracing
Closed	marmoute	D9584 copies: avoid early return in _combine_changeset_copies
Closed	marmoute	D9499 copies-rust: record overwrite when merging
Closed	marmoute	D9498 copies-rust: make the comparison aware of the revision being current merged
Closed	marmoute	D9497 copies-rust: start recording overwrite as they happens
Closed	marmoute	D9496 copies-rust: rename Oracle.is_ancestor to Oracle.is_overwrite
Closed	marmoute	D9495 copies-rust: use the `entry` API for copy information too
Closed	marmoute	D9494 copies-rust: use the entry API to overwrite deleted entry
Closed	marmoute	D9493 copies-rust: tokenize all paths into integer
Closed	marmoute	D9492 copies-rust: pre-introduce a PathToken type and use it where applicable
Closed	marmoute	D9491 copies-rust: add smarter approach for merging small mapping with large mapping
Closed	marmoute	D9426 copies-rust: hide most of the comparison details inside a closure
Closed	marmoute	D9425 copies-rust: move the mapping merging into a else clause
Closed	marmoute	D9424 copies-rust: extract conflicting value comparison in its own function
Closed	marmoute	D9423 copies: no longer cache the ChangedFiles during copy tracing
Closed	marmoute	D9422 copies: iterate over children directly (instead of parents)
Closed	marmoute	D9581 copies: document the current algorithm step

Diff 25816

tests/test-copies-chain-merge.t

	information.			information.

	merging with unrelated change does not interfere with the renames			merging with unrelated change does not interfere with the renames
	---------------------------------------------------------------			---------------------------------------------------------------

	- rename on one side			- rename on one side
	- unrelated change on the other side			- unrelated change on the other side

	$ case_desc="simple merge"			$ case_desc="simple merge - A side: multiple renames, B side: unrelated update"

	$ hg up 'desc("b-1")'			$ hg up 'desc("b-1")'
	1 files updated, 0 files merged, 1 files removed, 0 files unresolved			1 files updated, 0 files merged, 1 files removed, 0 files unresolved
	$ hg merge 'desc("a-2")'			$ hg merge 'desc("a-2")'
	1 files updated, 0 files merged, 1 files removed, 0 files unresolved			1 files updated, 0 files merged, 1 files removed, 0 files unresolved
	(branch merge, don't forget to commit)			(branch merge, don't forget to commit)
	$ hg ci -m "mBAm-0 $case_desc - one way"			$ hg ci -m "mBAm-0 $case_desc - one way"
	$ hg up 'desc("a-2")'			$ hg up 'desc("a-2")'
	1 files updated, 0 files merged, 0 files removed, 0 files unresolved			1 files updated, 0 files merged, 0 files removed, 0 files unresolved
	$ hg merge 'desc("b-1")'			$ hg merge 'desc("b-1")'
	1 files updated, 0 files merged, 0 files removed, 0 files unresolved			1 files updated, 0 files merged, 0 files removed, 0 files unresolved
	(branch merge, don't forget to commit)			(branch merge, don't forget to commit)
	$ hg ci -m "mABm-0 $case_desc - the other way"			$ hg ci -m "mABm-0 $case_desc - the other way"
	created new head			created new head
	$ hg log -G --rev '::(desc("mABm")+desc("mBAm"))'			$ hg log -G --rev '::(desc("mABm")+desc("mBAm"))'
	@ 12 mABm-0 simple merge - the other way			@ 12 mABm-0 simple merge - A side: multiple renames, B side: unrelated update - the other way
	\|\			\|\
	+---o 11 mBAm-0 simple merge - one way			+---o 11 mBAm-0 simple merge - A side: multiple renames, B side: unrelated update - one way
	\| \|/			\| \|/
	\| o 5 b-1: b update			\| o 5 b-1: b update
	\| \|			\| \|
	o \| 4 a-2: e -move-> f			o \| 4 a-2: e -move-> f
	\| \|			\| \|
	o \| 3 a-1: d -move-> e			o \| 3 a-1: d -move-> e
	\|/			\|/
	o 2 i-2: c -move-> d			o 2 i-2: c -move-> d
	e-2 g -move-> f			e-2 g -move-> f
	f-1: rename h -> i			f-1: rename h -> i
	f-2: rename i -> d			f-2: rename i -> d
	g-1: update d			g-1: update d
	h-1: b -(move)-> d			h-1: b -(move)-> d
	i-0 initial commit: a b h			i-0 initial commit: a b h
	i-1: a -move-> c			i-1: a -move-> c
	i-2: c -move-> d			i-2: c -move-> d
	mABm-0 simple merge - the other way			mABm-0 simple merge - A side: multiple renames, B side: unrelated update - the other way
	mAEm-0 simple merge - one way			mAEm-0 simple merge - one way
	mBAm-0 simple merge - one way			mBAm-0 simple merge - A side: multiple renames, B side: unrelated update - one way
	mBC-revert-m-0			mBC-revert-m-0
	mBCm-0 simple merge - one way			mBCm-0 simple merge - one way
	mBCm-1 re-add d			mBCm-1 re-add d
	mBDm-0 simple merge - one way			mBDm-0 simple merge - one way
	mBFm-0 simple merge - one way			mBFm-0 simple merge - one way
	mCB-revert-m-0			mCB-revert-m-0
	mCBm-0 simple merge - the other way			mCBm-0 simple merge - the other way
	mCBm-1 re-add d			mCBm-1 re-add d

	merging with unrelated change does not interfere with the renames			merging with unrelated change does not interfere with the renames
	---------------------------------------------------------------			---------------------------------------------------------------

	- rename on one side			- rename on one side
	- unrelated change on the other side			- unrelated change on the other side

	$ hg log -G --rev '::(desc("mABm")+desc("mBAm"))'			$ hg log -G --rev '::(desc("mABm")+desc("mBAm"))'
	o 12 mABm-0 simple merge - the other way			o 12 mABm-0 simple merge - A side: multiple renames, B side: unrelated update - the other way
	\|\			\|\
	+---o 11 mBAm-0 simple merge - one way			+---o 11 mBAm-0 simple merge - A side: multiple renames, B side: unrelated update - one way
	\| \|/			\| \|/
	\| o 5 b-1: b update			\| o 5 b-1: b update
	\| \|			\| \|
	o \| 4 a-2: e -move-> f			o \| 4 a-2: e -move-> f
	\| \|			\| \|
	o \| 3 a-1: d -move-> e			o \| 3 a-1: d -move-> e
	\|/			\|/
	o 2 i-2: c -move-> d			o 2 i-2: c -move-> d

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	25710		Feb 22 2021, 9:33 AM	★	★
Diff 2	25816	rHGe0ca928a3bf8be57c77b45357c83c4e94d6135d6	Feb 15 2021, 11:26 PM	★	★

This is an archive of the discontinued Mercurial Phabricator instance.

test-copies: improve description of the A+B caseClosedPublic

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 25816

tests/test-copies-chain-merge.t

test-copies: improve description of the A+B case
ClosedPublic

Revision Contents
Changeset List