2008/10/4 James Keenan via RT <[EMAIL PROTECTED]>: > On Fri Oct 03 08:55:12 2008, coke wrote: >> This is a patch-response to r31585, which required a particular >> version of Storable in one file. >> >> Attached, find a patch which bumps our core perl requirement to 5.8.6, >> the first time this particular version of Storable was included as a >> core module (`corelist -a Storable`). It: >> [snip] >> We've been talking about doing this bump to avoid this problem for >> some time. I'd rather fail during the initial attempt to run configure >> than have someone with 5.8.0 -then- fail by missing the Storable >> requirement below. >> > > I recall some discussion about going to 5.8.4 -- but not 5.8.6. 5.8.4 > is the level of Perl included in many distributions such as Debian. Are > you really sure you want to go to 5.8.6 just for one module? Why not > require that version of Storable directly?
I'm the one who did the change to force a better error message if lower than Storable 2.13 As I recall 5.8.4 adds the core overload support so that Storable-2.13 or greater works. But I'm not sure on my laptop and on my trip I have only limited ressources to check it out again. Tying down the specific perl version would be of course better in the discussed patch, but removing my patch will not help. Storable-2.13 is still required and should be marked as such. -- Reini Urban http://phpwiki.org/ http://murbreak.at/
