Mercurial > evolve
view tests/test-evolve-issue5832.t @ 5572:d7d954b6d518
evolve: remove swapping nodes logic while resolving divergence
Before this patch, while solving content-divergence if one of the
divergent cset need to relocate to resolution parent we used to set that
cset to "other" side; so it was like we are always going to relocate "other"
side which made us to swap the two if "divergent" cset need to relocate.
Above mentioned solution for relocating cset was not good and led us to incorrect
output in some cases.
So this patch, simplify the logic and remove the swapping thing and clearly depend
on resolution parent to check if any of the divergent cset needs to relocate.
This also means that now in some cases if both the divergent csets needs
to relocate, both will be relocated. Even before this patch, there are some cases
where both the divergent csets relocated but not in every case where it should
have.
Coming to the changes in test files:
1) changes in order of merging the two divergent csets is due to fact that
now "divergent" and "other" are not swapped and hence "divergent" will be at
local side and "other" will be at "other", which also results in change in
the hash id for some cases.
2) extra changesets: which is the result of the fact that now we don't miss
any case where both the divergent csets should be based on resolution parent
before performing the merge -> which caught some cases and performed relocation,
hence extra changesets. Although you can find that there is no changes in file
content because of these relocation, but I remember loosing some data because
of not relocating the "divergent", so why leave any risk out there by not
relocating "other" to the resolution parent.
author | Sushil khanchi <sushilkhanchi97@gmail.com> |
---|---|
date | Sun, 20 Sep 2020 09:22:04 +0530 |
parents | 143ac9ac77a3 |
children | 29bc269bcc96 |
line wrap: on
line source
Test for issue 5832 present at https://bz.mercurial-scm.org/show_bug.cgi?id=5832 ================================================================================ Setup ===== $ cat >> $HGRCPATH <<EOF > [phases] > publish = False > [alias] > glog = log -GT "{rev}:{node|short} {desc}\n ({bookmarks}) {phase}" > [extensions] > EOF $ echo "evolve=$(echo $(dirname $TESTDIR))/hgext3rd/evolve/" >> $HGRCPATH $ hg init issue5832 $ cd issue5832 $ echo base > base $ hg ci -Aqm "added base" $ echo a > a $ hg ci -Aqm "added a" $ echo b > b $ hg ci -Aqm "added b" $ hg up .^^ 0 files updated, 0 files merged, 2 files removed, 0 files unresolved $ echo c > c $ echo d > d $ hg ci -Aqm "added c and d" $ hg merge 2 files updated, 0 files merged, 0 files removed, 0 files unresolved (branch merge, don't forget to commit) $ hg ci -m "merge commit" $ hg glog @ 4:b9b387427a53 merge commit |\ () draft | o 3:9402371b436e added c and d | | () draft o | 2:a1da0651488c added b | | () draft o | 1:1b24879c5c3c added a |/ () draft o 0:bde1d2b6b5e5 added base () draft $ hg up 1b24879c5c3c 0 files updated, 0 files merged, 3 files removed, 0 files unresolved $ echo foo > a $ hg amend 2 new orphan changesets $ hg up bde1d2b6b5e5 0 files updated, 0 files merged, 1 files removed, 0 files unresolved $ echo c > c $ hg ci -Aqm "added c" $ hg up .^ 0 files updated, 0 files merged, 1 files removed, 0 files unresolved $ echo d > d $ hg ci -Aqm "added d" $ hg glog @ 7:5841d7cf9893 added d | () draft | o 6:62fb70414f99 added c |/ () draft | o 5:7014ec2829cd added a |/ () draft | * 4:b9b387427a53 merge commit | |\ () draft +---o 3:9402371b436e added c and d | | () draft | * 2:a1da0651488c added b | | () draft | x 1:1b24879c5c3c added a |/ () draft o 0:bde1d2b6b5e5 added base () draft $ hg prune -r 9402371b436e --succ 62fb70414f99 --succ 5841d7cf9893 --split 1 changesets pruned $ hg glog @ 7:5841d7cf9893 added d | () draft | o 6:62fb70414f99 added c |/ () draft | o 5:7014ec2829cd added a |/ () draft | * 4:b9b387427a53 merge commit | |\ () draft +---x 3:9402371b436e added c and d | | () draft | * 2:a1da0651488c added b | | () draft | x 1:1b24879c5c3c added a |/ () draft o 0:bde1d2b6b5e5 added base () draft Checking what evolve is trying to do $ hg evolve --dry-run --any --all move:[2] added b atop:[5] added a hg rebase -r a1da0651488c -d 7014ec2829cd could not solve instability, ambiguous destination: parent split across two branches Resolving instability using `hg evolve` $ hg evolve --any --all --config ui.interactive=True <<EOF > 1 > EOF move:[2] added b atop:[5] added a move:[4] merge commit ancestor of '7235ef625ea3' split over multiple topological branches. choose an evolve destination: 1: [62fb70414f99] added c 2: [5841d7cf9893] added d q: quit the prompt enter the index of the revision you want to select: 1 move:[9] merge commit atop:[6] added c $ hg glog o 10:28a0775ac832 merge commit |\ () draft | o 8:2baf8bae7ea4 added b | | () draft | | @ 7:5841d7cf9893 added d | | | () draft o---+ 6:62fb70414f99 added c / / () draft o / 5:7014ec2829cd added a |/ () draft o 0:bde1d2b6b5e5 added base () draft $ cd .. Test for issue5833 present at https://bz.mercurial-scm.org/show_bug.cgi?id=5833 =============================================================================== $ hg init issue5833 $ cd issue5833 $ echo base > base $ hg ci -Aqm "added base" $ echo a > a $ hg ci -Aqm "added a" $ echo b > b $ hg ci -Aqm "added b" $ hg up .^^ 0 files updated, 0 files merged, 2 files removed, 0 files unresolved $ echo c > c $ echo d > d $ hg ci -Aqm "added c and d" $ hg merge 2 files updated, 0 files merged, 0 files removed, 0 files unresolved (branch merge, don't forget to commit) $ hg ci -m "merge commit" $ hg glog @ 4:b9b387427a53 merge commit |\ () draft | o 3:9402371b436e added c and d | | () draft o | 2:a1da0651488c added b | | () draft o | 1:1b24879c5c3c added a |/ () draft o 0:bde1d2b6b5e5 added base () draft $ hg log -r 'p1(.)' changeset: 3:9402371b436e parent: 0:bde1d2b6b5e5 user: test date: Thu Jan 01 00:00:00 1970 +0000 summary: added c and d $ hg up bde1d2b6b5e5 0 files updated, 0 files merged, 4 files removed, 0 files unresolved $ echo l > l $ hg ci -Aqm "added l" $ hg pick -r 1b24879c5c3c picking 1:1b24879c5c3c "added a" 2 new orphan changesets $ hg up bde1d2b6b5e5 0 files updated, 0 files merged, 2 files removed, 0 files unresolved $ echo c > c $ hg ci -Aqm "added c" $ hg up .^ 0 files updated, 0 files merged, 1 files removed, 0 files unresolved $ echo d > d $ hg ci -Aqm "added d" $ hg glog @ 8:5841d7cf9893 added d | () draft | o 7:62fb70414f99 added c |/ () draft | o 6:5568b87b1491 added a | | () draft | o 5:0a6281e212fe added l |/ () draft | * 4:b9b387427a53 merge commit | |\ () draft +---o 3:9402371b436e added c and d | | () draft | * 2:a1da0651488c added b | | () draft | x 1:1b24879c5c3c added a |/ () draft o 0:bde1d2b6b5e5 added base () draft $ hg prune -r 9402371b436e --succ 5841d7cf9893 --succ 62fb70414f99 --split 1 changesets pruned $ hg glog @ 8:5841d7cf9893 added d | () draft | o 7:62fb70414f99 added c |/ () draft | o 6:5568b87b1491 added a | | () draft | o 5:0a6281e212fe added l |/ () draft | * 4:b9b387427a53 merge commit | |\ () draft +---x 3:9402371b436e added c and d | | () draft | * 2:a1da0651488c added b | | () draft | x 1:1b24879c5c3c added a |/ () draft o 0:bde1d2b6b5e5 added base () draft $ hg evolve --any --all --dry-run move:[2] added b atop:[6] added a hg rebase -r a1da0651488c -d 5568b87b1491 could not solve instability, ambiguous destination: parent split across two branches $ hg evolve --any --all --config ui.interactive=True <<EOF > 2 > EOF move:[2] added b atop:[6] added a move:[4] merge commit ancestor of 'cdf2ea1b9312' split over multiple topological branches. choose an evolve destination: 1: [62fb70414f99] added c 2: [5841d7cf9893] added d q: quit the prompt enter the index of the revision you want to select: 2 move:[10] merge commit atop:[8] added d $ hg glog o 11:460e6e72b7f9 merge commit |\ () draft | o 9:da76bb7cd904 added b | | () draft @ | 8:5841d7cf9893 added d | | () draft +---o 7:62fb70414f99 added c | | () draft | o 6:5568b87b1491 added a | | () draft | o 5:0a6281e212fe added l |/ () draft o 0:bde1d2b6b5e5 added base () draft $ cd .. Test for issue5946 present at https://bz.mercurial-scm.org/show_bug.cgi?id=5946 =============================================================================== issue with computing dependency with split and merge $ hg init issue5946 $ cd issue5946 $ echo base > base $ hg ci -Aqm "added base" $ echo a > a $ hg ci -Aqm "added a" $ echo b > b $ hg ci -Aqm "added b" $ hg up .^^ 0 files updated, 0 files merged, 2 files removed, 0 files unresolved $ echo c > c $ echo d > d $ hg ci -Aqm "added c and d" The next line is the only difference from issue5833 above, i.e. the order of the parents is reversed $ hg co 2 2 files updated, 0 files merged, 2 files removed, 0 files unresolved $ hg merge 2 files updated, 0 files merged, 0 files removed, 0 files unresolved (branch merge, don't forget to commit) $ hg ci -m "merge commit" $ hg glog @ 4:b9b387427a53 merge commit |\ () draft | o 3:9402371b436e added c and d | | () draft o | 2:a1da0651488c added b | | () draft o | 1:1b24879c5c3c added a |/ () draft o 0:bde1d2b6b5e5 added base () draft $ hg log -r 'p1(.)' changeset: 2:a1da0651488c user: test date: Thu Jan 01 00:00:00 1970 +0000 summary: added b $ hg up bde1d2b6b5e5 0 files updated, 0 files merged, 4 files removed, 0 files unresolved $ echo l > l $ hg ci -Aqm "added l" $ hg pick -r 1b24879c5c3c picking 1:1b24879c5c3c "added a" 2 new orphan changesets $ hg up bde1d2b6b5e5 0 files updated, 0 files merged, 2 files removed, 0 files unresolved $ echo c > c $ hg ci -Aqm "added c" $ hg up .^ 0 files updated, 0 files merged, 1 files removed, 0 files unresolved $ echo d > d $ hg ci -Aqm "added d" $ hg glog @ 8:5841d7cf9893 added d | () draft | o 7:62fb70414f99 added c |/ () draft | o 6:5568b87b1491 added a | | () draft | o 5:0a6281e212fe added l |/ () draft | * 4:b9b387427a53 merge commit | |\ () draft +---o 3:9402371b436e added c and d | | () draft | * 2:a1da0651488c added b | | () draft | x 1:1b24879c5c3c added a |/ () draft o 0:bde1d2b6b5e5 added base () draft $ hg prune -r 9402371b436e --succ 5841d7cf9893 --succ 62fb70414f99 --split 1 changesets pruned $ hg glog @ 8:5841d7cf9893 added d | () draft | o 7:62fb70414f99 added c |/ () draft | o 6:5568b87b1491 added a | | () draft | o 5:0a6281e212fe added l |/ () draft | * 4:b9b387427a53 merge commit | |\ () draft +---x 3:9402371b436e added c and d | | () draft | * 2:a1da0651488c added b | | () draft | x 1:1b24879c5c3c added a |/ () draft o 0:bde1d2b6b5e5 added base () draft $ hg evolve --any --all --dry-run move:[2] added b atop:[6] added a hg rebase -r a1da0651488c -d 5568b87b1491 could not solve instability, ambiguous destination: parent split across two branches $ hg evolve --any --all --config ui.interactive=True <<EOF > 2 > EOF move:[2] added b atop:[6] added a ancestor of 'b9b387427a53' split over multiple topological branches. choose an evolve destination: 1: [62fb70414f99] added c 2: [5841d7cf9893] added d q: quit the prompt enter the index of the revision you want to select: 2 move:[4] merge commit atop:[8] added d move:[10] merge commit atop:[9] added b $ hg glog o 11:578c938ebd2e merge commit |\ () draft | o 9:da76bb7cd904 added b | | () draft @ | 8:5841d7cf9893 added d | | () draft +---o 7:62fb70414f99 added c | | () draft | o 6:5568b87b1491 added a | | () draft | o 5:0a6281e212fe added l |/ () draft o 0:bde1d2b6b5e5 added base () draft