Page MenuHomePhabricator

Remove "Web Application Only" classes from libphutil
ClosedPublic

Authored by epriestley on Sep 2 2019, 1:43 PM.
Tags
None
Referenced Files
F13051011: D20773.largetrue.diff
Fri, Apr 19, 3:19 AM
F13051010: D20773.id49535.diff
Fri, Apr 19, 3:19 AM
F13051009: D20773.id49533.diff
Fri, Apr 19, 3:18 AM
F13051008: D20773.id.diff
Fri, Apr 19, 3:18 AM
Unknown Object (File)
Thu, Apr 11, 4:28 AM
Unknown Object (File)
Fri, Apr 5, 10:11 AM
Unknown Object (File)
Tue, Apr 2, 12:09 AM
Unknown Object (File)
Feb 10 2024, 4:37 AM
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