Page MenuHomePhabricator

Remove obsolete "arc land" flags: --update-with-merge/rebase, --delete-remote
ClosedPublic

Authored by epriestley on Jun 9 2017, 1:58 PM.
Tags
None
Referenced Files
F19750252: D18108.id.diff
Sun, Feb 15, 10:55 PM
F19587009: D18108.diff
Feb 2 2026, 6:34 AM
F19562388: D18108.id.diff
Jan 31 2026, 1:23 PM
F19466016: D18108.id43565.diff
Jan 3 2026, 7:26 AM
F19275568: D18108.id43564.diff
Dec 23 2025, 12:38 AM
F18982415: D18108.diff
Nov 17 2025, 6:45 AM
F18931202: D18108.id43564.diff
Nov 10 2025, 2:40 AM
F18859662: D18108.diff
Nov 2 2025, 4:44 AM
Subscribers
None

Details

Summary

Fixes T12815. During the last update to "arc land", some flags were disabled but remained in place in case we needed to retain them.

It now seems reasonably clear that we do not. The "rebase" and "merge" strategies for landing were replaced by a better "headless" strategy which seems to avoid the original issues, so these flags no longer do anything or reasonably could do anything.

--delete-remote is slightly more ambiguous (e.g., see T12650 and maybe others) but the only real use case is "git push = save changes".

Test Plan

Ran arc land --update-with-rebase, was told the flag does not exist.
Grepped for affected flags/symbols.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from Remove obsolete "arc land" flags to Remove obsolete "arc land" flags: --update-with-merge/rebase, --delete-remote.
This revision is now accepted and ready to land.Jun 9 2017, 2:13 PM
This revision was automatically updated to reflect the committed changes.