Reviews by Nigel Horne



IP-China (0.03) ***

There's no need for this module. You can already do the same thing with:

use CGI::Lingua;

if(CGI::Lingua->new()->country() eq 'cn') {

print "In China\n";

Mail-Sendmail (0.79) ***

It works, but it doesn't do what it says on the tin. The module's name implies that it uses sendmail to send mail, it does not it talks RFC(2)821 to a given server.

And its lack of support for SMTP AUTH is a draw back for some of us forced to use 3rd party sites for hosting.

LWP-UserAgent-Cached (0.04)

Why not use http::cache::transparent? It does the same thing and this seems to re-invent the wheel.

YAML-Syck (1.22) *****

More portable to old versions of Perl than YAML::XS - so I'm sticking with YAML::Syck since it "just works". Thanks!

Mail-Milter (0.07) *

From what little the documentation gives away this may be a useful module. However the lack of documentation is prohibitive for its use, so I'll stick with Sendmail::Milter which has a good example in its synopsis.

What it needs is a simple HOW-TO. The documentation doesn't even tell you how to set the TCP port for your milter to listen on.

HTML-Clean (0.8) **

Nice idea, but buggy and unsupported. I can only now suggest this module is deleted or someone takes it over.

CGI-Cache (1.4201) ***

Nice idea - but I can't help but think it would be better to be re-written to use CHI.

App-perlbrew (0.25) *

Nice idea, but basically it doesn't work. I tried it on a couple of different platforms (one Linux and the other Darwin) and try as I might I always came across problems, usually associated with paths, perl5libs and the like not being set up so e.g. cpan installations wouldn't work or the wrong version of Perl was used.

Once the bugs are fixed I'm sure it'll be useful, but more work should be put into testing before (a) it's released and (b) new features are added.

Locale-SubCountry (1.45) *****

Excellent and useful software. Thanks.

Email-Fingerprint (0.29) **

Nice idea, but until it can get over its build problems and lack of portability, I can't use it

HTML-CheckArgs (0.11) *

Hasn't been changed for a number of years and it fails many installation tests. Is it still being maintained? If not then either it should be deleted or be handed over to someone else.

Sys-CPU (0.52) **

Pointless reimplementation of Sys::Info::Device::CPU

HTTP-Cache-Transparent (1.0) ****

It works on simple applications, but on enterprise level programs it soon shows that it hasn't been tested that much and it has too many bugs to allow it to be rolled out on important applications.

The time taken for bugs to be fixed also causes concern for its use on mission critical programs.

Devel-NYTProf (4.04) ****

It took some work to find out how to profile my code as well as CPAN modules, but once I worked it out, the code works well.

Mail-Decency (0.1.5) *

Since it is Postfix only, it should have been called Postfix::Decency. The name Mail::Decency is misleading and shouldn't have been accepted.

Mac-Carbon (0.82) *

Doesn't compile and bugs reported that it doesn't compile sit around for ages, so I don't think it's supported any more.

Probably should be deleted.

Test-WWW-Selenium (1.21) *

Doesn't work. Even a test program fails because the Java backend can't start Firefox. I guess this is more Selenium problem than a T:W:S problem, but without a working back-end what's the point of a front-end?

Net-Whois-IANA (0.23) **

Too much is hard coded so that it dies (literally) if the remote end (which can't be changed) can't be contacted.

It needs more testing, and now that it's 2 years since the last release I wonder if it's still supported.

Sendmail-Milter (0.18) *

Needs to have access to the sendmail source to install, which in this day and age shouldn't be needed.

HTML-FormatData (0.10) *

Fails to install and it hasn't been touched since 2005 - perhaps support has lapsed and it should be dropped.

TermReadKey (2.30)

Fails to install - my bug report from months ago has been ignored. Perhaps development is no longer active?

Mail-ClamAV (0.17) *

Fails to install on Linux/PPC


Starting "make" Stage
make[1]: Entering directory `/root/.cpan/build/Mail-ClamAV-0.17/_Inline/build/Mail/ClamAV'
/usr/bin/perl /usr/lib/perl5/5.8.3/ExtUtils/xsubpp -typemap /usr/lib/perl5/5.8.3/ExtUtils/typemap ClamAV.xs > ClamAV.xsc && mv ClamAV.xsc ClamAV.c
gcc -c -I/root/.cpan/build/Mail-ClamAV-0.17 -I/usr/local/include -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBUGGING -fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -I/usr/include/gdbm -O2 -g -pipe -fsigned-char -DVERSION=\"0.17\" -DXS_VERSION=\"0.17\" -fPIC "-I/usr/lib/perl5/5.8.3/ppc-linux-thread-multi/CORE" ClamAV.c
ClamAV.xs: In function `clamav_perl_constant':
ClamAV.xs:334: error: `CL_SCAN_DISABLERAR' undeclared (first use in this function)
ClamAV.xs:334: error: (Each undeclared identifier is reported only once
ClamAV.xs:334: error: for each function it appears in.)
make[1]: *** [ClamAV.o] Error 1
make[1]: Leaving directory `/root/.cpan/build/Mail-ClamAV-0.17/_Inline/build/Mail/ClamAV'

A problem was encountered while attempting to compile and install your Inline
C code. The command that failed was:


The build directory was:

To debug the problem, cd to the build directory, and inspect the output files.

at /root/.cpan/build/Mail-ClamAV-0.17/blib/lib/Mail/ line 537
BEGIN failed--compilation aborted at /root/.cpan/build/Mail-ClamAV-0.17/blib/lib/Mail/ line 537.
Compilation failed in require.
BEGIN failed--compilation aborted.
make: *** [ClamAV.inl] Error 2

/usr/bin/make -- NOT OK
Running make test

Can't test without successful make
Running make install

make had returned bad status, install seems impossible


Text-Aspell (0.04) *

Fails to install on Linux/x86 (Fedora Core 3):
pan> install Text::Aspell
CPAN: Storable loaded ok
Going to read /root/.cpan/Metadata

Database was generated on Sun, 22 May 2005 15:06:23 GMT
Running install for module Text::Aspell
Running make for H/HA/HANK/Text-Aspell-0.04.tar.gz
CPAN: Digest::MD5 loaded ok
CPAN: Compress::Zlib loaded ok
Checksum for /root/.cpan/sources/authors/id/H/HA/HANK/Text-Aspell-0.04.tar.gz ok
Scanning cache /root/.cpan/build for sizes
Removing previously used /root/.cpan/build/Text-Aspell-0.04 Going to build H/HA/HANK/Text-Aspell-0.04.tar.gz

Checking if your kit is complete...
Looks good
Writing Makefile for Text::Aspell
cp blib/lib/Text/
/usr/bin/perl /usr/lib/perl5/5.8.5/ExtUtils/xsubpp -noprototypes -typemap /usr/lib/perl5/5.8.5/ExtUtils/typemap -typemap typemap Aspell.xs > Aspell.xsc && mv Aspell.xsc Aspell.c
gcc -c -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBUGGING -fno-strict-aliasing -pipe -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -I/usr/include/gdbm -O2 -g -pipe -m32 -march=i386 -mtune=pentium4 -DVERSION=\"0.04\" -DXS_VERSION=\"0.04\" -fPIC "-I/usr/lib/perl5/5.8.5/i386-linux-thread-multi/CORE" Aspell.c
Running Mkbootstrap for Text::Aspell ()
chmod 644
rm -f blib/arch/auto/Text/Aspell/
gcc -shared -L/usr/local/lib Aspell.o -o blib/arch/auto/Text/Aspell/ -laspell
chmod 755 blib/arch/auto/Text/Aspell/
cp blib/arch/auto/Text/Aspell/
chmod 644 blib/arch/auto/Text/Aspell/
Manifying blib/man3/Text::Aspell.3pm

/usr/bin/make -- OK
Running make test
PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0,'blib/lib', 'blib/arch')" t/*.t
t/test....FAILED test 7

Failed 1/21 tests, 95.24% okay
Failed Test Stat Wstat Total Fail Failed List of Failed
t/test.t 21 1 4.76% 7
Failed 1/1 test scripts, 0.00% okay. 1/21 subtests failed, 95.24% okay.
make: *** [test_dynamic] Error 255

/usr/bin/make test -- NOT OK
Running make install

make test had returned bad status, won't install without force