On Wed Jul 04 10:47:38 2007, radu@yx.ro wrote:
Show quoted text> Makefile.PL requires Cwd 3.19, but this looks superfluous because no
> function exported by Cwd is used internally by DBIx::Class.
>
> Could this requirement of Cwd be removed from Makefile.PL?
> Module::Install, the one using Cwd, already requires it.
>
> The reason I'm pleading for this change is to not be forced to either
> change DBIx-Class Makefile.PL manually or upgrade Cwd manually on
> systems like Red Hat Enterprise Linux 4, where perl 5.8.5 (Cwd version
> 2.19) is used.
>
> Thank you!
The reason for the 3.19 requirement (actually, I think technically we
really only needed it to be 3.18, but that doesn't make any difference
for your case) was load-time performance. Deep within the guts of some
modules used by some modules used by DBIx::Class during the
->load_classes(), Cwd gets used. However, there was a performance bug
in older versions of Cwd that was slowing us down significantly
(specifically, it was shelling out to call the "pwd" command over and
over instead of using the native getcwd() or similar call).
We could make it an optional recommendation, but then the Makefile.PL
would prompt users, which I think we're not fond of causing either
(although CPAN does have facilities for defaulting the answers).
Probably the bigger issue is for us to come up with a clean way to
specify optional feature deps and recommended deps while giving some
environment-variable-based control over whether to prompt at all, and
what the options should default as.