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
F17836832: D20070.id.diff
Sat, Jul 26, 4:36 PM
F17599327: D20070.diff
Tue, Jul 8, 6:39 PM
Unknown Object (File)
Jun 16 2025, 11:00 AM
Unknown Object (File)
Jun 6 2025, 5:55 PM
Unknown Object (File)
May 7 2025, 6:57 PM
Unknown Object (File)
May 6 2025, 11:58 PM
Unknown Object (File)
Apr 27 2025, 2:59 PM
Unknown Object (File)
Apr 26 2025, 5:22 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