Item8870: Change default dependency type from perl
to external
so unspecified code is not loaded.
Priority: Normal
Current State: Closed
Released In: 1.1.0
Target Release: minor
Applies To: Engine
Component: Configure
Branches:
IF a dependency is missing the "Type" field, dependency.pm assumes that the module is type "perl" - implying a Foswiki/TWiki extension and will "eval" the module to determine if it is installed.
It would be safer to only issue the eval for dependencies explicitly marked cpan or perl, and assume external if the type is unknown.
It would probably also be preferable if
BuildContrib warned that the dependency type was not set when building a release.
--
GeorgeClark - 09 Apr 2010
Also, it would be helpful if Dependency.pm reported the type of module it determined to not be installed. Adding module type to the module message.
--
GeorgeClark - 09 Apr 2010