Page MenuHomePhabricator

In "arc land", if rebasing a range fails, attempt to "reduce" it
ClosedPublic

Authored by epriestley on Mar 3 2021, 9:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 15 2024, 4:49 AM
Unknown Object (File)
Feb 11 2024, 11:45 AM
Unknown Object (File)
Feb 8 2024, 7:31 PM
Unknown Object (File)
Jan 9 2024, 7:58 AM
Unknown Object (File)
Jan 6 2024, 5:09 PM
Unknown Object (File)
Jan 5 2024, 1:14 PM
Unknown Object (File)
Dec 28 2023, 6:23 AM
Unknown Object (File)
Dec 27 2023, 1:42 PM
Subscribers
None

Details

Summary

Ref T13576. See that task for discussion.

When a user runs arc land --pick A, they may be selecting a range of commits ("X..Y") which have ancestors ("V..W") that should NOT land.

We must slice "X..Y" out of history before we can merge it, to avoid landing changes from "V..W".

When "X..Y" is simple and linear, we can rebase the range to pick our desired slice out of history.

When "X..Y" includes merge commits, we frequently can not, and I could not identify any simple alternative. The best alternative I came up with is this "reduce" operation:

  • squash "into" onto Y, producing S, to guarantee there are no natural conflicts;
  • squash S onto X^, producing T, to get rid of the merge commits;
  • rebase T onto "into", producing R, to slice "X..Y" out of history;
  • squash R onto "into", producing Q. (R and Q will be the same, but this simplies the code.)

This feels flimsy and fragile, but I can't immediately find a way to break it. See T13576 for more discussion.

Test Plan
  • Applied conflicting changes to example.txt in master and feature1.
  • Ran arc land, got a merge conflict.
  • Resolved the conflict with git merge master.
  • Ran arc land.
    • Before: merge conflict.
    • After: arc land resolves the merge correctly.
  • Stacked feature2 on feature1, and made various mutations to feature1 and feature2, then ran arc land --pick feature2. Changes made in feature1 should not land, and they mostly do not. See T13576.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 3 2021, 9:41 PM
This revision was automatically updated to reflect the committed changes.