Page MenuHomePhabricator

Allow users to create buildables from diffs
ClosedPublic

Authored by hach-que on Nov 9 2013, 6:32 AM.
Tags
None
Referenced Files
F18739413: D7546.id.diff
Wed, Oct 1, 9:19 PM
F18735876: D7546.id17037.diff
Wed, Oct 1, 3:48 AM
F18732595: D7546.diff
Tue, Sep 30, 6:56 PM
F18619179: D7546.diff
Mon, Sep 15, 1:55 AM
F18589743: D7546.diff
Fri, Sep 12, 4:10 AM
F18512971: D7546.diff
Fri, Sep 5, 8:47 AM
F17945491: D7546.id17037.diff
Jul 31 2025, 12:15 PM
F17931903: D7546.id.diff
Jul 30 2025, 7:32 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T1049: Implement Harbormaster
Commits
Restricted Diffusion Commit
rPeccbbce9a282: Allow users to create buildables from diffs
Summary

Now that diffs have PHIDs we can create buildables for them.

This also adds buildable.diff in the variables list so the diff ID is available, and it also fixes the Cancel button on "Edit Plan" page so it redirects to the right place.

Test Plan

Created a buildable from a diff, ran a build plan against it that had echo ${buildable.diff} and got the right ID. Also tested the "Edit Plan" cancel redirect.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

(This has a tiny dependency on D7544 for DifferentialRevision -> DifferentialDiff in VariableBuildStepImplementation, so I'm holding it until that lands or we need it for something else.)