example42 / puppet-perl

Puppet module to manage Perl and CPAN modules
Other
4 stars 26 forks source link

Perl-doc required for modules #1

Open lieutdan13 opened 11 years ago

lieutdan13 commented 11 years ago

The perl-doc package is required to determine if the modules have been installed. Otherwise, the modules exec will attempt to run every time, causing undesired output. I will fix this bug, but before I make ANY changes, I wanted to ask you how you would go about fixing this.

My thoughts: In params.pp, add a $doc_package variable. In init.pp, add a package definition similar to $perl::package.

Let me know what you think.

alvagante commented 11 years ago

That makes sense. This approach is ok for me. I would add the extra quirk that if use supplies an empy $doc_package argument, no doc package is installed.