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

random UTF-8-MAC failures #39

Closed
beckermr opened this issue Dec 6, 2023 · 8 comments · Fixed by #41
Closed

random UTF-8-MAC failures #39

beckermr opened this issue Dec 6, 2023 · 8 comments · Fixed by #41

Comments

@beckermr
Copy link
Member

beckermr commented Dec 6, 2023

Following up on this thread (conda-forge/git-feedstock#50).

We should either package a patch from this project

https://github.com/fumiyas/libiconv-utf8mac

or build this project as the libiconv on osx.

@beckermr
Copy link
Member Author

cc @conda-forge/core any thoughts here?

@jakirkham
Copy link
Member

Has this been reported upstream? If so, are there patches upstream that could be used?

@beckermr
Copy link
Member Author

Yes please read the associated issue in the GitHub feedstock. The upstream devs refuse to incorporate the apple encoding.

@hmaarrfk
Copy link
Contributor

see conda-forge/git-feedstock#50 (comment)

but I don't really understand what UTF-8"-MAC" is? isn't UTF-8 a deterministic encoding of strings? what changes with UTF-8"-MAC"?

@beckermr
Copy link
Member Author

IDK what changes either, but they are different.

@mbargull
Copy link
Member

isn't UTF-8 a deterministic encoding of strings?

The same Unicode string can be represented in different ways.
There different normalizations: https://en.wikipedia.org/wiki/Unicode_normalisation#Normalization


It's always a good thing to look out for what other distributors to.
For Linux I usually refer to Fedora/Debian/Arch's work.
For macOS it's usually Homebrew.

Homebrew has https://github.com/Homebrew/homebrew-core/blob/9911c3276bcc2a8f2e8a8359370c217e43cd8104/Formula/lib/libiconv.rb#L26 which is https://github.com/Homebrew/formula-patches/blob/9be2793af22750a2d1d053af0a9e2257d4ce7f4e/libiconv/patch-utf8mac.diff from 2015 -- i.e., a very low-maintenance patch.
I diff'ed https://github.com/fumiyas/libiconv-utf8mac against upstream and it is essentially the exact same patch.

It's probably reasonable for us to just apply that patch here on osx-* targets.

(N.B.: I'm not a Mac dev or user, so this is only an "outsider" assessment.)

@beckermr
Copy link
Member Author

I think carrying that patch would be fine.

@jakirkham
Copy link
Member

Yeah patching the existing package would be my preference as well

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

Successfully merging a pull request may close this issue.

4 participants