Control: reassign -1 irussian
Control: tags -1 + patch pending

El sáb, 14 ene 2023 a las 15:39, Jason Lee Quinn
(<jason.lee.quinn+deb...@gmail.com>) escribió:
>
> Package: dictionaries-common
> Version: 1.29.3
> Followup-For: Bug #1028473
> X-Debbugs-Cc: jason.lee.quinn+deb...@gmail.com
>
> Thank you for the reply.
>
> If these dictionaries are installed,
> where are they located? I've searched
> /usr/lib/ispell and many other places can only find
> american and british dictionaries on my machine.

Hi,

You should find relevant files under different dirs. In one of my boxes

$ dir /usr/lib/ispell/ /usr/share/ispell/ /var/lib/ispell/
/var/lib/dictionaries-common/ispell/

/usr/lib/ispell/:
american.aff    castellano.hash  english.aff    espanol.hash
spanish.hash
american.hash    default.aff     espa~nol.aff    README.select-ispell
castellano.aff    default.hash     espa~nol.hash    spanish.aff

/usr/share/ispell/:
american.med+.mwl.gz  american.mwl.gz  english.aff  espa~nol.mwl.gz

/var/lib/dictionaries-common/ispell/:
iamerican  ispanish

/var/lib/ispell/:
american.compat  american.hash    american.remove  espa~nol.compat
espa~nol.hash  espa~nol.remove  README

If there is no trace of those dicts in your dirs, synaptic is
upgrading something else (A virtual machine?)

> Also where does the "contains illegal characters" message
> actually come from? Whatever the source of that messgae,
> I'm having trouble tracking it down. A techincal explaination
> about why the message is harmless would also be interesting
> for me. Perhaps the message itself and the logic that produces
> it could be improved to provide a nicer user experience.

Munched word in line 39 of original russian ispell dict contains
whitespace, which is allowed only as word separator, not in the middle
of a word. Then ispell (and aspell) complains about it and skips that
word, that is the message. This is harmless because its only result is
that word is skipped.

Attached patch should strip that word before package is built, thus
making the message go away. I am reassigning this bug report to
irussian (I am also uploader for it)  and will upload a package with
this fix unless maintainer disagree.

Thanks again for your feedback.

-- 
Agustin
diff --git a/debian/rules b/debian/rules
index 565a92f..2ce6cfc 100755
--- a/debian/rules
+++ b/debian/rules
@@ -28,7 +28,7 @@ export LANG LC_ALL
 override_dh_auto_build:
 	# Generate ispell dictionary.
 	grep -h '[3]' $(DICTIONARIES) | tr '\243\263' '\305\345' > yo_subst.koi
-	cat $(DICTIONARIES) yo_subst.koi |./sortkoi8 | uniq > $(ILANGUAGE).dict
+	cat $(DICTIONARIES) yo_subst.koi |./sortkoi8 | uniq | LC_ALL=C grep -v ' ' > $(ILANGUAGE).dict
 	sed -e "s/^\#[ye]//;s/^\#koi/wordchars/" $(ILANGUAGE).aff.koi > $(ILANGUAGE).aff
 	# Generate traditional ispell hash needed by i2myspell.
 	buildhash $(ILANGUAGE).dict $(ILANGUAGE).aff $(ILANGUAGE).hash

Reply via email to