Homec4science

Fix issue in Mercurial repos with duplicate branch heads

Authored by epriestley <git@epriestley.com> on Jul 13 2014, 15:55.

Description

Fix issue in Mercurial repos with duplicate branch heads

Summary:
Fixes T5613. A branch may have multiple heads in Mercurial, but executeOne() expects exactly one result.

Load them all instead. Equivalently, we could limit(1), but it's likely that we'll use the cursors in the future to reduce the number of VCS operations we do, so this is probably a little more along the lines where we're headed.

Test Plan: Poked around some repos.

Reviewers: chad, richardvanvelzen

Reviewed By: richardvanvelzen

Subscribers: epriestley

Maniphest Tasks: T5613

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

Details

Committed
epriestley <git@epriestley.com>Jul 13 2014, 15:55
Pushed
aubortJan 31 2017, 17:16
Parents
rPH9cb6b2cfcc59: Remove user-independent date and time functions from Phabricator
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH66a30ef97bbe: Fix issue in Mercurial repos with duplicate branch heads (authored by epriestley <git@epriestley.com>).Jul 13 2014, 15:55