Skip to content
This repository has been archived by the owner on Jun 11, 2019. It is now read-only.

php package name required by module installation #100

Open
apengue opened this issue Aug 18, 2015 · 1 comment
Open

php package name required by module installation #100

apengue opened this issue Aug 18, 2015 · 1 comment

Comments

@apengue
Copy link

apengue commented Aug 18, 2015

in module.pp, on line 91 (last before closing parenthesis) there is:

require => Package['php'],

it should be

require => Package[$php::package],

otherwise, installing another php package (php55, php56 from another repo (epel/remi as example for Fedora/Centos/RHEL) will cause the module part to be ignored as the require clause will be missed.


update:
same in init.php:
251: package { "${php::package}":
263: require => Package["${php::package}"],

@alvagante
Copy link
Member

Good point, would you mind to submit a PR with the fixes?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants