D12619: auto-upgrade: skip the operation if the repository cannot be locked

2022-05-06 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This seems like a fine default behavior for now. If some users wants something more aggressive we can make the behavior configurable in the future. REPOSITORY

D12616: debuglock: make the command more useful in non-interactive mode

2022-05-06 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The existing prompt mode simply release the lock immediately in non-interactive. That is quite useless in the test so now the non-interactive mode simply wait

D12618: auto-upgrade: add a test case where the repository is already locked

2022-05-06 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This show the current behavior is the repository to auto-upgrade is already locked. The current behavior is to abort, which is probably not great. Now

D12617: wait-on-file: properly wait on any file and symlink

2022-05-06 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 utility more useful, for example to wait on a lock file. We also an explicit -L check since the lock are "weird" symlink. REPOSITORY rHG

D12613: auto-upgrade: introduce a way to auto-upgrade to/from tracked-hint

2022-05-06 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 what we introduce for `share-safe`, but apply tot he tracked-hint feature. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D12614: auto-upgrade: introduce a way to auto-upgrade to/from dirstate-v2

2022-05-06 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 what we introduce for `share-safe`, but apply tot he tracked-hint feature. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D12611: auto-upgrade: introduce a way to auto-upgrade to/from share-safe

2022-05-06 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 the first "automatic-upgrade" capabilities. In the following commits, similar features are coming for other "fast to upgrade" format. This is

D12615: auto-upgrade: add a test case with no permission to lock the repository

2022-05-06 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This show the current behavior is the repository is unlockable. The current behavior is to abort, which is probably not great. Now that we have a proper

D12612: upgrade: split some logic from UpgradeOperation

2022-05-06 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The automatic-upgrade and the upgrade-repo code path should be able to use the same code. However that code often need an UpgradeOperation object to function.

D12610: rust: make requirements public

2022-05-06 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY These can be used by any client crates (including `rhg`), no need to make them private to the crate. REPOSITORY rHG Mercurial BRANCH default REVISION

D12484: help: set the large-file-limit to 10MB

2022-04-07 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 a minor increase (5%) and makes the doc much clearer. REPOSITORY rHG Mercurial BRANCH stable REVISION DETAIL

D12483: help: clarify the unit of `ui.large-file-limit` config

2022-04-07 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Its might be a bit confusing, especially since `large-file.min-size` uses MB. REPOSITORY rHG Mercurial BRANCH stable REVISION DETAIL

D12481: debuglock: ignore ENOENT error when unlocking

2022-04-06 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 consistent with the main `lock.release` code. REPOSITORY rHG Mercurial BRANCH stable REVISION DETAIL https://phab.mercurial-scm.org/D12481

D12480: run-tests: introduce "forward-slash" version of everything on windows

2022-04-06 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 be useful for some shell invocation. REPOSITORY rHG Mercurial BRANCH stable REVISION DETAIL https://phab.mercurial-scm.org/D12480 AFFECTED

D12479: tests-racy-mutation: pass the editor through config instead of env variable

2022-04-06 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY On Windows msys seems to do awful mangling of the environment variable content that confuses everything to the death. Going through the config works fine, so

D12437: path: explicitly declare the `bookmarks.mode` suboptions

2022-04-04 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 documentation and discovery. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12437 AFFECTED

D12436: path: explicitly declare the `pushrev` suboptions

2022-04-04 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 documentation and discovery. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12436 AFFECTED

D12435: path: explicitly declare the `multi-urls` suboptions

2022-04-04 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 documentation and discovery. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12435 AFFECTED

D12434: path: explicitly declare the `bookmarks.mode` suboptions

2022-04-04 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 documentation and discovery. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12434 AFFECTED

D12398: discovery: also audit the number of queries done

2022-03-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY In addition to the number of roundtrip, we now also track the number of queries we perform, this is useful to assert the tradeoff between number of roundtrip

D12399: perf-util: add a `compare-discovery-case` script

2022-03-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 script run the same discovery case using multiple variants of the algorithm and report differences in behavior, especially regarding the numbers of

D12397: search-discovery-case: display more information about the interresting case

2022-03-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY We display information about the total number of revs and the common/missing numbers. This is useful to spot the interresting case. REPOSITORY rHG Mercurial

D12396: subsetmaker: rework the antichain generation to be usable

2022-03-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Before this, antichain computation can run for 10s of hours without completion in sight. We use a more direct approach in the computation to keep the

D12395: subsetmaker: use SortedSet for the scratch variant

2022-03-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 provides a massive speedup on wide repository with many heads. For example on mozilla-try, this move from un-usable slow to fairly instant. REPOSITORY

D12394: subsetmaker: stabilize the computation of `scratch` subset

2022-03-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY `heads` is set, order of the element are not deterministic and we need to stabilize that if we want to get reproducible results. REPOSITORY rHG Mercurial

D12393: hgignore: ignore .testtimes in more location

2022-03-22 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY See the inline comment. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12393 AFFECTED FILES .hgignore

D12388: ci: use the `v1.0` flavor of the docker images in the CI

2022-03-19 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This new versioning will help use to maintain backward compatibility in the docker image. This will be useful to deal with mismatch between default/stable in

D12382: test: use `wait-on-file` in `test-racy-mutations.t`

2022-03-17 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The official utility scale its timeout with the run-tests.py one. So lets use it. REPOSITORY rHG Mercurial BRANCH stable REVISION DETAIL

D12372: debugdiscovery: fix a typo in the help

2022-03-11 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/D12372 AFFECTED FILES mercurial/debugcommands.py CHANGE DETAILS diff --git

D12214: windows: skip a section of a test that is legitimately broken on windows

2022-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 See the comment in the test itself. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12214 AFFECTED FILES

D12201: tracked-key: remove the dual write and rename to tracked-hint

2022-02-17 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The dual-write approach was mostly useless. As explained in the previous version of the help, the key had to be read twice before we could cache a value.

D12200: tracked-file: rename the format option to use `use-`

2022-02-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 is more consistent with the other options. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12200 AFFECTED

D12198: tracked-key: make it possible to upgrade to and downgrade from the feature

2022-02-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 seems rather important if we want people to start using it. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D12199: tracked-key: update the requirement value

2022-02-17 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY We renamed the config option but we forgot to change the actual value… REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D12196: upgrade: prepare code (and output) for the idea of upgrading share

2022-02-16 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This would work for a subset of action only. Our first target is dirstate-v2. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D12197: upgrade: make dirstate-v2 format variant compatible with share

2022-02-16 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This only affects the dirstate and is safe to upgrade in the share. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D12184: dirstate-tracked-key: update the config value to match latest discussion

2022-02-15 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Special cases are not special enough, we align the option name on the other. The `version` value is undocumented for now as it can only have a single

D12181: narrow: add support for merging add and remove outside of the tracked set

2022-02-15 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 This extend f1eb77dceb36 to test and support the

D12161: help: make the help for `safe-mismatch` real section

2022-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 extra new lines mean the minirst engine were not seeing them as section… meaning they could not be addressed directly. REPOSITORY rHG Mercurial BRANCH

D12160: help: avoid repeated value in the safe-mistmatch help

2022-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 This was copy pasted and overlooked when it got in. REPOSITORY rHG Mercurial BRANCH stable REVISION DETAIL https://phab.mercurial-scm.org/D12160

D12159: help: add missing `.` in config reference

2022-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 `hg help` command is still confused at the number of dots, but at least the reference is correct. REPOSITORY rHG Mercurial BRANCH stable REVISION

D12133: phabricator-refresh: add a magic value to skip it in the CI

2022-02-03 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY When maintaining (light) fork, it is useful to be able to disable this. REPOSITORY rHG Mercurial BRANCH stable REVISION DETAIL

D12129: dirstate-v2: rename the configuration to enable the format

2022-02-01 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. marmoute added a comment. This patch is intended for stable. REVISION SUMMARY The rename of the old experimental name was overlooked before the 6.0 release. We rename

D12128: branchmap: stop writing cache for uncommitted data

2022-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 If we are about to write the branch while a transaction is active. we delay that write. After the transaction is closed, we flush all the write we delayed

D12127: transaction: add a way to know a transaction has been finalized

2022-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 This will be useful to fix the timing of the branchmap on disk caching. REPOSITORY rHG Mercurial BRANCH stable REVISION DETAIL

D12126: branchmap: Add a test about writing branchmap and aborted transaction

2022-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 It turn out that we update the branchmap on disk whenever we recompute it… including in a middle of a transaction. This means writing the new branchmap too

D12125: transaction: do not rely on a global variable to post_finalize file

2022-01-31 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY We can just add a new argument to the `addfilegenerator` function. This is more explicit and therefor clearer and less error prone. REPOSITORY rHG

D12124: dirstate: introduce a "tracked-key" feature

2022-01-31 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY A new format variant is introduced. When used, a `tracked-key` file will be generated. That file will be update when the set of tracked file might have

D12123: dirstate: rename the filegenerator used for writing

2022-01-31 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY We will need a different name in the next changesets. Changing the name is actually not that trivial so we do it in its own changeset. REPOSITORY rHG

D12122: dirstate: use a context manager when writing the dirstate

2022-01-31 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 sure the file is closed in a timely manner. We define a lambda for the file opening. It might seems a bit overkill here but a future changeset

D12119: narrow: allow merging non-conflicting change outside of the narrow spec

2022-01-30 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 We use the mergestate to carry information about these merge action and reprocess them at commit time to apply the necessary

D12117: merge-actions: have an attribute for narrow safetiness

2022-01-30 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 the core doing narrow filtering to process action without explicitely listing all possible actions. This is important to make the actions system

D12118: merge-actions: add some information about the "changes" the action do

2022-01-30 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 when processing merges action outside of the narrow-spec. "support" outside of narrow file on commit REPOSITORY rHG Mercurial BRANCH

D12114: merge-actions: make merge action a full featured object

2022-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 This open the way for having "smarter" value as action, making the usage code simpler and more flexible. We have to explicitly use __bytes__ call in a

D12116: merge-actions: add an explicite "no_op" attribute

2022-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 This make the MergeAction smarter and able to describe themself. This is useful to help introducing more MergeAction object that better the complexity of the

D12115: merge-actions: gather all created action into a set

2022-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 This is mostly to demonstrate we can do this before we start adding more specialized set. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D12113: convert: use the merge action constant

2022-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 constant exists, lets use them. Otherwise we cannot make these constant more powerful. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D12112: large-file: use the merge action constant

2022-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 constant exists, lets use them. Otherwise we cannot make these constant more powerful. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL

D12111: merge: stop using merge action for pathconflict option

2022-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 This is not the b'r' you are looking for. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12111 AFFECTED FILES

D12110: merge: break up a not-so-one-liner for readability

2022-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/D12110 AFFECTED FILES mercurial/merge.py CHANGE DETAILS diff --git

D12109: merge: break up a not-so-one-liner for readability

2022-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/D12109 AFFECTED FILES mercurial/merge.py CHANGE DETAILS diff --git

D12108: merge: break up a not-so-one-liner for readability

2022-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/D12108 AFFECTED FILES mercurial/merge.py CHANGE DETAILS diff --git

D12107: merge: break up a not-so-one-liner for readability

2022-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/D12107 AFFECTED FILES mercurial/merge.py CHANGE DETAILS diff --git

D12106: merge: break up two not-so-one-liner for extra readability

2022-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/D12106 AFFECTED FILES mercurial/merge.py CHANGE DETAILS diff --git

D12105: merge: break up a not-so-one-liner for readability

2022-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/D12105 AFFECTED FILES mercurial/merge.py CHANGE DETAILS diff --git

D12104: merge: break up a not-so-one-liner for readability

2022-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 (even if not fully satisfied this time) REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12104 AFFECTED FILES

D12103: merge: break up a not-so-one-liner for readability

2022-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/D12103 AFFECTED FILES mercurial/merge.py CHANGE DETAILS diff --git

D12102: merge: break up a not-so-one-liner for readability

2022-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/D12102 AFFECTED FILES mercurial/merge.py CHANGE DETAILS diff --git

D12101: merge: break up a not-so-one-liner for readability

2022-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/D12101 AFFECTED FILES mercurial/merge.py CHANGE DETAILS diff --git

D12096: stream-clone: allow to change persistent-nodemap format during stream clone

2022-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 Persistent nodemap affect the store format. However it is fairly isolated and fast to general locally. So not making it a fixed part of the stream clone is

D12094: debugbuilddag: add a flag to allow running it from a non-empty repository

2022-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 Allow that by default seems "dangerous", but having a flag to make it possible will be useful to help building some repository incrementally. The newly

D12095: stream-clone: add 5000 changesets to test-clone-stream-format

2022-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 make sure the changelog is not inlined. We needs this to test coming behavior change around persistent-nodemap. REPOSITORY rHG Mercurial BRANCH

D12093: persistent-nodemap: properly delete all nodemap files on downgrade

2022-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 Previously, we were only deleting the docket file… REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D12093

D12091: stream-clone: filter possible missing requirements using all supported one

2022-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 The `supportedformat` requirements is missing some important requirements and it seems better to filter out with all requirements we know, not just an

D12083: requirements: add an official `REVLOG_COMPRESSION_ZSTD` const

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Such constant was missing and its value was missing from the set of requirements that needs to be preserved through stream clone. This did not had any

D12084: stream-clone: also filter the requirement we put in the bundle 2

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY We were wrongly putting irrelevant requirements in the bundle and the receiving side was getting confused, treating them as being missing while still putting

D12080: stream-requirements: smoother matching in test-ssh-proto.t

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The value of `streamreqs` changes depending of the format used, but this is not relevant for this tests. So we doing a smarter matching of the line to avoid

D12082: stream-requirements: smoother matching in test-http-protocol.t

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The value of `streamreqs` changes depending of the format used, but this is not relevant for this tests. So we doing a smarter matching of the line to avoid

D12081: stream-requirements: smoother matching in test-ssh-bundle1.t

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The value of `streamreqs` changes depending of the format used, but this is not relevant for this tests. So we doing a smarter matching of the line to avoid

D12079: stream-requirements: smoother matching in test-hgweb-commands.t

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The value of `streamreqs` changes depending of the format used, but this is not relevant for this tests. So we doing a smarter matching of the line to avoid

D12078: stream-requirements: smoother matching in test-treemanifest.t

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The value of `streamreqs` changes depending of the format used, but this is not relevant for this tests. So we doing a smarter matching of the line to avoid

D12077: stream-requirements: smoother matching in test-ssh-proto-unbundle.t

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY The value of `streamreqs` changes depending of the format used, but this is not relevant for this tests. So we doing a smarter matching of the line to avoid

D12074: test-http-bad-server: abstract the `streamreqs` value

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Now that socket closing are decided much closer to the actual closure, we can use fuzzy matching for this part of the exchange that changes on a regular

D12076: test-bundle: split some variant in there own section

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Same rational as the previous commit about test-bundle.t. These line are quite volatile and having dedicated block will make their update clearer and simpler.

D12075: test-bundle: split each variant in there own section

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY Having conditional line matching for this kind of big block is quite inconvenient to update. The new way should be simpler to maintain. REPOSITORY rHG

D12073: test-http-bad-server: use the new pattern-reading for a test-case

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This test case is now less sensitive to change of unrelated bits of the client/server exchange. Since this introduce some churn in the output, we do it

D12072: test-http-bad-server: use the new pattern-reading for a test-case

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This test case is now less sensitive to change of unrelated bits of the client/server exchange. Since this introduce some churn in the output, we do it

D12071: test-http-bad-server: use the new pattern-reading for a test-case

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This test case is now less sensitive to change of unrelated bits of the client/server exchange. Since this introduce some churn in the output, we do it

D12070: test-http-bad-server: use the new pattern-reading for a test-case

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This test case is now less sensitive to change of unrelated bits of the client/server exchange. Since this introduce some churn in the output, we do it

D12069: test-http-bad-server: use the new pattern-reading for a test-case

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This test case is now less sensitive to change of unrelated bits of the client/server exchange. Since this introduce some churn in the output, we do it

D12067: test-http-bad-server: move on test from bytes-count to pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This specific test case is now using the new "close when a pattern is seen" approach compared to the more fragile "that many bytes were sent" approach.

D12068: test-http-bad-server: introduce socket closing after reading a pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY We introduce the `close-after-recv-patterns` option. It has the same goal as `close-after-send-patterns` with a slightly different implementation. Reading

D12066: test-http-bad-server: move on test from bytes-count to pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This specific test case is now using the new "close when a pattern is seen" approach compared to the more fragile "that many bytes were sent" approach.

D12065: test-http-bad-server: move on test from bytes-count to pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This specific test case is now using the new "close when a pattern is seen" approach compared to the more fragile "that many bytes were sent" approach.

D12064: test-http-bad-server: move on test from bytes-count to pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This specific test case is now using the new "close when a pattern is seen" approach compared to the more fragile "that many bytes were sent" approach.

D12063: test-http-bad-server: move on test from bytes-count to pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This specific test case is now using the new "close when a pattern is seen" approach compared to the more fragile "that many bytes were sent" approach.

D12062: test-http-bad-server: move on test from bytes-count to pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This specific test case is now using the new "close when a pattern is seen" approach compared to the more fragile "that many bytes were sent" approach.

D12061: test-http-bad-server: move on test from bytes-count to pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This specific test case is now using the new "close when a pattern is seen" approach compared to the more fragile "that many bytes were sent" approach.

D12060: test-http-bad-server: move on test from bytes-count to pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This specific test case is now using the new "close when a pattern is seen" approach compared to the more fragile "that many bytes were sent" approach.

D12059: test-http-bad-server: move on test from bytes-count to pattern

2022-01-24 Thread marmoute (Pierre-Yves David)
marmoute created this revision. Herald added a reviewer: hg-reviewers. Herald added a subscriber: mercurial-patches. REVISION SUMMARY This specific test case is now using the new "close when a pattern is seen" approach compared to the more fragile "that many bytes were sent" approach.

  1   2   3   4   5   6   7   8   9   10   >