HomePhabricator

[land-bot] Cleanup unused code paths for diff patches

Description

[land-bot] Cleanup unused code paths for diff patches

Summary:
It turns out that not as much code can be shared between
landing revisions and diff patches as I originally thought.
Anything that is shared can be composed of smaller scripts that are
called by two separate implementations.

Leaving the two implementations together in one script creates an
unnecessary hazard for testing and authoring future patches. It's best
to clean this up and re-introduce a separate code flow when there is
something ready for handling diff patches.

This patch has no change in behavior for the revision patch code flow.

Depends on D7421

Note: The name land-patch.sh is not entirely appropriate anymore.
But I left it as-is in this patch to facilitate code review. I will rename
it to something more appropriate when I introduce the sister implementation.

Test Plan:
CONDUIT_TOKEN=<token> ./land-patch.sh Dxxxx
Pick a reviewed, but not yet landed, revision for Dxxxx. The script will
fail, but it should patch and run smoke tests as expected.

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D7422

Details

Provenance
jasonbcoxAuthored on Sep 10 2020, 21:44
jasonbcoxPushed on Sep 10 2020, 22:38
Reviewer
Restricted Project
Differential Revision
D7422: [land-bot] Cleanup unused code paths for diff patches
Parents
rABCa54e99032985: [land-bot] Extract revision status check code
Branches
Unknown
Tags
Unknown