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