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

	F15502034: D20981.id49992.diff
	Sun, Apr 13, 10:26 PM

	F15490330: D20981.id49995.diff
	Fri, Apr 11, 4:12 PM

	F15488447: D20981.id49993.diff
	Thu, Apr 10, 10:30 PM

	F15485746: D20981.id.diff
	Thu, Apr 10, 2:04 AM

	F15483259: D20981.diff
	Wed, Apr 9, 11:08 AM

	F15422299: D20981.id49992.diff
	Sat, Mar 22, 7:04 AM

	F15422298: D20981.id.diff
	Sat, Mar 22, 7:04 AM

	F15422297: D20981.id49993.diff
	Sat, Mar 22, 7:04 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...