Page MenuHomePhabricator

Try to load arcanist libraries that are checked out alongside arcanist

Authored by solarnz on Feb 17 2020, 6:19 AM.
This revision can not be accepted until the required legal agreements have been signed.


Group Reviewers
Blessed Reviewers
Required Signatures
L28 Phacility Individual Contributor License Agreement

This tries to emulate the old libphutil resolution rules to
include libraries alongside the arcanist directory

This attempts to fix the issue at

Test Plan

Ran this locally, and it is picking up the custom library we
use internally at, that we check out alongside the
arcanist repository

Diff Detail

rARC Arcanist
Lint ErrorsExcuse: I didn't touch this code
Errorscripts/arcanist.php:563XHP5Use of Undeclared Variable
Errorscripts/arcanist.php:573XHP5Use of Undeclared Variable
Errorscripts/arcanist.php:597XHP5Use of Undeclared Variable
No Unit Test Coverage
Build Status
Buildable 23866
Build 32834: arc lint + arc unit

Event Timeline

solarnz created this revision.Feb 17 2020, 6:19 AM
solarnz requested review of this revision.Feb 17 2020, 6:19 AM

@epriestley, I believe has already signed the Corporate CLA, cc @joshuaspence

I don't have a Freelancer CLA on file. (This is near the top of my queue of things to fix anyway so it's probably moot.)

This is likely fixed, at least for the moment, by D21004.

solarnz abandoned this revision.Feb 17 2020, 10:23 PM

Yep, D21004 seems to have done the job

Content Hidden

The content of this revision is hidden until the author has signed all of the required legal agreements.