1471687 - (remove-arc) Remove local dependency on Arcanist from new Phabricator command-line app

Status: RESOLVED FIXED

(Conduit :: moz-phab, enhancement, P2)

Description

[Mark Côté [:mcote]]

A better solution for commit-series support in Phabricator, once we have finished bug 1471678, is to write a command-line app that does not require Arcanist.  Rather, it will transmit all relevant information to a remote server, which will itself wrap Arcanist (continue to use Arcanist underneath lessens the chance of a Phabricator upgrade breaking our functionality).  The main benefit here is that users will no longer require a local Arcanist installation.  We can also do away with the git-cinnabar fork of Arcanist, as the client or server could do the commit-hash translations before invoking Arcanist.

Comment 1

[Mark Côté [:mcote]]

This bug is still relevant and part of the moz-phab performance-improvement plan (https://docs.google.com/document/d/14v33dMrAvs9KlhYVdbN7yYqJpubs8XFpnrDlsZTPBW0/edit#); however, we will be simplifying this and not using a separate server-side component.  moz-phab will instead talk directly to Phabricator via Conduit APIs (extending them if necessary).

Comment 5

[Piotr Zalewa [:zalun]]

Attachment: Bug 1471687 - Prevent from installing Arc when updating. r=glob,mhentges

Comment 6

[Zeid Zabaneh [:zeid] (pto)]

Attachment: Bug 1471687: Remove arcanist support r=sheehan,dkl

WIP, DO NOT MERGE.

• remove --no-arc configuration flag and support
• remove tests that test with --no-arc
• do not checkout revisions when submitting patches

Comment 7

[Sylvestre Ledru [:Sylvestre]]

Attachment: Bug 1471687 - Remove "arc_last_check" from the default configuration r?glob

Comment 8

[Sylvestre Ledru [:Sylvestre]]

Attachment: Bug 1471687 - Followup - Remove references to "arc" in the doc r?glob