On Wed, Oct 27, 2010 at 5:57 PM, donavan <[email protected]> wrote:
> Just to follow up on 5112 I have a dirty patch that appears to work.
> Nominally tested it on 10.4, 10.5, & 10.6. 10.4 now applies catalogs
> instead of failing. All versions successfully manage a test services
> state as well.
>
> Does anyone have a better suggestion than '-o /dev/stdout'? Seems a
> mite hacky to me.
I actually think you've come up with a reasonably clean solution given
the hackiness of the underlying tools :)
> Also I think that the 10.4 machines are going to a have a \ ( slash )
> file in whatever puppets working dir was. plutil seems to have been
> interpreting as literal file name.
Can you elaborate? I'm not understanding the problem here.
> --- lib/ruby/site_ruby/1.8/puppet/provider/service/launchd.rb
> +++ lib/ruby/site_ruby/1.8/puppet/provider/service/launchd.rb
> @@ -56,7 +56,7 @@
> # Read a plist, whether its format is XML or in Apple's "binary1"
> # format.
> def self.read_plist(path)
> - Plist::parse_xml(plutil('-convert', 'xml1', '-o', '-', path))
> + Plist::parse_xml(plutil('-convert', 'xml1', '-o', '/dev/stdout',
> path))
> end
>
> # returns a label => path map for either all jobs, or just a single
--
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.
