Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

require iconv and mbstring extensions #269

Merged
merged 5 commits into from
May 20, 2017
Merged

require iconv and mbstring extensions #269

merged 5 commits into from
May 20, 2017

Conversation

glensc
Copy link
Member

@glensc glensc commented May 20, 2017 •
edited

as zend/mail and our mime_helper require unconditionally iconv extension
and mbstring is also used unconditionally in some places in Eventum.

@glensc
Misc::countBytes: require mbstring
9a137cc
@glensc
Mime_Helper::encodeQuotedPrintable require iconv
5166a01
@glensc
update changelog
195085c 
as iconv/mbstring were already required (or fatal error occurred),
no special note needed
@glensc glensc changed the title WIP: require iconv and mbstring extensions require iconv and mbstring extensions May 20, 2017
@glensc glensc added this to the 3.2.1 milestone May 20, 2017
@glensc glensc merged commit d9b8cf7 into master May 20, 2017
@glensc glensc deleted the require-iconv-mbstring branch May 20, 2017 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
3.2.1
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@glensc