Re: [Puppet Users] Going insane : Puppet PHP module for multiple distributions

2012-03-30 Thread Aaron Grewell
If you try to handle unlike OSes in the same class you're in for lots of debugging and logic issues. I agree with Walter, you'll want to subclass them. If you name your subclasses according to the results of the built-in osfamily fact (which should return RedHat for both RHEL and CentOS) you can

Re: [Puppet Users] Going insane : Puppet PHP module for multiple distributions

2012-03-29 Thread Walter Heck
>From what I've seen in other modules, the more common way is to do something like this: class php {    case $::operatingsystem {        'Gentoo': {                include php::gentoo        }        /(RedHat|CentOS)/: {                include php::redhat        } } And then implement the c

[Puppet Users] Going insane : Puppet PHP module for multiple distributions

2012-03-29 Thread Matthias Saou
Hi, Before making any further changes to my existing PHP module, I'd like to get a few recommendations from anyone who has already faced this problem... My existing module supports Fedora, RHEL and clones. I'd like to extend support to more distributions, by adding a typical "params" class to it.