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

Localisation vs Musl (glibc locales, iconv, icu and anything related) #39

Closed
ChristianYeah opened this issue Apr 23, 2018 · 6 comments
Closed

Comments

@ChristianYeah
Copy link

ChristianYeah commented Apr 23, 2018

Original Title of the Issue: wrong charset in iconv

Hi, I'm using php spreadsheet to read some csv, when I set the input encoding, I will get the error below
iconv(): Wrong charset, conversion from `gbk' to `UTF-8//IGNORE//TRANSLIT' is not allowed

I rebuilt the iconv ext with gnu libiconv and this works

Is it possible to add this feature in the future, that will be great

@Andrewsville
Copy link

This is not a problem in this package but much deeper. See docker-library/php#240 (which leads you to akrennmair/newsbeuter#364 (comment) which leads you to https://wiki.musl-libc.org/functional-differences-from-glibc.html#iconv which says that musl (used by Alpine Linux) does not support transliteration). Anyway, the first mentioned issue contains a working (we use it, too) workaround in comments.

@hernandev
Copy link
Member

Thank you @Andrewsville for the clarification.

Indeed, the problem is way deeper and there's not much of a solution on the packaging step.

I'll look into some kind of workaround that may be embedded, but for now, I'll let this as it is.

Letting this open and changing to Localisation vs Musl

@hernandev hernandev changed the title wrong charset in iconv Localisation vs Musl (glibc locales, iconv, icu and anything related) Aug 26, 2018
@hernandev
Copy link
Member

Hey, anyone did progress on the issue? I'm planning on documenting alternatives on the subject, and make some proposals to change on the core.

Thank you for any feedback and help!

@roynasser
Copy link

I'm interested in following this too. We have some issues related to this for iconv and gettext.

@ShalokShalom
Copy link

Me too.

@hernandev
Copy link
Member

Never actually run into something not fixable via a 3rd party package or some code workaround, and the implications are limited, and I also don't have the skills to make it work.

I have the curiosity of learning it, but currently can't.

So closing it now, as no one had any toughts on it as well.

Thank you all for the interest and support!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants