D9513: phab-refresh: do not error out when the stack is empty

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Actually, empty stack is easier to get than I expected (and harmless). When a stack is publish, heptapod will detect the even, empty the stack (triggering CI)

D9512: formatting: re-blacken match.py

2020-12-02 Thread mharbison72 (Matt Harbison)
mharbison72 created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9512 AFFECTED FILES mercurial/match.py CHANGE DETAILS diff --git

D9511: statprof: separate functions and "line", assume 4 digit line numbers

2020-12-02 Thread spectral (Kyle Lippincott)
spectral created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Previously, the profile output looked like this (I've removed many lines that are mostly inconsequential): | 100.0% 0.02s hg:

D9510: statprof: fix off-by-one-line error in output

2020-12-02 Thread spectral (Kyle Lippincott)
spectral created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY martinvonz claims they thought that this was intentional, but couldn't remember the reasoning for it. I can't understand why it would be preferable, and I

D9509: rebase: clear merge state when aborting in-memory merge on dirty working copy

2020-12-02 Thread martinvonz (Martin von Zweigbergk)
martinvonz created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REPOSITORY rHG Mercurial BRANCH stable REVISION DETAIL https://phab.mercurial-scm.org/D9509 AFFECTED FILES hgext/rebase.py tests/test-rebase-inmemory.t CHANGE

D9508: tests: show that in-memory rebase leaves state when working copy is dirty

2020-12-02 Thread martinvonz (Martin von Zweigbergk)
martinvonz created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY When in-memory rebase falls back to on-disk rebase, it checks if the working copy is dirty. If it is, it aborts the rebase. However, it leaves the rebase

D9507: run-tests: allow some slack about 'waiting on lock' message

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY It is common to run the tests on very loaded machine when concurrent run might take a bit longer. Such message are usually harmless, but anoying as they break

D9506: pull: flush stdin after the `pull from` message

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY That message can end up being flushed after some stderr message in some case, leading to confusing output. REPOSITORY rHG Mercurial BRANCH default

D9504: tests: use the right python when running dummyssh for narrow

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: durin42. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY some plateform no longer have a `python` executable. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9505: tests: explicitly skip the lock warning in some remotefilelog tests

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The output was conditional zed, so lets official skip it instead. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

mercurial@46008: 14 new changesets

2020-12-02 Thread Mercurial Commits
14 new changesets in mercurial: https://www.mercurial-scm.org/repo/hg/rev/32dcd783a985 changeset: 45995:32dcd783a985 user:Pierre-Yves David date:Tue Dec 01 02:07:15 2020 +0100 summary: upgrade: move optimisation to something more declarative

Re: Test notifications | Pipeline #358 has failed for branch/default | c086d56d

2020-12-02 Thread Georges Racinet
Hi, That was me testing that upgrade tomorrow should really manage to notify the mailing-list upon build failures. Cheers, -- Georges Racinet https://octobus.net, https://about.heptapod.host, https://heptapod.net GPG: BF5456F4DC625443849B6E58EE20CA44EF691D39, sur serveurs publics

Test notifications | Pipeline #358 has failed for branch/default | c086d56d

2020-12-02 Thread Test please Ignore
Your pipeline has failed. Project: Test notifications ( http://localhost:3000/root/test-notifications ) Branch: branch/default ( http://localhost:3000/root/test-notifications/-/commits/branch/default ) Commit: c086d56d (

mercurial@45994: 7 new changesets

2020-12-02 Thread Mercurial Commits
7 new changesets in mercurial: https://www.mercurial-scm.org/repo/hg/rev/c6da62004c0f changeset: 45988:c6da62004c0f user:Pulkit Goyal <7895pul...@gmail.com> date:Fri Nov 27 21:32:42 2020 +0530 summary: tests: update test-chg.t with output change

D9503: copies: avoid materializing a full directory map during copy tracing

2020-12-02 Thread spectral (Kyle Lippincott)
spectral created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Materializing a full copy of every directory in a treemanifest repo can be quite expensive, even with a narrow matcher. For flat manifest repos, this should

Re: [PATCH STABLE] diff: do not concatenate immutable bytes while building a/b bodies (issue6445)

2020-12-02 Thread Augie Fackler
queued for stable, thanks > On Nov 27, 2020, at 22:00, Yuya Nishihara wrote: > > # HG changeset patch > # User Yuya Nishihara > # Date 1606529754 -32400 > # Sat Nov 28 11:15:54 2020 +0900 > # Branch stable > # Node ID 1e93d856abc0ecb7f537af5c77e0d75ffe93db08 > # Parent

D9502: tests: conditionalize output in test-ssh.t with chg+py3

2020-12-02 Thread pulkit (Pulkit Goyal)
pulkit created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Because of our wrapping around sys.std* and python3 internal buffering, the output order changes. The change in order seems like harmless because just few

D9501: dispatch: disable line ending normalization on sys.stdin if its None

2020-12-02 Thread pulkit (Pulkit Goyal)
pulkit created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Fixes test-chg.t on python 3 with chg. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D9501 AFFECTED FILES mercurial/dispatch.py

D9500: procutils: don't try to get `.buffer` if sys.stdin is None

2020-12-02 Thread pulkit (Pulkit Goyal)
pulkit created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY While hunting down following test failure of test-chg.t on Python 3, I stumbled the case when `.buffer` is not available as sys.stdin is None. -

mercurial-devel | Pipeline #13739 has failed for branch/default | 86d3fcf1

2020-12-02 Thread Heptapod
Your pipeline has failed. Project: mercurial-devel ( https://foss.heptapod.net/octobus/mercurial-devel ) Branch: branch/default ( https://foss.heptapod.net/octobus/mercurial-devel/-/commits/branch/default ) Commit: 86d3fcf1 (

mercurial-devel | Pipeline #13735 has failed for branch/default | 6d85b8a8

2020-12-02 Thread Heptapod
Your pipeline has failed. Project: mercurial-devel ( https://foss.heptapod.net/octobus/mercurial-devel ) Branch: branch/default ( https://foss.heptapod.net/octobus/mercurial-devel/-/commits/branch/default ) Commit: 6d85b8a8 (

D9498: copies-rust: make the comparison aware of the revision being current merged

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This make no significant performance change in practice (all ±2%) in practice, but it will help us to distinct between some semantically different cases later

D9499: rust-copies: record overwrite when merging

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY When detecting fresh value (from current rev) overwriting older ones during merges, we also record this overwrite for to help potential future comparison.

D9497: copies-rust: start recording overwrite as they happens

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY If a revision has information overwriting data from another revision, the overwriting revision is a descendant of the overwritten one. So we could warm the

D9496: copies-rust: rename Oracle.is_ancestor to Oracle.is_overwrite

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The core information that we want here is about "does information from revision X overwrite information in Y". To do so, we check is X is an ancestors of Y,

D9495: copies-rust: use the `entry` API for copy information too

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The code end up being complicated, but it split out the case were we add a new entry from the case were we overwrite one. And that is the goal here, being

D9494: copies-rust: use the entry API to overwrite deleted entry

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This is more efficient, more idiomatic and more compact. The main motivation for this change is to cleanup that area before start to do "overwrite"

D9493: copies-rust: tokenize all paths into integer

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Copy information for each changesets tend to affect a small new number of path. However, each of these path might be handled a large number of time. Handling

D9492: copies-rust: pre-introduce a PathToken type and use it where applicable

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Handling string all the time is quite slow. So we are about to use more efficient "token". We do some of the churn early to clarify the next changesets.

D9491: copies-rust: add smarter approach for merging small mapping with large mapping

2020-12-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The current approach (finding the smaller updated set) works great when the mapping have similar size, but do a lot of unnecessary work when one side is

D9490: rust: use NodePrefix::from_hex instead of hex::decode directly

2020-12-02 Thread SimonSapin
SimonSapin created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This adds support for prefixes with an odd number of hex digits. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL