evolve: always create commit when resolving divergence When resolving content-divergence, the final commit we create may end up empty (which means that Mercurial won't even create it). We've had code for handling that in evolve ever since 41bf6c27a122 (evolve: stabilize now handle conflicting changeset, 2012-08-23). However, that resolved the issue by marking on the divergent commits as successor. As Pierre-Yves has pointed out (in other code reviews), we should instead be creating a new successor. So that's what this patch does. It does that by setting `ui.allowemptycommit` while creating the final commit. However, that is not enough, because we may end up creating the same nodeid as already existed (we'd then end up trying to mark the "new" commit a successor of itself). To solve that, we add some salt to the commit extras. That salt affects lots of tests.

tests: replace an X+X revset with something meaningful The change in aab2afc2bd82 seem to have been make too hastily.

tests: update section headers and comments in obslog-related tests

tests: use common obshistory_setup.sh script instead of manual setup This is now consistent with other test-evolve-obshistory*.t tests.

tests: remove "second diverged revision" from a phase divergence test It was basically testing the same thing twice with a different comment.

obshistory: omit duplicate notes of obsolescence operations When folding changesets, it's possible to provide a --note flag to e.g. fold or prune, which will store a note for that operation. But since there would be more than one marker for the fold operation and the provided note would be stored in all of them, _markersnotes() previously would return the same text message many times. We don't really want to see the same thing over and over, so let's make notes a set(). This will also make unrelated operations with the same note show the message only once, which is probably an improvement too.

changelog: adjust The changes only apply to newly created sqlite database. There are code pathes that drop existing ones, but that's a more complicated condition.

changelog: mention the sqlite fix

evolve: extract function for relocating divergent commit There was some duplication here, so let's extract to a function. I'm going to add a little bit more shared code next.

caches: preserve permissions of top-level .hg When using hg on a shared filesystem on UNIX, cache files normally inherit the permissions of the .hg directory. This is most commonly used to ensure everything is writable by all users. The sqlite3 cache files don't have a way to directly set the permission, so check if a special mode is necessary and try to apply them to newly created database files.

changelog: mention obsorigin template keyword

templatekw: add obsorigin keyword

obshistory: add predecessorsandmarkers()

obshistory: add a function to group predsets by fold-id

obshistory: add _predecessorsverb() similar to _successorsetverb()

obshistory: add a docstring to patchavailable()

tests: avoid referring to commits by hash in a few places This patch means that the next patch can be written without manual editing of the test cases.

tests: add more verification of output to divergence tests It's not obvious what the obsmarkers should look like in these tests and adding them highlights a bug that happens when commit we create to resolve the divergence doesn't get created because it would be empty. I also added a few graphlog calls to check that the divergence-resolution commit is in the right place (has right parentage).

tests: add debugobsolete calls before many obslog calls As requested by Pierre-Yves. The main reason is to catch superfluous creation of obsmarkers, as well as any future changes to how the obsmarkers get created.

stablerangecache: sanity check subranges Try to detect invalid conditions on insert as would result in deep recursions and final aborts much later. This has been observed on two different machines and the check makes it hopefully possible to find the origin of the problem.

obslog: use unfiltered repo for --no-graph implementation too repo.unfiltered() is used for graph version of obslog, because if we want to filter non-local changesets, we only have to provide --filternonlocal flag on the command line and _nodesandmarkers() will handle filtering. But that means for obslog without -f to work correctly we need to provide unfiltered repo to that function. With this patch both implementations of obslog (--graph and --no-graph) see the same repo.

tests: don't assume that echo will unescape \\

tests: avoid non-portable sed command sed -i is a GNU extension and the i command without newline as well. Just create the file from scratch with printf to side-step the issue.

tests: check for file existance in a portable way The precise output of ls is highly OS specific, so just use the simpler test command.

tests: deal with "ls" vs "ls -A" difference on 2BSD derived systems ls on 2BSD derived systems historically defaults to -A for root, so use it unconditionally for consistent results.

tests: silence potential warnings about uname -o The option is not supported on the BSDs, but also not relevant as it is only used to detect msys.

evolve: remove unused divergenceflag and splitflag of MultipleSuccessorsError

evolve: consider all obsolete ancestors when finding new parent for orphan Both the old and the new logic have in common that the orphan changeset is moved to the tipmost successor of one or more ancestors of the orphan changeset. The previous logic considered only the first non-pruned ancestor of the orphan changeset. The new logic considers all obsolete ancestors of the orphan changeset. The logic in _possibledestination() had to be changed to be consistent with the new logic.

py3: fix debugstablerangecache

topic: provide cache to successorssets() in _singlesuccessor() Since we're using obsutil.successorssets() in a loop, it may help performance to have cache.

evolve: provide cache to successorssets() in _singlesuccessor() Since we're using obsutil.successorssets() in a loop, it may help performance to have cache.

evolve: provide cache to successorssets() in _possibledestination() Since we're using obsutil.successorssets() in a loop, it may help performance to have cache.

obslog: support --all with --no-graph

obslog: support --filternonlocal with --no-graph

obshistory: restructure templates a bit, drop default verb ("pruned") We always have a verb provided to the templates now, this should be fine.

obshistory: prepare data for displaymarkers() in a separate function

obshistory: make patchavailable() handle predecessors too

tests: make sure not to test the same thing twice A block under this one already tests obslog --no-graph with -f. Right now there's no difference in output, but maybe some day obslog --no-graph will actually support --filternonlocal.

obshistory: rename precursors to predecessors in displayrevs()

obshistory: rename _debugobshistorydisplaymarker() to displaymarkers() Apart from the obvious change, the function now can show multiple markers, not just one.

obshistory: rename graph/revs-displaying functions Same idea as the previous patch, but with a "display" prefix.

obshistory: rename node/ctx-displaying functions Not just for debug, and yes we know it's obshistory-related.

branching: merge with stable

topic: use generationssubrel less directly, because it was recently renamed

auto-publish: issue the capabilities in all cases It seems better to message client that we support the capabilities in all cases.

packaging: add an entry to debian/changelog mentioning the previous change

packaging: bump minimum required version of mercurial in debian/control

[debian] package 9.3.1-3 for bionic ubuntu

tests: update output to match 1b8fd4af3318 in core CORE-TEST-OUTPUT-UPDATE: 1b8fd4af3318

manifest: now that there's only one legacy.py, exclude it After the last patch, this file is only included because of package_dir in setup.py (i.e. just once), so we can ignore it with just one clause. But package_dir starts with "./", so we need to add it to the pattern.

manifest: don't include .py files from hgext3rd/ twice They are already included by default because we've given package_dir to setup().

manifest: just include all .py .sh and .t files from tests/ recursively

manifest: make sure exclusions and prunes are last, because order matters

branching: merge 9.3.1 release into default

packaging: mark as developer version

Added tag 9.3.1 for changeset 8d955635cf45

packaging: prepare version 9.3.1

test-compat: merge mercurial-4.7 into mercurial-4.6

test-compat: merge mercurial-4.8 into mercurial-4.7

test-compat: merge mercurial-4.9 into mercurial-4.8