On Wed, Mar 18, 2009 at 06:31:36AM -0400, Trevor Vaughan wrote: > > I.e. > > class pam-ldap { > include "openldap-client" > } > > If anyone does know of a problem with this, could you shed some light on it?
There are several potential problems with it; the way puppet works, the point at which a class is included can be very important. For example, if the openldap-client class is sensitive to global variables, which might be set differently in different contexts, then this might include the class early, before those have been set. It also forces the inclusion of that specific class, rather than any class that might inherit from it. So it all hangs on the way the OP uses these classes, but I do find it very important in puppet to avoid unnecessary dependences. -- Bruce Hierophant: someone who remembers, when you are on the way down, everything you did to them on the way up. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Puppet Users" group. To post to this group, send email to puppet-users@googlegroups.com To unsubscribe from this group, send email to puppet-users+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/puppet-users?hl=en -~----------~----~----~----~------~----~------~--~---