From bb4667cb8490633baf36097cd289cea29aca39e1 Mon Sep 17 00:00:00 2001 From: epriestley Date: Fri, 2 Oct 2015 06:37:17 -0700 Subject: [PATCH] Fix WorkingCopy step to read correct commit variables Summary: Ref T9252. This variable was always wrong but we fell back to just resetting to `HEAD` before. Use the correct variable name. Test Plan: Verified variable name. Reviewers: chad, hach-que Reviewed By: hach-que Maniphest Tasks: T9252 Differential Revision: https://secure.phabricator.com/D14224 --- .../HarbormasterLeaseWorkingCopyBuildStepImplementation.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/applications/harbormaster/step/HarbormasterLeaseWorkingCopyBuildStepImplementation.php b/src/applications/harbormaster/step/HarbormasterLeaseWorkingCopyBuildStepImplementation.php index a1486af257..001a981a60 100644 --- a/src/applications/harbormaster/step/HarbormasterLeaseWorkingCopyBuildStepImplementation.php +++ b/src/applications/harbormaster/step/HarbormasterLeaseWorkingCopyBuildStepImplementation.php @@ -152,7 +152,7 @@ final class HarbormasterLeaseWorkingCopyBuildStepImplementation $repository = $repositories[$repository_phid]; - $commit = idx($variables, 'repository.commit'); + $commit = idx($variables, 'buildable.commit'); $ref_uri = idx($variables, 'repository.staging.uri'); $ref_ref = idx($variables, 'repository.staging.ref'); if ($commit) {