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
MIME-Version: 1.0
X-Mailer: MIME-tools 5.418 (Entity 5.418)
X-RT-Original-Encoding: utf-8
Content-Type: multipart/mixed; boundary="----------=_1187911274-14648-7"
Content-Length: 0
Content-Type: text/plain; charset="utf8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
Content-Length: 788
Download (untitled) / with headers
text/plain 788b
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
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="----------=_1187911274-14648-6"
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Length: 0
Content-Type: text/plain; charset="utf8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 0
Content-Type: text/plain; charset="utf-8"; name="diff.text"
Content-Disposition: inline; filename="diff.text"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: ascii
Content-Length: 115
Download diff.text
text/plain 115b
47c47 < set_spell_cmd('aspell -l --lang=en'); --- > set_spell_cmd('aspell list --lang=en'); 74a75 > Caputo
MIME-Version: 1.0
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Disposition: inline
Message-Id: <rt-3.6.HEAD-14634-1187924006-1507.28966-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf8"
Content-Transfer-Encoding: binary
From: JDHEDDEN [...] cpan.org
X-RT-Original-Encoding: utf-8
X-RT-Original-Encoding: utf-8
Content-Length: 1067
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.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.