On Apr 7, 2004, at 9:35 AM, John Peacock wrote:'use version' in M::B EU::MM directly find the first line containing an assignment to the scalar $VERSION eval the right side if the result is not a version object, make it one keep the $VERSION as an object for comparison purposes stringify $VERSION only when required (say for the dist filename)
That might be the right way to do it - if so, then we should do the same with versions in 'requires' and 'recommends' and so on.
I'm basically happy with the current shape of version.pm (i.e. I think the major complaints have been laid to rest). But I am not completely comfortable with recommending that it be used in widespread production modules (does M::B qualify as that ;).
That being said, if you are willing to fork your code so we can work on this, I'm game! I suspect trying to integrate version.pm would scare up the remaining edge conditions that I haven't thought of yet.
...One thing that came up in the recent P5P discussion of versions and stringification was the idea that version.pm could offer a custom function which could be used to extract $VERSION from a random module
Would that help?
Yeah, it might. It would at least make it your problem instead of mine. ;-)
Whose code do you think I was going to borrow/steal to put in version.pm? ;-)
John
-- John Peacock Director of Information Research and Technology Rowman & Littlefield Publishing Group 4501 Forbes Boulevard Suite H Lanham, MD 20706 301-459-3366 x.5010 fax 301-429-5748
