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