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)
Wed, Apr 17, 4:37 PM
Unknown Object (File)
Wed, Apr 17, 3:10 PM
Unknown Object (File)
Sun, Apr 14, 3:21 PM
Unknown Object (File)
Wed, Apr 10, 8:46 PM
Unknown Object (File)
Sat, Mar 30, 10:28 AM
Unknown Object (File)
Thu, Mar 28, 6:40 AM
Unknown Object (File)
Feb 19 2024, 12:08 PM
Unknown Object (File)
Feb 10 2024, 4:47 AM
View All 73 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
Branch
cache2
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/config/check/PhabricatorExtensionsSetupCheck.php:14XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 21770
Build 29708: Run Core Tests
Build 29707: arc lint + arc unit

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 47920

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