Um, because Luke told me to. The reasoning, I believe, is that the error is symmetrical (happens at a similar level, etc) to other ParseErrors. I'm not tied to it.
On Tue, Oct 13, 2009 at 6:46 PM, Steven Jenkins <[email protected]> wrote: > > On Tue, Oct 13, 2009 at 9:29 PM, <[email protected]> wrote: >> >> From: Rein Henrichs <[email protected]> >> >> Use Puppet::Util.execute to run the config_version command and reraise >> its potential Puppet::ExecutionFailure exception as a more useful >> Pupppet::ParseError >>... >> - �...@version = %x{#{Puppet[:config_version]}}.chomp >> + �...@version = Puppet::Util.execute([Puppet[:config_version]]).strip >> + >> + rescue Puppet::ExecutionFailure => e >> + raise Puppet::ParseError, "Unable to set config_version: >> #{e.message}" >> end >> > > Why Puppet::ParseError? > > Steven > > > > -- Rein Henrichs http://reductivelabs.com --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---
