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
Unknown Object (File)
Sat, Jan 25, 4:12 PM
Unknown Object (File)
Sat, Jan 25, 4:12 PM
Unknown Object (File)
Sat, Jan 25, 4:12 PM
Unknown Object (File)
Fri, Jan 24, 11:42 AM
Unknown Object (File)
Thu, Jan 23, 9:24 PM
Unknown Object (File)
Wed, Jan 22, 5:37 AM
Unknown Object (File)
Tue, Jan 21, 12:06 PM
Unknown Object (File)
Tue, Jan 21, 9:29 AM
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