packager.rb in omnibus-5.6.0

- old
+ new

@@ -24,10 +24,11 @@
     autoload :DEB,      "omnibus/packagers/deb"
     autoload :Makeself, "omnibus/packagers/makeself"
     autoload :MSI,      "omnibus/packagers/msi"
     autoload :APPX,     "omnibus/packagers/appx"
     autoload :PKG,      "omnibus/packagers/pkg"
+    autoload :PKGSRC,   "omnibus/packagers/pkgsrc"
     autoload :Solaris,  "omnibus/packagers/solaris"
     autoload :IPS,      "omnibus/packagers/ips"
     autoload :RPM,      "omnibus/packagers/rpm"
 
     #
@@ -45,10 +46,11 @@
       "aix"      => BFF,
       "solaris"  => Solaris,
       "ips"      => IPS,
       "windows"  => [MSI, APPX],
       "mac_os_x" => PKG,
+      "smartos"  => PKGSRC,
     }.freeze
 
     #
     # Determine the packager(s) for the current system. This method returns the
     # class, not an instance of the class.
@@ -71,10 +73,10 @@
         package_types = klass.is_a?(Array) ? klass : [ klass ]
 
         if package_types.include?(APPX) &&
             !Chef::Sugar::Constraints::Version.new(version).satisfies?(">= 6.2")
           log.warn(log_key) { "APPX generation is only supported on Windows versions 2012 and above" }
-          package_types = package_types - [APPX]
+          package_types -= [APPX]
         end
 
         package_types
       else
         log.warn(log_key) do