Page MenuHomePhabricator
Differential D20070

Remove "iconv" PHP extension dependency
ClosedPublic
Actions

Authored by epriestley on Jan 30 2019, 10:51 PM.
Tags
None
Referenced Files
F19568307: D20070.id.diff
Jan 31 2026, 10:03 PM
F19532655: D20070.id.diff
Jan 20 2026, 12:37 PM
F19528912: D20070.id.diff
Jan 19 2026, 12:41 PM
F19510295: D20070.id47920.diff
Jan 11 2026, 6:58 AM
F19508032: D20070.diff
Jan 10 2026, 12:51 AM
F19085445: D20070.diff
Dec 2 2025, 6:14 PM
F18811393: D20070.id47931.diff
Oct 20 2025, 3:08 AM
F18747309: D20070.id.diff
Oct 3 2025, 3:32 PM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13232: Mac OS X Mojave, SIP, Code Signing, and Apache
Commits
rP87b0ef883905: Remove "iconv" PHP extension dependency
Summary

Depends on D20069. Ref T13232. This is a very, very weak dependency and we can reasonably polyfill it.

Test Plan

Grepped for iconv in libphutil, arcanist, and Phabricator.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 30 2019, 10:51 PM
Harbormaster completed remote builds in B21770: Diff 47920.Jan 30 2019, 10:53 PM
epriestley requested review of this revision.Jan 30 2019, 10:53 PM
amckinley accepted this revision.Jan 31 2019, 2:47 AM
This revision is now accepted and ready to land.Jan 31 2019, 2:47 AM
Closed by commit rP87b0ef883905: Remove "iconv" PHP extension dependency (authored by epriestley). · Explain WhyJan 31 2019, 3:47 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
scripts/
mail/
4 lines
src/
applications/
config/
check/
3 lines
controller/
2 lines

Diff 47931

View Options

scripts/mail/mail_handler.php

Loading...
View Options

src/applications/config/check/PhabricatorExtensionsSetupCheck.php

Loading...
View Options

src/applications/config/controller/PhabricatorConfigVersionController.php

Loading...
Phabricator · Built by Phacility