Hi,


On Tue, 2012-06-26 at 11:47 +0200, Felix Frank wrote:
> Hi,
> 
> On 06/26/2012 10:56 AM, Stephen Gran wrote:
> > There is one difference that is most likely a bug.  try this manifest:
> > 
> > ./test.pp
> > class { 'wobble': }
> > class { 'wibble': }
> 
> I disagree on this being a bug.
> 
> This syntax is for parameterized classes. Such classes cannot be
> declared more than once, because the declaration binds parameter values
> that cannot be sanely changed at a later point of compilation.
> 
> So multiple usage of this syntax is not supported.
> 
> I'd suggest avoiding it if it's not needed pass actual class parameters.

I disagree that instantiating a class twice with identical (especially
default) parameters should result in an error.  A class is a singleton,
unlike a define, so should not be indexed solely on it's name.  It
should be an error to try to redefine the parameters that you
instantiate a class with, of course.

However, I see that this is probably not a trivial change, and I don't
think it's that big a deal, in practice.  It does mean jumping through
some hoops when you have something like multiple application classes
using a single parameterized configuration class, but it can be lived
with.

Cheers,
-- 
Stephen Gran
Senior Systems Integrator - guardian.co.uk

Please consider the environment before printing this email.
------------------------------------------------------------------
Visit guardian.co.uk - newspaper of the year

www.guardian.co.uk    www.observer.co.uk     www.guardiannews.com 

On your mobile, visit m.guardian.co.uk or download the Guardian
iPhone app www.guardian.co.uk/iphone
 
To save up to 30% when you subscribe to the Guardian and the Observer
visit www.guardian.co.uk/subscriber 
---------------------------------------------------------------------
This e-mail and all attachments are confidential and may also
be privileged. If you are not the named recipient, please notify
the sender and delete the e-mail and all attachments immediately.
Do not disclose the contents to another person. You may not use
the information for any purpose, or store, or copy, it in any way.
 
Guardian News & Media Limited is not liable for any computer
viruses or other material transmitted with or as part of this
e-mail. You should employ virus checking software.

Guardian News & Media Limited

A member of Guardian Media Group plc
Registered Office
PO Box 68164
Kings Place
90 York Way
London
N1P 2AP

Registered in England Number 908396

-- 
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.

Reply via email to