D10153: test: update expected output in test-http.t

2021-03-10 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 introduced in a4c19a162615 and is wrong and unstable. So we glob

D10152: remotefilelog: remove unused import

2021-03-10 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 no longer used since a4c19a162615 . REPOSITORY rHG Mercurial BRANCH

D10121: releasenotes: use the right APO to access the 'sections'

2021-03-05 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Preventing direct access to the underlying dict fix a breakage introduced by the refactoring in d3df397e7a59

D10088: test-copies: add a test updating file content while salvaging it

2021-03-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY A deleted file is brought back during a merge. Its content is changed in the same go. This reveal some issue with the upgrade code. REPOSITORY rHG

D10087: test-copies: add a test updating file content while merging a pure overwrite

2021-03-02 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY As the current test misbehavior show, this confuse various part of the code: - the committed file is different depending of the commit source, - the

D10084: copies: tests and fix parallel computation of changed file information

2021-03-01 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 was mixing variable name and misbehaving in some case. This changeset fix it and also add a tests to validate it does not regress. Without the fix,

D10064: re2: byteify some regex used to get check re2 availability

2021-02-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Changeset 687b865b95ad failed to properly byteify this because it was a raw

D10059: test-copies: introduce merge chains test for the P/Q merges

2021-02-22 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 similar to the chaining test we have for the A/E merges. The current result of this tests is wrong, as for the other test of the same familly.

D10058: test-copies: add a case involving the `b` and a new `r` branch

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY That case involve a branch overwriting copies information from the other one. It is similar to the `mBFm` / `mFBm` case except this case actual content merge

D10057: test-copies: introduce case combining the `p` and `q` branch

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY That case involves conflicting copies information from each branch. It is similar to the `mAEm` / `mEAm` case except this case actual content merge is

D10056: test-copies: add a `q` branch similar to the `e` but on the new files

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This branch will be used to create merge case that mirror ones involving the `e` branch. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D10055: test-copies: add a `p` branch similar to the `a` but on the new files

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This branch will be used to create merge case that mirror ones involving the `a` branch. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D10054: test-copies: move the new files in the `i` branch

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The initial branch is moving some of the "same content" file around. We do the same for the "different content" before start to implement more tests. The

D10053: test-copies: add 3 new files with their own content

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The new `p`, `q` and `r` file mirror the `a`, `b` and `h` but with different content for each files. This will be used to create "mirror" test case that

D10052: test-copies: introduce merge chaing test for the A/E + change tests

2021-02-22 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 similar to the chaining test we have for the A/E merges (without change). The current result of this tests is wrong, as for they "without change

D10051: test-copies: add a "change during merge" variant to the A+E test

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Checks inline comment for details. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10051 AFFECTED FILES

D10050: test-copies: filter out the linkrev part of `debugindex`

2021-02-22 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 not really useful for the tests and is a large source of churn when adding more revisions to the tests. REPOSITORY rHG Mercurial BRANCH default

D10049: test-copies: use "case-id" instead of revision number when listing sidedata

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The revision number are not very informative and can change when new changeset are added, creating unnecessary churn. We change the template before adding

D10048: test-copies: remove revision number from log

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute 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/D10048 AFFECTED FILES tests/test-copies-chain-merge.t CHANGE DETAILS diff --git

D10047: test-copies: add subcase titles for various "conflicting" information variant

2021-02-22 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 the transitions between case clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10047 AFFECTED

D10046: test-copies: improve description of the B+F case

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10046 AFFECTED FILES

D10045: test-copies: improve description of the C+H case

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10045 AFFECTED FILES

D10044: test-copies: improve description of the B+C "revert/restore" case

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10044 AFFECTED FILES

D10043: test-copies: improve description of the G+C case

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10043 AFFECTED FILES

D10042: test-copies: improve description of the G+F case

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10042 AFFECTED FILES

D10041: test-copies: improve description of the D+G case

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10041 AFFECTED FILES

D10040: test-copies: improve description of the A+E case

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10040 AFFECTED FILES

D10039: test-copies: improve description of the B+D case

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10039 AFFECTED FILES

D10035: test-copies: don't use empty file for "same content" cases

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY For main case (using filelog or sidedata), this lead to the following hash change. Changesets: - 01c2f5eabdc4ce2bdee42b5f86311955e6c8f573 →

D10038: test-copies: improve description of the B+C case

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10038 AFFECTED FILES

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

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will make its role clearer. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D10037 AFFECTED FILES

D10036: test-copies: use intermediate variable some commit descriptions

2021-02-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Right now, everything mostly says "simple merge", we want to use something a bit more descriptive. Before doing any changes, we do most of the churn. This

Re: Which parts of Evolve can be upstreamed?

2021-02-12 Thread Pierre-Yves David
On 2/11/21 11:54 PM, Martin von Zweigbergk via Mercurial-devel wrote: On Mon, Feb 8, 2021 at 11:28 PM Pierre-Yves David wrote: The short answer is: They are comment in multiple source files about their status and the part that can be uptreamed. For a longer answer I need to double check

D9981: hooks: introduce a `:run-with-plain` option for hooks

2021-02-10 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This option control if HGPLAIN should be set or not for the hooks. This is the first step to give user some control of the HGPLAIN setting for they hooks.

D9982: hooks: add a `auto` value for `hooks.*run-with-plain`

2021-02-10 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY That setting restore the behavior pre-5.6. The current HGPLAIN value is simply passed to the hooks. This allow user who needs it to fully mitigate the

D9980: procutil: add a way to unset and environment variable

2021-02-10 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The only way to not get the HGPLAIN behavior, is to not have the variable set at all. Having it set with -any- value will trigger the behavior. Currently

D9979: hooks: add some test about HGPLAIN setting and hooks

2021-02-10 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY In Mercurial 5.7, hooks are now ran with HGPLAIN set, which is a behavior change in. I could not find explicit test about it so I am adding one. The next

D9978: hooks: forbid ':' in hook name

2021-02-10 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The `:` character is a special separator in the config and it seems same do to the same for hooks. This is necessary to improve the experience around the

D9975: tweak-default: no longer enable the experimental return code

2021-02-10 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The change is quite new and undocumented (since it is experimental) so it seems premature to make it available in tweak default. In addition, I am not sure

D9974: cmdutil: add a missing byte prefix to string introduce in 976b26bdd0d8

2021-02-09 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The change is missing a the `b'foo'` prefix to make it a bytestring. This lead to a traceback in some third party extension. It is unclear to me why the

Re: Which parts of Evolve can be upstreamed?

2021-02-09 Thread Pierre-Yves David
emains. I think most of us agree that it would be good to have it upstreamed at some point. Are there some uncontroversial parts that I can start moving upstream? The obslog and prune commands seem like good candidates to me. What do you think?

D9947: ci-fix: backed out changeset d4c8b4b90ecb

2021-02-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 changeset is part of a series that break Continuous integration on python 2 for about a week. As not concrete solution have been found so far the safer

D9948: Bci-fix: backed out changeset 45afff0f530a

2021-02-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 changeset is part of a series that break Continuous integration on python 2 for about a week. As not concrete solution have been found so far the safer

D9946: copy-tracing: add a --compute flag to debugchangedfiles

2021-02-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 will help analysis of possible misbehaving cases. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9946

D9945: debugdiscovery: add flags to run discovery on subsets of the local repo

2021-02-01 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Generating new repository using strip of local clone is very expensive for large repositories. And such large repository are the most likely to requires

D9927: config: use level to properly deal with value priority

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY A higher priority alias will now take precedence over lower priority ones. This was a requirements step before using alias more widely, especially to

D9926: config: track the "level" of a value

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Config value now remember the "level" of the config that loaded it. This will be used to ensure good priority management for alias. REPOSITORY rHG

D9924: config: use a new `alter` method in `fixconfig`

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The `set` function is doing various work related to metadata (eg: the source, later the level). However the `fixconfig` call only updating some values

D9925: config: track "source" along side value

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Currently the source is stored in a entirely different way than the data. This is impractical. Especially if we are about to add more of such metadata. So

D9923: config: use the right API to access git-submodule

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute 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/D9923 AFFECTED FILES hgext/convert/git.py CHANGE DETAILS diff --git

D9922: config: use the right API to access template access

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Preventing direct access to the underlying dict will help a coming refactoring of `config`. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9921: config: use the right API to access subrepository section

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Preventing direct access to the underlying dict will help a coming refactoring of `config`. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9920: config: test priority involving alias and cli

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute 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/D9920 AFFECTED FILES tests/test-config.t CHANGE DETAILS diff --git

D9919: config: test priority involving alias and include

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute 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/D9919 AFFECTED FILES tests/test-config.t CHANGE DETAILS diff --git

D9918: config: test priority involving alias

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute 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/D9918 AFFECTED FILES tests/test-config.t CHANGE DETAILS diff --git

D9917: config: test priority involving the command line

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute 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/D9917 AFFECTED FILES tests/test-config.t CHANGE DETAILS diff --git

D9916: config: test priority involving include

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute 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/D9916 AFFECTED FILES tests/test-config.t CHANGE DETAILS diff --git

D9915: config: add a test for priority when includes are involved

2021-01-29 Thread marmoute (Pierre-Yves David)
marmoute 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/D9915 AFFECTED FILES tests/test-config.t CHANGE DETAILS diff --git

D9890: clang-format: issue version information in the skip message

2021-01-27 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will help people to configure their black REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9890 AFFECTED

D9889: heptapod-ci: indicate which version of clang-format is used for the run

2021-01-27 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will help understand possible mismatch REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9889 AFFECTED FILES

D9888: black: issue version information in the skip message

2021-01-27 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will help people to configure their black REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9888 AFFECTED

D9887: heptapod-ci: indicate which version of black is used for the run

2021-01-27 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will help understand possible mismatch REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9887 AFFECTED FILES

D9886: heptapod-ci: add a default value for HG_CI_IMAGE_TAG

2021-01-27 Thread marmoute (Pierre-Yves David)
marmoute 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/D9886 AFFECTED FILES contrib/heptapod-ci.yml CHANGE DETAILS diff --git

D9872: command-namespace: use `::` are the command separator

2021-01-25 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This was discussed on the mailing list / phab and `::` got a couple of positive feedback. I dediced to not introduce automatic folding of `::`, I feel

D9868: run-test: avoid byte issue when replacing output file of python test

2021-01-25 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Otherwise we get error like:: FileNotFoundError: [Errno 2] No such file or directory: "b'…/tests/test-minirst.py'.out" REPOSITORY rHG Mercurial

D9869: minirst: respect escaping in definition list key

2021-01-25 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY If we start adding `:` in command names… we need minirst to not choke on them. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9871: perf: test the formatting of a command help

2021-01-25 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Since we are about to addd ':' in command name I want this covered. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9870: help: escape ':' (as '\:') when generating command names

2021-01-25 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY We need this before we can use ":" in command names. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9870

D9865: store: use `endswith` to detect revlog extension

2021-01-25 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Suggested by Gregory Szorc. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9865 AFFECTED FILES

Re: Getting the revisions of .hgtags programatically

2021-01-24 Thread Pierre-Yves David
is list. Thanks Roy ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel -- Pierre-Yves David ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://w

D9841: share-share: have the hint issue more consistently and point to the right doc

2021-01-20 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This should help user in trouble to find solution in the documentation. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9840: doc: point to the main share-safe doc in the "mismatch" config

2021-01-20 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY User should be able to find their way with that. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9840 AFFECTED

D9839: doc: improves the share-safe documentation

2021-01-20 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The associated behavior is now clearer and we point to the other configuration that control the mismatching behavior. REPOSITORY rHG Mercurial BRANCH

D9837: doc: relocate doc for `share.safe-mismatch.source-safe.warn`

2021-01-20 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The documentation for the option is now right after its parent configuration: `share.safe-mismatch.source-safe` REPOSITORY rHG Mercurial BRANCH

D9838: doc: remove the section about share-safe from its verbose gating

2021-01-20 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The feature is no longer experimental. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9838 AFFECTED FILES

D9829: clang-format: reorder includes to appease the formatter

2021-01-18 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The back order was introduced in d0225a22040c . REPOSITORY rHG Mercurial BRANCH

D9826: callback: add a new_cl_rev_callback

2021-01-18 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will be useful for consistency and cache warming. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9826

D9828: changelog: use revision number as the cache key internally

2021-01-18 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This allow both rev and node request to match the cache. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9828

D9827: changelog: cache the latest `changelogrevision` access

2021-01-18 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This will speedup repeated call to the same data. Which can be common when warming various cache after adding a revision. REPOSITORY rHG Mercurial BRANCH

D9825: cext: add Python 3.10 support

2021-01-18 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY - Replace "Py_TYPE(obj) = type;" with "Py_SET_TYPE(obj, type);" - Add pythoncapi_compat.h header file to get Py_SET_TYPE() on Python 2.7-3.8. Header file

D9820: purge: move extension into core mercurial

2021-01-18 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The motivation is simple: it's nicer to avoid gating basic functionality. To reduce the risk of people shooting themselves in the feet, `--confirm` is

D9819: purge: also deal with directory with --confirm

2021-01-18 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Getting the exact number is simple (you have to project the result of previous deletion to detect super-directory with only about-to-be-deleted content). So

D9818: purge: add a --confirm option

2021-01-18 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The options provide a prompt to the user before permanent deletion are made. The prompt is currently not aware of directory deletion. I'll fix this in the

D9817: relnote: remove the reference to `debugstrip`

2021-01-18 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The `debug` namespace is not intended for end user and advertising it is a path to confusion and trouble. I think we should wait for the `admin` namespace to

D9813: discovery: add config options to control sample size

2021-01-17 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This get useful for running analysis of the current algorithm. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9810: persistent-nodemap: also exchange the nodemap data over the wire

2021-01-17 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY It appears that 2 bytes are never equals to 3 bytes. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D9810

D9809: persistent-nodemap: catch the right exception on python

2021-01-17 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY On Python 2, Missing file are IOError, not OSError, apparently. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9803: test-regression: backing out D9640 (63c923fd7fa8)

2021-01-16 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Since D9640 was pushed, we have seen failure on at least: - test-commit-interactive.t - test-i18n.t - test-log.t

D9802: copies: explicitly filter out existing file in graftcopies

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The `graftcopies` function does something very strange (maybe even wrong), it calls `_filter` with a pair of changeset that does not match the one used to

D9801: discovery: add a devel.discovery.exchange-heads

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Currently all discovery start with testing local and remote heads. For analysis purpose we make it possible to disable that initial "handshake" and start

D9799: discovery: add a discovery.grow-sample.rate

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This allow to control the effect of the growth rate on the discovery process while doing analysis. REPOSITORY rHG Mercurial BRANCH default REVISION

D9798: discovery: add a `devel', b'discovery.grow-sample`

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY That option make it possible to disable the "sample growing" behavior when doing analysis and comparison. REPOSITORY rHG Mercurial BRANCH default

D9800: discovery: move some debug output closer to were it belong

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY I assume these debug output, increment and comment drifted over time. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9797: discovery: document the `devel.discovery.randomize` option

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Gratuitous improvement as I was passing by this config section to add more. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9796: copies: add an devel option to trace all files

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Filelog based copy tracing only trace copy for file that have been added. This is a trade off between skipping some rare copy case in exchange for avoiding

D9795: copies: simplify the conditional for _filter's case 3

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The conditional is much simpler and the test are actually happier. This clarification of the conditional will also be necessary to properly support tracing

D9794: copies: clarify which case some conditional are handling

2021-01-15 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 the function a bit clearer. The middle conditional get no label because we about about to remove it. See next changeset for details. REPOSITORY

D9793: copies: fix some comment in _filter

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The scenario the comment describes match case 6, not case 2. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9792: heptapod-ci: allow testing with docker image other than :latest

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The project have a default config of latest for this variable. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D9789: clone: make sure we warm the cache after a clone

2021-01-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This work around any deviciency/limitation of the clone process. In our case this ensure the persistent nodemap exist with valid content. Ideally, the

<    7   8   9   10   11   12   13   14   15   16   >