Page MenuHomePhabricator

Fix Mercurial commit history ordering
ClosedPublic

Authored by epriestley on Dec 5 2017, 1:38 PM.
Tags
None
Referenced Files
F19504324: D18817.diff
Fri, Jan 9, 4:42 AM
F19502317: D18817.diff
Thu, Jan 8, 4:02 PM
F19479555: D18817.diff
Wed, Jan 7, 9:56 PM
F19471891: D18817.diff
Tue, Jan 6, 7:57 PM
F18866299: D18817.id45157.diff
Nov 3 2025, 4:29 PM
F18855121: D18817.id45157.diff
Nov 1 2025, 3:23 AM
F18853609: D18817.diff
Oct 31 2025, 6:15 PM
F18845749: D18817.diff
Oct 29 2025, 2:17 PM
Subscribers
None

Details

Summary

See https://discourse.phabricator-community.org/t/diffusion-observed-mercurial-repository-history-broken/825.

In D18769, I rewrote this from using the --branch flag (which is unsafe and does not function on branches named --config=x.y and such).

However, this rewrite accidentally changed the result order, which impacted Mercurial commit hisotry lists and graphs. Swap the order of the constraints so we get newest-to-oldest again, as expected.

Test Plan

Viewed a Mercurial repository's history graph, saw sensible chronology after the patch.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable