1217569 - Mercurial 3.6 compatibility for pushlog

Status: RESOLVED FIXED

(Developer Services :: Mercurial: Pushlog, defect)

Description

[Gregory Szorc [:gps]]

There are some test failures with the pushlog extension on Mercurial 3.6RC. Looks like we'll need to perform a little porting.

Comment 1

[Gregory Szorc [:gps]]

Attachment: MozReview Request: pushlog: optionally perform dockeywords manipulation (bug 1217569); r?smacleod

pushlog: optionally perform dockeywords manipulation (bug 1217569); r?smacleod

dockeywords was removed in Mercurial 3.6 and the behavior is automatic
if the regular keywords dict is updated.

Comment 2

[Gregory Szorc [:gps]]

Attachment: MozReview Request: pushlog: don't use weakref to repo (bug 1217569); r?smacleod

pushlog: don't use weakref to repo (bug 1217569); r?smacleod

The old behavior won't work in Mercurial 3.6 because of
https://bz.mercurial-scm.org/show_bug.cgi?id=4915.

This is arguably a bug in Mercurial. However, looking at the Mercurial
code, relying on the same exact object for the repo is kinda difficult,
as filters are involved and these create new repo instances.

This commit removes the weakref. It also stops caching the pushlog
instance on the repo. There is no state on the instance and recreating
it is cheap. We arguably should have been doing this all along.

Since this was the last piece of pushlog that didn't work with 3.6, I
bumped the testwith property to indicate 3.6 compatibility.

Comment 3

[Steven MacLeod [:smacleod]]

Comment on attachment 8677937 [details]
MozReview Request: pushlog: optionally perform dockeywords manipulation (bug 1217569); r?smacleod

https://reviewboard.mozilla.org/r/23057/#review20571

Comment 4

[Steven MacLeod [:smacleod]]

Comment on attachment 8677938 [details]
MozReview Request: pushlog: don't use weakref to repo (bug 1217569); r?smacleod

https://reviewboard.mozilla.org/r/23059/#review20573

Comment 5

[Gregory Szorc [:gps]]

https://hg.mozilla.org/hgcustom/version-control-tools/rev/4246ddb54f526958e1cfa40a0d4c6dff8882e9a3
pushlog: optionally perform dockeywords manipulation (bug 1217569); r=smacleod

https://hg.mozilla.org/hgcustom/version-control-tools/rev/fd8f2df5ffe1cf043a7ae51f26672a749a595b0e
pushlog: don't use weakref to repo (bug 1217569); r=smacleod

Comment 6

[Gregory Szorc [:gps]]

Deployed.