Differential D20981

Merge a small amount of remaining "libphutil/" code with Phabricator, break libphutil dependency
ClosedPublic
Actions

Authored by epriestley on Feb 12 2020, 11:11 PM.

Tags

None

Referenced Files

	F19469619: D20981.id49992.diff
	Mon, Jan 5, 1:38 AM

	F19469555: D20981.id49993.diff
	Mon, Jan 5, 12:35 AM

	F19468274: D20981.id.diff
	Sun, Jan 4, 1:37 AM

	F19303733: D20981.id49992.diff
	Wed, Dec 24, 5:16 AM

	F18810075: D20981.id49993.diff
	Oct 19 2025, 7:18 PM

	F18777282: D20981.id.diff
	Oct 11 2025, 1:44 AM

	F18777232: D20981.id49993.diff
	Oct 11 2025, 1:28 AM

	F18749345: D20981.id.diff
	Oct 4 2025, 3:26 AM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13395: Strip libphutil for parts
Commits: rP35a18146a28f: Merge a small amount of remaining "libphutil/" code with Phabricator, break…

Summary

Ref T13395. Moves a small amount of remaining "libphutil/" code into "phabricator/" and stops us from loading "libphutil/".

Test Plan

Browsed around; there are likely remaining issues.

Diff Detail

Repository

Branch

split5

Lint

Lint Passed

Severity	Location	Code	Message
Advice	src/infrastructure/daemon/PhutilDaemonOverseer.php:126	XHP16	TODO Comment
Advice	src/infrastructure/daemon/PhutilDaemonOverseer.php:210	XHP16	TODO Comment
Advice	src/infrastructure/daemon/PhutilDaemonPool.php:78	XHP16	TODO Comment
Advice	src/infrastructure/daemon/PhutilDaemonPool.php:248	XHP16	TODO Comment

Unit

Tests Passed

Build Status

Buildable 23818
Build 32761: Run Core Tests
Build 32760: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 12 2020, 11:11 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 12 2020, 11:12 PM

Harbormaster failed remote builds in B23818: Diff 49992!

Adjustments to daemons.

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 12 2020, 11:15 PM

Harbormaster failed remote builds in B23819: Diff 49993!

Depends on harness changes to even plausibly build.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 12 2020, 11:17 PM

This revision was landed with ongoing or failed builds.

Closed by commit rP35a18146a28f: Merge a small amount of remaining "libphutil/" code with Phabricator, break… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP35a18146a28f: Merge a small amount of remaining "libphutil/" code with Phabricator, break….

Revision Contents
Changeset List

Path

Size

externals/

cldr/

cldr_windows_timezones.xml

769 lines

resources/

timezones/

generate-timezone-map.php

46 lines

windows-timezones.json

126 lines

scripts/

init/

12 lines

src/

__phutil_library_map__.php

applications/

calendar/

parser/

ics/

PhutilICSParser.php

4 lines

infrastructure/

daemon/

PhutilDaemon.php

393 lines

PhutilDaemonHandle.php

506 lines

PhutilDaemonOverseer.php

405 lines

PhutilDaemonOverseerModule.php

71 lines

PhutilDaemonPool.php

360 lines

support/

startup/

PhabricatorStartup.php

11 lines

Diff 49992

externals/cldr/cldr_windows_timezones.xml

Loading...

resources/timezones/generate-timezone-map.php

Loading...

resources/timezones/windows-timezones.json

Loading...

scripts/init/lib.php

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/calendar/parser/ics/PhutilICSParser.php

Loading...

src/infrastructure/daemon/PhutilDaemon.php

Loading...

src/infrastructure/daemon/PhutilDaemonHandle.php

Loading...

src/infrastructure/daemon/PhutilDaemonOverseer.php

Loading...

src/infrastructure/daemon/PhutilDaemonOverseerModule.php

Loading...

src/infrastructure/daemon/PhutilDaemonPool.php

Loading...

support/startup/PhabricatorStartup.php

Loading...