@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Fix an issue where Mercurial pushes would consider only the first and last commits
Summary:
Fixes T5197. `hg log --rev x --rev y` means "rev x, and also rev y".
Use `--rev x:y`, which means "all commits between x and y, inclusive".
Test Plan: Pushed 4 commits at once, got 4 commits in push log.
Reviewers: btrahan
Reviewed By: btrahan
Subscribers: epriestley
Maniphest Tasks: T5197
Differential Revision: https://secure.phabricator.com/D9309
+2
-2
+2
-2
src/applications/diffusion/engine/DiffusionCommitHookEngine.php
+2
-2
src/applications/diffusion/engine/DiffusionCommitHookEngine.php
···
695
695
$futures['old']->updateEnv('HG_PENDING', null);
696
696
697
697
$futures['commits'] = $repository->getLocalCommandFuture(
698
-
"log --rev %s --rev tip --template %s",
699
-
hgsprintf('%s', $hg_node),
698
+
'log --rev %s --template %s',
699
+
hgsprintf('%s:%s', $hg_node, 'tip'),
700
700
'{node}\1{branches}\2');
701
701
702
702
// Resolve all of the futures now. We don't need the 'commits' future yet,