HomePhabricator

Fix Mercurial commit history ordering

Authored by epriestley on Dec 5 2017, 1:35 PM.

Description

Fix Mercurial commit history ordering

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.

Reviewers: amckinley

Reviewed By: amckinley

Differential Revision: https://secure.phabricator.com/D18817

Details

Committed
epriestleyDec 5 2017, 8:12 PM
Pushed
epriestleyDec 5 2017, 8:12 PM
Reviewer
amckinley
Differential Revision
D18817: Fix Mercurial commit history ordering
Parents
rP46d496b8cc06: Fix error for URL's that could mean several commits
Branches
Unknown
Tags
Unknown
Build Status
Buildable 18929
Build 25526: Run Core Tests