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 U affix entry. #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

agmartin2
Copy link

Hi, Sergio,

I am upgrading esperanto Debian package to your version available from https://kovro.heliohost.org/eo/tools/ispelleo.tar.bz2. aspell affix file is generated from original ispell affix file by means of ispellaff2myspell, a conversion tool I wrote a while ago to convert from ispell affix format to old myspell2 affix format that is supported by aspell.

To have aspell work with the resulting file I had to do a couple of things.

  1. Seems aspell does not like the '' flag and triggers an error. I changed it to '!' in .aff and .dic files and seems to go well.

  2. There seems to be a problem with one of the affix rules (U) and aspell complains about it when building eo aspell hash. Error message is

Error: /usr/lib/aspell//eo_affix.dat:406: The condition "u" does not guarantee that "i" can always be stripped.

Looking at 'U' flag I think aspell seems right about this. I would expect this merge request to make 'U' flag behave as expected.

Regards,

Otherwise, when building eo hash aspell complains with

Error: /usr/lib/aspell//eo_affix.dat:406: The condition
"u" does not guarantee that "i" can always be stripped.

and aspell seems right about this.

Signed-off-by: Agustin Martin Domingo <agmartin@debian.org>
@agmartin2
Copy link
Author

Hi, noticed a typo in my above message due to \ being considered an escape char.

  1. Seems aspell does not like the '\' flag and triggers an error. I changed it to '!' in .aff and .dic files and seems to go 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 this pull request may close these issues.

1 participant