1593271 - (hg52) [meta] Mercurial 5.2 upgrade and compatibility

Status: RESOLVED FIXED

(Developer Services :: Mercurial: hg.mozilla.org, enhancement)

Comment 1

[Connor Sheehan [:sheehan]]

Attachment: hgext: mark extensions as 5.2 compatible and update various tests (Bug 1593271) r?glob

No major code changes required for compatibility with Mercurial 5.2,
however many tests have slightly changed output under the new Mercurial
version. Most of the changes come from changeset d7304434390f in upstream
Mercurial[1]. This changeset moves the line about added changes into the
transaction summary output. As a result, our pretxnchangegroup hook
output for the pushlog appears slightly earlier in the output. In some
other cases, the "added changes" line disappears as the transaction output
does not display when the transaction fails due to a hook or some other
reason, which is more correct behaviour in my opinion.

A few tests output changes are left out of this upgrade for now, specifically
tests with lots of output conditional on various versions of Mercurial.
Rather than tinker with updating the test output, I'm going to clean up
all those tests after the 5.2 upgrade on hg.mo is complete.

[1] https://mercurial-scm.org/repo/hg-committed/rev/d7304434390f

Comment 2

[Connor Sheehan [:sheehan]]

Attachment: hghooks: fix `test-changelog-correctness.t` (Bug 1593271) r?glob

The function signature used in mercurial.changelog.changelog.add
changed between 5.1 and 5.2. Adding *args and **kwargs papers
over the issue.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/025a9255b998
hgext: mark extensions as 5.2 compatible and update various tests r=glob
https://hg.mozilla.org/hgcustom/version-control-tools/rev/45833ba5ee20
hghooks: fix `test-changelog-correctness.t` r=glob

Comment 4

[Connor Sheehan [:sheehan]]

Attachment: vcsreplicator/bootstrap: update output to reflect new order in Mercurial 5.2 (Bug 1593271) r?mhentges,smacleod,zeid

This output now appears later, and has been updated in other tests
in changeset 025a9255b998b82f8.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/97bafa0e4e66
vcsreplicator/bootstrap: update output to reflect new order in Mercurial 5.2 r=mhentges