Merge branch 'COOK-584'

dmg/README.md

@@ -1,23 +1,35 @@
 Description
 ===========
 
-Lightweight resource and provider to install OS X applications (.app) from dmg files
+Lightweight resource and provider to install OS X applications (.app) from dmg files.
+
+Changes
+=======
+
+## v0.6.0:
+
+- option to install software that is an .mpkg inside a .dmg
+- ignore failure on chmod in case mode is already set, or is root owned
 
 Requirements
 ============
 
-Platform: Mac OS X
+## Platform:
+
+* Mac OS X
 
 Resources and Providers
 =======================
 
-`dmg_package`
--------------
+dmg\_package
+------------
 
 This resource will install a DMG "Package". It will retrieve the DMG from a remote URL, mount it using OS X's `hdid`, copy the application (.app directory) to the specified destination (/Applications), and detach the image using `hdiutil`. The dmg file will be stored in the `Chef::Config[:file_cache_path]`. If you want to install an application that has already been downloaded (not using the `source` parameter), copy it to the appropriate location. You can find out what directory this is with the following command on the node to run chef:
 
     knife exec -E 'p Chef::Config[:file_cache_path]' -c /etc/chef/client.rb
 
+Optionally, the LWRP can install an "mpkg" package using installer(8).
+
 # Actions:
 
 * :install - Installs the application.
@@ -28,6 +40,7 @@ This resource will install a DMG "Package". It will retrieve the DMG from a remo
 * `source` - remote URL for the dmg to download if specified. Default is nil.
 * `destination` - directory to copy the .app into. Default is /Applications.
 * `checksum` - sha256 checksum of the dmg to download. Default is nil.
+* `type` - type of package, "app" or "mpkg". Default is "app". When using "mpkg", the destination must be /Applications.
 * `volumes_dir` - Directory under /Volumes where the dmg is mounted. Not all dmgs are mounted into a /Volumes location matching the name of the dmg. If not specified, this will use the name attribute.
 * `dmg_name` - Specify the name of the dmg if it is not the same as `app`, or if the name has spaces.
 
@@ -69,6 +82,13 @@ Install MacIrssi to `~/Applications` from the local file downloaded to the cache
       action :install
     end
 
+Install Virtualbox to `/Applications` from the .mpkg:
+
+    dmg_package "Virtualbox" do
+      source "http://dlc.sun.com.edgesuite.net/virtualbox/4.0.8/VirtualBox-4.0.8-71778-OSX.dmg"
+      type "mpkg"
+    end
+
 To do
 =====
 
@@ -82,7 +102,6 @@ Some things that would be nice to have at some point.
 * use hdiutil to mount/attach the disk image
 * automatically detect the `volumes_dir` where the image is attached
 * be able to automatically accept license agreements
-* install software that is a .pkg inside a .dmg instead of just .app's
 
 License and Author
 ==================

dmg/metadata.rb

@@ -3,5 +3,5 @@
 license          "Apache 2.0"
 description      "LWRP to install OS X applications from dmgs"
 long_description IO.read(File.join(File.dirname(__FILE__), 'README.md'))
-version          "0.5.2"
+version          "0.6.0"
 supports         "mac_os_x"

dmg/providers/package.rb

@@ -43,11 +43,18 @@ def load_current_resource
       not_if "hdiutil info | grep -q 'image-path.*#{dmg_file}'"
     end
 
-    execute "cp -r '/Volumes/#{volumes_dir}/#{new_resource.app}.app' '#{new_resource.destination}'"
+    case new_resource.type
+    when "app"
+      execute "cp -r '/Volumes/#{volumes_dir}/#{new_resource.app}.app' '#{new_resource.destination}'"
+    when "mpkg"
+      execute "sudo installer -pkg /Volumes/#{volumes_dir}/#{new_resource.app}.mpkg -target /"
+    end
+
     execute "hdiutil detach '/Volumes/#{volumes_dir}'"
 
     file "#{new_resource.destination}/#{new_resource.app}.app/Contents/MacOS/#{new_resource.app}" do
       mode 0755
+      ignore_failure true
     end
 
   end

dmg/resources/package.rb

@@ -24,6 +24,7 @@
 attribute :checksum, :kind_of => String, :default => nil
 attribute :volumes_dir, :kind_of => String, :default => nil
 attribute :dmg_name, :kind_of => String, :default => nil
+attribute :type, :kind_of => String, :default => "app"
 attribute :installed, :kind_of => [TrueClass, FalseClass], :default => false
 
 def initialize(name, run_context=nil)