On Jul 22, 2007, at 11:34 PM, David Golden wrote:

[Patch is against the current SVN head]

This time, test-first development bit me in the a**.  Making sense of
compat.t and cleaning it up so I could (a) figure out how to add the
tests I wanted (with and without prerequisites) and (b) keep track of
what was breaking along the way took about 100 times more effort than
having compat.t add 'require 5.XXXXXX' when it should.  But maybe it
will provide for an easier experience for the next person to hack on
Compat.pm.

Along the way, I found that perl_version_to_float() never checked to
see if it was a float to begin with, which turned '5.008008' into
'5.8008'.  Fixed in this patch as well.

Note -- this patch does *not* put 'perl' in the PREREQ_PM, as the
other thread seemed to be leading towards a new, improved keyword for
a minimum perl version rather than continuing to use requires and
EU::MM doesn't support it anyway.

Comments and critiques welcome.


No comments, no critiques - applied.  Thanks.

 -Ken

Reply via email to