Page MenuHomePhabricator
Differential D10089

Make artifacts imply dependencies on build steps
ClosedPublic
Actions

Authored by hach-que on Jul 31 2014, 2:03 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 28, 5:34 AM
Unknown Object (File)
Wed, May 28, 5:05 AM
Unknown Object (File)
Wed, May 28, 4:58 AM
Unknown Object (File)
Tue, May 27, 3:54 PM
Unknown Object (File)
Mon, May 26, 8:46 AM
Unknown Object (File)
Thu, May 8, 7:45 AM
Unknown Object (File)
Mon, May 5, 8:52 PM
Unknown Object (File)
Apr 18 2025, 3:48 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP0f355756f5ca: Make artifacts imply dependencies on build steps
Summary

This makes input artifacts imply the appropriate build step dependencies in the build plan. That is, if you use a host artifact in a build step, it will then implicitly depend on the 'Lease Host' step.

Test Plan

Viewed the build plan with the artifacts, saw the dependencies. Ran a build, saw everything execute in the correct order.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

hach-que updated this revision to Diff 24264.Jul 31 2014, 2:03 AM
hach-que retitled this revision from to Make artifacts imply dependencies on build steps.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 31 2014, 2:03 AM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B1953: Diff 24264.Jul 31 2014, 2:05 AM
epriestley accepted this revision.Jul 31 2014, 2:15 AM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/harbormaster/step/HarbormasterBuildStepImplementation.php
121

This should probably be array_merge(...), in case an artifact eventually has a numerical key which collides with a natural key in dependsOn.

This revision is now accepted and ready to land.Jul 31 2014, 2:15 AM
hach-que updated this revision to Diff 24265.Jul 31 2014, 2:26 AM
hach-que edited edge metadata.

Use array_merge

hach-que closed this revision.Jul 31 2014, 2:27 AM
hach-que updated this revision to Diff 24266.

Closed by commit rP0f355756f5ca (authored by @hach-que).

Harbormaster completed remote builds in B1954: Diff 24265.Jul 31 2014, 2:27 AM

Revision Contents
Changeset List

PathSize
src/
applications/
harbormaster/
controller/
1 line
engine/
3 lines
step/
34 lines

Diff 24266

View Options

src/applications/harbormaster/controller/HarbormasterPlanViewController.php

Loading...
View Options

src/applications/harbormaster/engine/HarbormasterBuildGraph.php

Loading...
View Options

src/applications/harbormaster/step/HarbormasterBuildStepImplementation.php

Loading...
Phabricator · Built by Phacility