Closed Bug 1529176 Opened 5 years ago Closed 5 years ago

always use our modified arc fork from moz-phab

Categories

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

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: glob, Assigned: zalun)

References

Details

(Keywords: conduit-triaged)

Attachments

(1 file)

in order to address some issues with arc, mostly around prompting, we'll be lightly forking it (see bug 1529175).

this bug is to update moz-phab to automatically clone and update this fork.

  • git will be required to use moz-phab
  • the first-run experience will involve cloning the arcanist and libphutil repos into ~/.mozbuild/moz-phab/
  • removal of the arc_command setting
  • updating install docs

We should somehow work around the install-certificate feature.
When there is no other arc installed user would need to ~/.mozbuild/moz-phab/arcanist/bin/arc install-certificate

(In reply to Piotr Zalewa [:zalun] from comment #1)

We should somehow work around the install-certificate feature.
When there is no other arc installed user would need to ~/.mozbuild/moz-phab/arcanist/bin/arc install-certificate

should be straight forward to check ~/.arcrc if the current hostname is present, and call arc install-certificate.

just need to ensure we check the right file on windows; probably best to look at arc's logic to determine how it finds/names arcrc.

Assignee: glob → pzalewa

Calling arc install-certificate would require filling in the prompt.
Currently, I'm displaying the command with a full path to our arc.
I think we should write directly to ~/.arcrc.
I will submit WIP to Phabricator.

Blocks: 1530945
Attachment #9047059 - Attachment description: Bug 1529176 - WIP install arc → Bug 1529176 - Use special arc version
Attachment #9047059 - Attachment description: Bug 1529176 - Use special arc version → Bug 1529176 - Use special arc version r=glob
Priority: -- → P3
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: