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

Update unidecode recipe #5351

Merged
merged 1 commit into from Mar 11, 2018
Merged

Update unidecode recipe #5351

merged 1 commit into from Mar 11, 2018

Conversation

johnmastro
Copy link
Contributor

Brief summary of what the package does

Unidecode provides functions to transliterate Unicode text to ASCII. It is a reimplementation of the Python module of the same name.

Direct link to the package repository

https://github.com/sindikat/unidecode

Your association with the package

I am a contributor.

Relevant communications with the upstream package maintainer

The author/maintainer, @sindikat, merged a pull request from me which requires these changes.

Checklist

Please confirm with x:

@purcell
Copy link
Member

purcell commented Mar 18, 2018

Late to this, but I'd be amazed if Emacs doesn't already have functionality equivalent to this in the standard library. :-)

BTW, the recipe could have been defined slightly more simply with :files (:defaults "data").

purcell added a commit that referenced this pull request Mar 18, 2018
@johnmastro
Copy link
Contributor Author

Thanks for pushing the simplification!

I asked on help-gnu-emacs yesterday whether Emacs has equivalent functionality built-in and Eli Zaretskii responded. He says that ucs-normalize.el can be used to remove accents, and some scripts have input methods that support transliteration, but nothing as general as Unidecode.

@purcell
Copy link
Member

purcell commented Mar 25, 2018

@johnmastro Good to know. This certainly looks useful - thanks for sharing.

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 this pull request may close these issues.

None yet

3 participants