Page MenuHomePhabricator

Remove "Web Application Only" classes from libphutil
ClosedPublic

Authored by epriestley on Sep 2 2019, 1:43 PM.
Tags
None
Referenced Files
F15478212: D20773.largetrue.diff
Mon, Apr 7, 9:06 PM
F15476306: D20773.id.largetrue.diff
Mon, Apr 7, 5:17 AM
F15473506: D20773.id.diff
Sat, Apr 5, 10:37 PM
F15469348: D20773.diff
Fri, Apr 4, 12:19 PM
F15460848: D20773.id.diff
Tue, Apr 1, 2:53 AM
F15420154: D20773.id49533.diff
Fri, Mar 21, 11:51 AM
F15415529: D20773.id49535.diff
Thu, Mar 20, 6:29 AM
F15400807: D20773.diff
Mon, Mar 17, 2:03 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable