archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Fix WorkingCopy step to read correct commit variables

Browse Source 
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
This commit is contained in:
epriestley
2015-10-02 06:37:17 -07:00
parent 2728a9f964
commit bb4667cb84
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/applications/harbormaster/step/HarbormasterLeaseWorkingCopyBuildStepImplementation.php
View File

@@ -152,7 +152,7 @@ final class HarbormasterLeaseWorkingCopyBuildStepImplementation
$repository = $repositories[$repository_phid]; $repository = $repositories[$repository_phid];
$commit = idx($variables, 'repository.commit'); $commit = idx($variables, 'buildable.commit');
$ref_uri = idx($variables, 'repository.staging.uri'); $ref_uri = idx($variables, 'repository.staging.uri');
$ref_ref = idx($variables, 'repository.staging.ref'); $ref_ref = idx($variables, 'repository.staging.ref');
if ($commit) { if ($commit) {