Bug#827502: hunspell integration Consts obsolete?

[Aaron Madlon-Kay]

Hi. I’m a core contributor to OmegaT.

The entire package is years out of date. Those constants haven’t existed since 2011 as best I can tell.

If this package isn’t going to be updated, I would suggest removing it entirely.

-Aaron


On Fri, 17 Jun 2016 09:36:30 +0200 Rene Engelhard <rene at debian.org> wrote:
> Package: omegat
> Version: 2.3.0.1+dfsg-3
> Severity: important
> 
> Hi,
> 
> On Thu, Jun 16, 2016 at 10:40:46PM -0400, Jeremy Bicha wrote:
> > Package:omegat
> > Version: 2.3.0.1+dfsg-4
> > Severity: serious
> > 
> > libhunspell-1.3-0 has been replaced by libhunspell-1.4-0 in Debian
> > unstable and testing.
> > 
> > I took a guess at the bug severity. I believe it's a RC issue because
> > upgraders who have libhunspell-1.3-0 installed will not receive any
> > bugfixes or security updates for that version of the library.
> 
> Even worse; are you sure this hunspell integration is even working? I see
> (both in stable and testing/unstable) only
> 
> --- a/src/org/omegat/util/OConsts.java
> +++ b/src/org/omegat/util/OConsts.java
> @@ -100,7 +100,11 @@
>      public static final String LEARNED_WORD_LIST_FILE_NAME = "learned_words.txt";^M
>  ^M
>      /** The name of the spell checking library */^M
> -    public static final String SPELLCHECKER_LIBRARY_NAME = "hunspell";^M
> +    public static final String SPELLCHECKER_LIBRARY_NAME = "hunspell-1.2";^M
> +^M
> +    /** directory of system dictionaries */^M
> +    public static final String SPELLCHECKER_SYSTEM_DICTIONARY_DIRECTORY =^M
> +        "/usr/share/myspell/dicts";^M
>  ^M
>      /** the native library directory */^M
>      public static final String NATIVE_LIBRARY_DIR = "native";^M
> 
> patched in.
> 
> SPELLCHECKER_LIBRARY_NAME coul dhave stayed hunspell to be compatible,
> hunspell-1.2 is long gone (even wheezy has 1.3) and /usr/share/myspell/dicts
> is compat stuff and maybe should be changed to /usr/share/hunspell/dicts?
> 
> (And yeah, I missed this package in the transition because it wasn't
> a Depends:)
> 
> Regards,
> 
> Rene
> 
>