Skip Menu |

This queue is for tickets about the threads CPAN distribution.

Report information
The Basics
Id: 28966
Status: resolved
Priority: 0/
Queue: threads

People
Owner: Nobody in particular
Requestors: cogent [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.64
Fixed in: 1.65



Subject: Bug in test suite for threads.pm
in t/pod.t, you use Test::Spelling. On line 47 of t/pod.t you have the following subroutine call: set_spell_cmd('aspell -l --lang=en'); There are at least two problems: 1. aspell doesn't appear to be okay with that syntax. -l and --lang are in fact synonymous, as far as I can tell. I think you want, instead: set_spell_cmd('aspell list --lang=en'); 2. With that change made, aspell finds a spelling error. The word "Caputo" is considered to be misspelled. Perhaps you've added "Caputo" to your local dictionary? Certainly this doesn't appear to cause many test failures, presumably because Test::Spelling is probably not extremely widely installed. Attached is a diff which changes 'aspell -l' to 'aspell list' and adds Caputo to the list of words at the end of t/pod.t
Subject: diff.text
47c47 < set_spell_cmd('aspell -l --lang=en'); --- > set_spell_cmd('aspell list --lang=en'); 74a75 > Caputo
From: JDHEDDEN [...] cpan.org
On Thu Aug 23 19:21:17 2007, COGENT wrote: Show quoted text
> in t/pod.t, you use Test::Spelling. On line 47 of t/pod.t you have the > following subroutine call: > set_spell_cmd('aspell -l --lang=en'); > > There are at least two problems: > 1. aspell doesn't appear to be okay with that syntax. -l and --lang > are in fact synonymous, as far as I can tell. I think you want, instead: > set_spell_cmd('aspell list --lang=en');
Using aspell 0.50.3, 'aspell help' shows: -l|list produce a list of misspelled words from standard input and --lang=<str> language code I don't profess to be expert as aspell (I only copied this from another module), but I'll change the syntax as you suggest. Show quoted text
> 2. With that change made, aspell finds a spelling error. The word > "Caputo" is considered to be misspelled. Perhaps you've added "Caputo" > to your local dictionary?
Interesting. It doesn't come up with my dictionary (Aspell-en-0.50-2-3), but I'll add it. I'm gonna as a few other names (like Rocco) to the list, too. Thanks for the report.