Include Arcanist, the client for Phabricator, in MozillaBuild

NEW
Unassigned

Status

task
2 years ago
6 months ago

People

(Reporter: xidorn, Unassigned)

Tracking

Details

Reporter

Description

2 years ago
It seems that we are moving to using Phabricator instead of MozReview... Phabricator needs Arcanist, and the latter is a PHP application... which means MozillaBuild should also include PHP runtime.

(I'm not a fan of this kind of things... but if we have decided to go that way, we should be prepared to that as soon as possible, I guess.)
I'd rather do bug 1366401 than ship PHP in MozillaBuild.
Reporter

Comment 2

2 years ago
If we don't need PHP, that would be even better :)
Arcanist isn't necessary if you're just doing a build. This task (if we do it at all) seems better suited for the version-control-tools repo and the bootstrap script.
With the announcement of Phabricator open for everyone, it sounds important to understand what's the intended way to use arcanist on Windows, should we wait for MozillaBuild, bootstrap or do a manual install of all the requirements? Or wait bug 1366401?

I must note that the Arcanist Quick Start page says "This won't work for Windows, see Arcanist User Guide: Windows for instructions.", and the Windows instructions page says "You need to install: Arcanist itself, see Arcanist User Guide. or Arcanist Quick Start.".
Flags: needinfo?(gps)
Fwiw, I'm currently trying the WSL, not sure if it will work as expected but it sounds like the best bet for now.
Redirecting to mcote to provide guidance on how to prioritize supporting Arcanist. (I'm not sure what the strategy for client-side review submission is and that strategy dictates how much effort should go into supporting Arcanist in `mach bootstrap`, MozillaBuild, etc.)
Flags: needinfo?(gps) → needinfo?(mcote)
We're still working out options.  It's going to take a little while to sort this out; I'll comment back here when I know what the plan is regarding a command-line client.
Flags: needinfo?(mcote)

Updated

6 months ago
No longer blocks: phab-switch
You need to log in before you can comment on or make changes to this bug.