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.

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

Fix #1698 and #3520 in another way #3921

Closed
wants to merge 2 commits into from
Closed

Conversation

k-takata
Copy link
Member

@k-takata k-takata commented Feb 8, 2019

#3520 uses ANSI API, then converts the results to 'encoding'.
However, it doesn't work when a user uses Unicode characters in his account name.

This reverts 8.1.0879 (and 8.1.0882) and uses Unicode API then coverts to 'encoding' to fix #1698.

@brammool brammool closed this in ec0f50a Feb 10, 2019
@k-takata k-takata deleted the fix-1698-3520 branch February 11, 2019 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

system() function doesn't support Unicode user name
1 participant