[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Failing spelling tests



Hi again,

I've sent that one too quickly. :-)

Axel Beckert wrote:
> > It seems that the command used is "aspell list -l en -p /dev/null" and
> > that Test::Spell has a "set_spell_cmd" that could be (ab)used for this
> > purpose.  I have not tested it - but if it works in and outside a clean
> > chroot, I certainly do not mind having it in the test case.
> 
> Having had a look at /usr/share/perl5/Test/Spelling.pm, I noticed that
> not aspell but spell from the package spell is Test::Spelling's top
> preference:
> 
> sub spellchecker_candidates {
>     # if they've specified a spellchecker, use only that one
>     return $SPELLCHECKER if $SPELLCHECKER;
> 
>     return (
>         'spell', # for back-compat, this is the top candidate ...
>         'aspell list -l en -p /dev/null', # ... but this should become first soon
>         'ispell -l',
>         'hunspell -l',
>     );
> }
[...]
> So having "spell" installed seems what made the lintian build fail.

So yes, hardcoding "aspell list -l en -p /dev/null" via set_spell_cmd
will fix this. And it will fix this in a less invasive way than a
"Build-Conflicts: spell" would do.

Will take care of it. Thanks for the helpful comments!

		Regards, Axel
-- 
 ,''`.  |  Axel Beckert <abe@debian.org>, http://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-    |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE


Reply to: