Page MenuHomePhabricator

Remove "Web Application Only" classes from libphutil
ClosedPublic

Authored by epriestley on Sep 2 2019, 1:43 PM.
Tags
None
Referenced Files
F15518933: D20773.id49533.diff
Sat, Apr 19, 6:13 PM
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
Mar 21 2025, 11:51 AM
F15415529: D20773.id49535.diff
Mar 20 2025, 6:29 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
Branch
strip2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23359
Build 32085: Run Core Tests
Build 32084: arc lint + arc unit