RSS | Module Info | Add a review of

Pod-Simple (3.07)

TBH I'm just using this to auto-generate pod using the batch script - a simple perl -MPod::Simple::HTMLBatch -e Pod::Simple::HTMLBatch::go /path/to/your/modules output_dir works fine.

We've used the cpan-tastic css sheet recommended by John and it's lovely. Unfortunately it only splits modules on a single level, so we get 90% of our modules in one big lump. Otherwise, highly recommended!

adam - 2008-10-07 05:36:38
Was this review helpful to you?  Yes No

ack (1.80)

Lovely grep -r -alike tool. In daily use here too. I like the way that "ack foo | grep bar" reverts back to including filenames on each line to make it more usable, and also that it skips .svn dirs.

Ack FTW!

adam - 2008-07-03 06:32:00
Was this review helpful to you?  Yes No

XML-GDOME (0.86)

Obsolete module - only works with a single version of GDome, which is now out of date. I suggest using an alternative XML library, eg XML::LibXML.

adam - 2006-02-20 02:01:41
Was this review helpful to you?  Yes No

HTML-Tidy (1.06)

As above, the module may be nice but fails tests. I believe this is because the tidylib has been updated but the perl module hasn't kept in step.

The problem in the test is due to the address tag in HTML - AFAIK it can't be nested or have a <center> tag inside it but an older version of tidylib allowed this, and the enclosed test tickles this particular case quite hard.

Installed OK from source after compiling tidylib from source (CVS, 25jan06) and tweaking the venus.t to take out the extra address tags.

As per previous commenter, it's a reet pain that you can't specify parameters in any sane way and have to put them in an external file.

You basically get two methods: parse and clean.

parse takes a dummy $filename first parameter, which is a bit odd - it will only parse strings, and $tidy->parse( $string ) doesn't work. It returns true even if there are errors in the HTML, which is also odd (truth is based on the success or failure of being able to get libtidy to look at the string)

I found the following code to be useful, but it does use a the private _tidy_messages interface...

my $errs = HTML::Tidy::_tidy_messages( $html );
if ($errs) {
print "there were errors\n";
#$errors is a listref of strings
}

The clean() method is much more sensible. my $cleaned = $tidy->clean( $html ); works as expected and returns the cleaned HTML (even though the docs say "Returns true if all went OK")

adam - 2006-01-25 08:39:53
Was this review helpful to you?  Yes No

SVN-Notify (2.51)

Docs are a little sketchy - no mention of HTML::ColorDiff in the perldocs, for example.

Would be nice to have a sensible example post-commit example using the available features, eg

svnnotify --repos-path "$1" --revision "$2" \
--with-diff \
--svnlook /usr/bin/svnlook \
--user-domain mydomain.com \
--to coders@mydomain.com \
--subject-prefix [SVN] \
--subject_cx \
--strip-cx-regex '^trunk/' \
--handler HTML::ColorDiff \
--svnweb-url 'http://myserver/websvn/listing.php?rev=%s'

ColorDiffs aren't quite as nice as cvsspam - it doesn't highlight differences within a line.

Otherwise very good - thanks a million!

adam - 2006-01-20 08:23:31
Was this review helpful to you?  Yes No

Test-MockDBI (0.61)

Useful, but limited.

Forced logging to stdout with no way to turn it off. No support for connect_cached or any of the hash fetching functions (eg fetchrow_hashref).

Interface is basic - the docs more or less encourage you to explicitly set the sql you're waiting for and return a fixed value when it is run. It *is* possible to do regexp matching on sql and return the result of a coderef, but it doesn't pass the sql to the code, so its use is limited.

The code is readable, but indenting is off, and it "use"s Data::Dumper twice :-)

adam - 2005-09-28 09:14:24
Was this review helpful to you?  Yes No

Perl.org sites : bugs | dev | history | jobs | learn | lists | use © Copyright 2002-2008 The Perl Foundation Site Information and Contacts