Page MenuHomePhabricator

Remove "Web Application Only" classes from libphutil
ClosedPublic

Authored by epriestley on Sep 2 2019, 1:43 PM.
Tags
None
Referenced Files
F18784964: D20773.id49533.diff
Mon, Oct 13, 3:59 PM
F18758417: D20773.id49535.diff
Sun, Oct 5, 10:18 PM
F18755318: D20773.id.largetrue.diff
Sun, Oct 5, 4:55 AM
F18755224: D20773.id49535.largetrue.diff
Sun, Oct 5, 4:27 AM
F18754528: D20773.id.diff
Sat, Oct 4, 11:21 PM
F18753992: D20773.largetrue.diff
Sat, Oct 4, 9:00 PM
F18746586: D20773.diff
Fri, Oct 3, 12:52 PM
F18739877: D20773.largetrue.diff
Wed, Oct 1, 11:22 PM
Subscribers
None

Details

Summary

Ref T13395. I want to merge libphutil into Arcanist ("the client part") and Phabricator ("the server part") since modern libphutil doesn't provide much value being standalone but makes verison management and deployment more complicated.

In service of this goal, remove a large number of classes which are only used at the web application level (by Phabricator) out of libphutil. The next change moves them into "phabricator/".

Test Plan

Grepped for all (?) affected symbols in libphutil and Arcanist, found no callsites.

Diff Detail

Repository
rPHU libphutil
Branch
strip2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23359
Build 32085: Run Core Tests
Build 32084: arc lint + arc unit