On Sat, Mar 27, 2010 at 1:20 PM, Markus Roberts <[email protected]> wrote: >>>> I'd much prefer the groups= method have this behaviour in it, rather than >>>> have it done at loading time. >>>> >>>> Everyone loads this util module but not everyone uses the groups= method, >>>> so >>>> just having the code in groups= will make testing/debugging much easier and >>>> just generally make the system simpler. >>> >>> I don't think I quite get what you mean. >>> >>> The groups= method gets delegated to Process.groups= ? It isn't >>> actually defined for this class at all normally. >> >> Oh now I think I know what you mean. You're saying we should always >> define self.groups= here and inside it decide whether or not to >> delegate it to Process or stub it out for OS X 10.6 ? >> >> I'm not entirely clear as to why that would be better though. > > Because it keeps the solution close to the problem. If you're working > around a problem in something you call out to, the idea is to keep the > work around as localized to the call-out as you can. Likewise, > dealing with questionable input should be done as close to where it > comes in as possible, etc. Border guards should be stationed near the > border.
Ok. For symmetry's sake, should :groups also be manually defined and delegated? Something bothers me about doing this for :groups= and not :groups. > > -- Markus > > -- > You received this message because you are subscribed to the Google Groups > "Puppet Developers" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]. > For more options, visit this group at > http://groups.google.com/group/puppet-dev?hl=en. > > -- nigel -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
