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
Unknown Object (File)
Thu, Apr 11, 12:24 PM
Unknown Object (File)
Fri, Apr 5, 3:14 PM
Unknown Object (File)
Sun, Mar 31, 8:57 AM
Unknown Object (File)
Wed, Mar 27, 8:14 PM
Unknown Object (File)
Sat, Mar 23, 2:12 PM
Unknown Object (File)
Mar 10 2024, 8:06 PM
Unknown Object (File)
Feb 21 2024, 7:07 PM
Unknown Object (File)
Feb 14 2024, 10:56 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.