Page MenuHomePhabricator

[Wilds] Remove include_path mangling and drop support for "externals/includes"
ClosedPublic

Authored by epriestley on Sep 18 2018, 8:07 PM.
Tags
None
Referenced Files
F13061780: D19692.diff
Fri, Apr 19, 7:58 PM
F13060280: D19692.id.diff
Fri, Apr 19, 5:37 PM
Unknown Object (File)
Tue, Apr 2, 7:42 PM
Unknown Object (File)
Mon, Apr 1, 2:50 AM
Unknown Object (File)
Sat, Mar 30, 6:46 AM
Unknown Object (File)
Sat, Mar 30, 6:46 AM
Unknown Object (File)
Sat, Mar 30, 6:46 AM
Unknown Object (File)
Mar 14 2024, 4:57 PM
Subscribers
None

Details

Summary

Ref T13098. This logic comes from D3243 and was theoretically used to let you install libphutil with Homebrew. Since libphutil no longer exists and the other use cases I can come up with are questionable/obsolete, remove it.

(As we move into T5055, I expect to provide better tools for bundling/managing external dependencies.)

Test Plan

Ran arc, same as the old arc.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.
This revision is now accepted and ready to land.Sep 20 2018, 12:17 AM
This revision was automatically updated to reflect the committed changes.