vanagon 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0277e8af73f3165f6e9b9dac09f09afbf92257d1
-  data.tar.gz: b6d18711b68b4ca2ce845bb1cae01bbf019dfd90
+  metadata.gz: 6faf9b7a2f17d4a0d10d3e9a050d066ba92ab7a0
+  data.tar.gz: 112f2f90afafb386461ec92ed67238fefddcf76d
 SHA512:
-  metadata.gz: b8cccacd01204b1474944ec8975597369883ea354e71e58724802ab27b2581af7a009dd3ab07fe770fadd121180fb35b53bed8ae6bd409b3e031839c5d353d4d
-  data.tar.gz: a8b4f78d37aa2844f1615b7aded498e6c5ca47c5d34d5a18e229125062928d1c6033d9bb97beb3725a1bfeb2250bf714262af187d2509014b4573e18c95bf338
+  metadata.gz: 8edf1722f845927b0ba7858b077bd3c57908f7395beee779aa068552ea7c5e46d7a827416b201b275fb90c947438162f1a0f42004a85662b1716a8e9f264161f
+  data.tar.gz: 85a5e28c1954033d3120d3e4ee72582f413982762ba430d3e1ebaa0e38127cfb994d3edfdaf921392a76fd7737ff5a6176469b006ff0a63905da653ed37a2607

data/README.md

@@ -106,6 +106,9 @@ A full path on disk for a private ssh key to be used in ssh and rsync
 communications. This will be used instead of whatever defaults are configured
 in .ssh/config.
 
+##### VANAGON\_SSH\_AGENT
+When set, Vanagon will forward the ssh authentication agent connection. 
+
 ##### VMPOOLER\_TOKEN
 Used in conjunction with the pooler engine, this is a token to pass to the
 vmpooler to access the API. Without this token, the default lifetime of vms

data/lib/vanagon/common/pathname.rb

@@ -1,3 +1,5 @@
+require 'pathname'
+
 class Vanagon
   class Common
     class Pathname
@@ -28,7 +30,7 @@ class Vanagon
       #   and exposed through the {#configfile?} method.
       # @return [Vanagon::Common::Pathname] Returns a new Pathname instance.
       def initialize(path, mode: nil, owner: nil, group: nil, config: false)
-        @path = File.expand_path(path)
+        @path = ::Pathname.new(path).cleanpath.to_s
         @mode ||= mode
         @owner ||= owner
         @group ||= group

data/lib/vanagon/component.rb

@@ -10,7 +10,7 @@ class Vanagon
     attr_accessor :name, :version, :source, :url, :configure, :build, :check, :install
     attr_accessor :environment, :extract_with, :dirname, :build_requires, :build_dir
     attr_accessor :settings, :platform, :patches, :requires, :service, :options
-    attr_accessor :directories, :replaces, :provides, :cleanup_source, :environment
+    attr_accessor :directories, :replaces, :provides, :cleanup_source
     attr_accessor :sources, :preinstall_actions, :postinstall_actions
     attr_accessor :preremove_actions, :postremove_actions
 
@@ -109,7 +109,7 @@ class Vanagon
         @source = Vanagon::Component::Source.source(@url, @options, workdir)
         @source.fetch
         @source.verify
-        @extract_with = @source.extract(@platform.tar) if @source.respond_to?(:extract)
+        @extract_with = @source.respond_to?(:extract) ? @source.extract(@platform.tar) : ':'
         @cleanup_source = @source.cleanup if @source.respond_to?(:cleanup)
         @dirname = @source.dirname
 
@@ -120,6 +120,9 @@ class Vanagon
 
         # If there is no source, we don't want to try to change directories, so we just change to the current directory.
         @dirname = './'
+
+        # If there is no source, there is nothing to do to extract
+        @extract_with = ':'
       end
     end
 

data/lib/vanagon/component/dsl.rb

@@ -158,7 +158,8 @@ class Vanagon
       # @param default_file [String] path to the default file relative to the source
       # @param service_name [String] name of the service
       # @param service_type [String] type of the service (network, application, system, etc)
-      def install_service(service_file, default_file = nil, service_name = @component.name, service_type: nil)
+      # @param service_hash [Hash] hash of options to parse for the service
+      def install_service(service_file, default_file = nil, service_name = @component.name, service_type: nil, service_hash: nil)
         case @component.platform.servicetype
         when "sysv"
           target_service_file = File.join(@component.platform.servicedir, service_name)
@@ -183,6 +184,9 @@ class Vanagon
           @component.service = OpenStruct.new(:name => service_name, :service_command => File.read(service_file).chomp)
           # Return here because there is no file to install, just a string read in
           return
+        when "windows"
+          @component.service = OpenStruct.new(name: service_name, options: service_hash)
+          return
         else
           fail "Don't know how to install the #{@component.platform.servicetype}. Please teach #install_service how to do this."
         end
@@ -207,7 +211,7 @@ class Vanagon
       # @param group  [String] group owner of the file
       def install_file(source, target, mode: '0644', owner: nil, group:  nil)
         @component.install << "#{@component.platform.install} -d '#{File.dirname(target)}'"
-        @component.install << "cp -p '#{source}' '#{target}'"
+        @component.install << "#{@component.platform.copy} -p '#{source}' '#{target}'"
         @component.add_file Vanagon::Common::Pathname.file(target, mode: mode, owner: owner, group: group)
       end
 
@@ -241,7 +245,10 @@ class Vanagon
       # @param target [String] path to the desired symlink
       def link(source, target)
         @component.install << "#{@component.platform.install} -d '#{File.dirname(target)}'"
-        @component.install << "ln -s '#{source}' '#{target}'"
+        # Use a bash conditional to only create the link if it doesn't already point to the correct source.
+        # This allows rerunning the install step to be idempotent, rather than failing because the link
+        # already exists.
+        @component.install << "([[ '#{target}' -ef '#{source}' ]] || ln -s '#{source}' '#{target}')"
         @component.add_file Vanagon::Common::Pathname.file(target)
       end
 

data/lib/vanagon/component/source/git.rb

@@ -27,10 +27,10 @@ class Vanagon
         def fetch
           puts "Cloning ref '#{@ref}' from url '#{@url}'"
           Dir.chdir(@workdir) do
-            git('clone', @url)
+            git("clone #{@url}", true)
             Dir.chdir(dirname) do
-              git('checkout', @ref)
-              git('submodule', 'update', '--init', '--recursive')
+              git("checkout #{@ref}", true)
+              git("submodule update --init --recursive", true)
               @version = git_version
             end
           end

data/lib/vanagon/component/source/http.rb

@@ -12,9 +12,6 @@ class Vanagon
         # Extensions for files we intend to unpack during the build
         ARCHIVE_EXTENSIONS = '.tar.gz', '.tgz', '.zip'
 
-        # Extensions for files we aren't going to unpack during the build
-        NON_ARCHIVE_EXTENSIONS = '.gem', '.ru', '.txt', '.conf', '.ini', '.gpg', '.rb', '.sh', '.csh', '.xml', '.vim', '.json', '.service'
-
         # Constructor for the Http source type
         #
         # @param url [String] url of the http source to fetch
@@ -106,11 +103,9 @@ class Vanagon
             when ".zip"
               return "unzip '#{@file}'"
             end
-          elsif NON_ARCHIVE_EXTENSIONS.include?(@extension)
-            # Don't need to unpack gems, ru, txt, conf, ini, gpg
-            return nil
           else
-            fail "Extraction unimplemented for '#{@extension}' in source '#{@file}'. Please teach me."
+            # Extension does not appear to be an archive
+            return ':'
           end
         end
 
@@ -121,24 +116,25 @@ class Vanagon
         def cleanup
           if ARCHIVE_EXTENSIONS.include?(@extension)
             return "rm #{@file}; rm -r #{dirname}"
-          elsif NON_ARCHIVE_EXTENSIONS.include?(@extension)
+          else
             # Because dirname will be ./ here, we don't want to try to nuke it
             return "rm #{@file}"
-          else
-            fail "Don't know how to cleanup for '#{@file}' with extension: '#{@extension}'. Please teach me."
           end
         end
 
         # Returns the extension for @file
         #
         # @return [String] the extension of @file
-        # @raise [RuntimeError] an exception is raised if the extension isn't in the current list
         def get_extension
-          extension_match = @file.match(/.*(#{Regexp.union(ARCHIVE_EXTENSIONS + NON_ARCHIVE_EXTENSIONS)})/)
+          extension_match = @file.match(/.*(#{Regexp.union(ARCHIVE_EXTENSIONS)})/)
           unless extension_match
-            fail "Unrecognized extension for '#{@file}'. Don't know how to extract this format. Please teach me."
+            if @file.split('.').last.include?('.')
+              return '.' +  @file.split('.').last
+            else
+              # This is the case where the file has no extension
+              return @file
+            end
           end
-
           extension_match[1]
         end
 
@@ -149,12 +145,8 @@ class Vanagon
         def dirname
           if ARCHIVE_EXTENSIONS.include?(@extension)
             return @file.chomp(@extension)
-          elsif NON_ARCHIVE_EXTENSIONS.include?(@extension)
-            # Because we cd into the source dir, using ./ here avoids special casing single file
-            # sources in the Makefile
-            return './'
           else
-            fail "Don't know how to guess dirname for '#{@file}' with extension: '#{@extension}'. Please teach me."
+            return './'
           end
         end
       end

data/lib/vanagon/driver.rb

@@ -92,7 +92,7 @@ class Vanagon
       puts e.backtrace.join("\n")
       raise e
     ensure
-      if @engine == "hardware"
+      if @engine.name == "hardware"
         @engine.teardown
       end
   end

data/lib/vanagon/engine/base.rb

@@ -4,13 +4,14 @@ require 'vanagon/errors'
 class Vanagon
   class Engine
     class Base
-      attr_accessor :target, :remote_workdir
+      attr_accessor :target, :remote_workdir, :name
 
       def initialize(platform, target = nil)
         @platform = platform
         @required_attributes = ["ssh_port"]
         @target = target if target
-        @target_user = "root"
+        @target_user = @platform.target_user
+        @name = 'base'
       end
 
       # This method is used to obtain a vm to build upon
@@ -57,7 +58,7 @@ class Vanagon
 
       def retrieve_built_artifact
         FileUtils.mkdir_p("output")
-        Vanagon::Utilities.rsync_from("#{@remote_workdir}/output/*", "#{@target_user}@#{@target}", "output", @platform.ssh_port)
+        Vanagon::Utilities.rsync_from("#{@remote_workdir}/output/*", "#{@target_user}@#{@target}", "output/", @platform.ssh_port)
       end
 
       # Ensures that the platform defines the attributes that the engine needs to function.

data/lib/vanagon/engine/docker.rb

@@ -7,6 +7,7 @@ class Vanagon
       # the docker engine to work
       def initialize(platform, target = nil)
         @docker_cmd = Vanagon::Utilities.find_program_on_path('docker')
+        @name = 'docker'
         super
         @required_attributes << "docker_image"
       end

data/lib/vanagon/engine/hardware.rb

@@ -52,6 +52,7 @@ class Vanagon
 
       def initialize(platform, target)
         Vanagon::Driver.logger.debug "Hardware engine invoked."
+        @name = 'hardware'
         @platform = platform
         @build_hosts = platform.build_hosts
         # Redis is the only backend supported in lock_manager currently

data/lib/vanagon/engine/local.rb

@@ -10,6 +10,7 @@ class Vanagon
       def initialize(platform, target = nil)
         @platform = platform
         @target = "local machine"
+        @name = 'local'
       end
 
       # Dispatches the command for execution

data/lib/vanagon/engine/pooler.rb

@@ -11,6 +11,7 @@ class Vanagon
         @token = load_token
         super
         @required_attributes << "vmpooler_template"
+        @name = 'pooler'
       end
 
       # This method loads the pooler token from one of two locations

data/lib/vanagon/platform.rb

@@ -6,7 +6,7 @@ class Vanagon
     attr_accessor :build_dependencies, :name, :vmpooler_template, :cflags, :ldflags, :settings
     attr_accessor :servicetype, :patch, :architecture, :codename, :os_name, :os_version
     attr_accessor :docker_image, :ssh_port, :rpmbuild, :install, :platform_triple
-    attr_accessor :package_type, :build_hosts
+    attr_accessor :target_user, :package_type, :find, :sort, :build_hosts, :copy
 
     # Platform names currently contain some information about the platform. Fields
     # within the name are delimited by the '-' character, and this regex can be used to
@@ -101,6 +101,10 @@ class Vanagon
       @ssh_port = 22
       @provisioning = []
       @install ||= "install"
+      @target_user ||= "root"
+      @find ||= "find"
+      @sort ||= "sort"
+      @copy ||= "cp"
     end
 
     # This allows instance variables to be accessed using the hash lookup syntax
@@ -226,7 +230,7 @@ class Vanagon
     #
     # @return [true, false] true if it is a windows variety, false otherwise
     def is_windows?
-      return !!@name.match(/^win-.*$/)
+      return !!@name.match(/^windows-.*$/)
     end
 
     # Utility matcher to determine is the platform is a linux variety

data/lib/vanagon/platform/deb.rb

@@ -29,7 +29,7 @@ class Vanagon
 
         # Lots of templates here
         ["changelog", "conffiles", "control", "docs", "dirs", "install", "preinst", "postinst", "postrm", "prerm", "rules"].each do |deb_file|
-          erb_file(File.join(VANAGON_ROOT, "templates/deb/#{deb_file}.erb"), File.join(deb_dir, deb_file), false, { :binding => binding })
+          erb_file(File.join(VANAGON_ROOT, "resources/deb/#{deb_file}.erb"), File.join(deb_dir, deb_file), false, { :binding => binding })
         end
 
         # These could be templates, but their content is static, so that seems weird.

data/lib/vanagon/platform/dsl.rb

@@ -7,6 +7,7 @@ require 'vanagon/platform/rpm/eos'
 require 'vanagon/platform/osx'
 require 'vanagon/platform/solaris_10'
 require 'vanagon/platform/solaris_11'
+require 'vanagon/platform/windows'
 require 'securerandom'
 require 'uri'
 
@@ -43,6 +44,8 @@ class Vanagon
                       Vanagon::Platform::Solaris10.new(@name)
                     when /^solaris-11/
                       Vanagon::Platform::Solaris11.new(@name)
+                    when /^windows-/
+                      Vanagon::Platform::Windows.new(@name)
                     else
                       fail "Platform not implemented for '#{@name}' yet. Please go do so..."
                     end
@@ -97,6 +100,27 @@ class Vanagon
         @platform.package_type = pkg_type
       end
 
+      # Set the path to find for the platform
+      #
+      # @param find_cmd [String] Full path to the find command for the platform
+      def find(find_cmd)
+        @platform.find = find_cmd
+      end
+
+      # Set the path to sort for the platform
+      #
+      # @param sort_cmd [String] Full path to the sort command for the platform
+      def sort(sort_cmd)
+        @platform.sort = sort_cmd
+      end
+
+      # Set the path to copy for the platform
+      #
+      # @param copy_cmd [String] Full path to the copy command for the platform
+      def copy(copy_cmd)
+        @platform.copy = copy_cmd
+      end
+
       # Set the path to rpmbuild for the platform
       #
       # @param rpmbuild_cmd [String] Full path to rpmbuild with arguments to be used by default

data/lib/vanagon/platform/osx.rb

@@ -66,10 +66,10 @@ class Vanagon
         script_dir = File.join(workdir, "scripts")
         FileUtils.mkdir_p(script_dir)
 
-        erb_file(File.join(VANAGON_ROOT, "templates/osx/project-installer.xml.erb"), File.join(workdir, "#{name}-installer.xml"), false, { :binding => binding })
+        erb_file(File.join(VANAGON_ROOT, "resources/osx/project-installer.xml.erb"), File.join(workdir, "#{name}-installer.xml"), false, { :binding => binding })
 
         ["postinstall", "preinstall"].each do |script_file|
-          erb_file(File.join(VANAGON_ROOT, "templates/osx/#{script_file}.erb"), File.join(script_dir, script_file), false, { :binding => binding })
+          erb_file(File.join(VANAGON_ROOT, "resources/osx/#{script_file}.erb"), File.join(script_dir, script_file), false, { :binding => binding })
           FileUtils.chmod 0755, File.join(script_dir, script_file)
         end
 

data/lib/vanagon/platform/rpm.rb

@@ -25,7 +25,7 @@ class Vanagon
       # @param name [String] name of the project
       # @param binding [Binding] binding to use in evaluating the packaging templates
       def generate_packaging_artifacts(workdir, name, binding)
-        erb_file(File.join(VANAGON_ROOT, "templates/rpm/project.spec.erb"), File.join(workdir, "#{name}.spec"), false, { :binding => binding })
+        erb_file(File.join(VANAGON_ROOT, "resources/rpm/project.spec.erb"), File.join(workdir, "#{name}.spec"), false, { :binding => binding })
       end
 
       # Method to derive the package name for the project

data/lib/vanagon/platform/solaris_10.rb

@@ -83,7 +83,7 @@ class Vanagon
         ["pkginfo", "depend", "preinstall", "preremove", "postinstall", "proto"].each do |template|
           target_dir = File.join(workdir, 'packaging')
           FileUtils.mkdir_p(target_dir)
-          erb_file(File.join(VANAGON_ROOT, "templates/solaris/10/#{template}.erb"), File.join(target_dir, template), false, { :binding => binding })
+          erb_file(File.join(VANAGON_ROOT, "resources/solaris/10/#{template}.erb"), File.join(target_dir, template), false, { :binding => binding })
         end
       end
 

data/lib/vanagon/platform/solaris_11.rb

@@ -43,7 +43,7 @@ class Vanagon
       def generate_packaging_artifacts(workdir, name, binding)
         target_dir = File.join(workdir, 'packaging')
         FileUtils.mkdir_p(target_dir)
-        erb_file(File.join(VANAGON_ROOT, "templates/solaris/11/p5m.erb"), File.join(target_dir, "#{name}.p5m"), false, { :binding => binding })
+        erb_file(File.join(VANAGON_ROOT, "resources/solaris/11/p5m.erb"), File.join(target_dir, "#{name}.p5m"), false, { :binding => binding })
       end
 
       # Generate the scripts required to add a group to the package generated.

data/lib/vanagon/platform/windows.rb

@@ -0,0 +1,241 @@
+class Vanagon
+  class Platform
+    class Windows < Vanagon::Platform
+      # The specific bits used to generate a windows package for a given project
+      #
+      # @param project [Vanagon::Project] project to build a windows package of
+      # @return [Array] list of commands required to build a windows package for the given project from a tarball
+      def generate_package(project)
+        case project.platform.package_type
+        when "msi"
+          return generate_msi_package(project)
+        when "nuget"
+          return generate_nuget_package(project)
+        else
+          raise Vanagon::Error, "I don't know how to build package type '#{project.platform.package_type}' for Windows. Teach me?"
+        end
+      end
+
+      # Method to derive the package name for the project
+      #
+      # @param project [Vanagon::Project] project to name
+      # @return [String] name of the windows package for this project
+      def package_name(project)
+        case project.platform.package_type
+        when "msi"
+          return msi_package_name(project)
+        when "nuget"
+          return nuget_package_name(project)
+        else
+          raise Vanagon::Error, "I don't know how to name package type '#{project.platform.package_type}' for Windows. Teach me?"
+        end
+      end
+
+      # Method to generate the files required to build a windows package for the project
+      #
+      # @param workdir [String] working directory to stage the evaluated templates in
+      # @param name [String] name of the project
+      # @param binding [Binding] binding to use in evaluating the packaging templates
+      def generate_packaging_artifacts(workdir, name, binding)
+        case @package_type
+        when "msi"
+          return generate_msi_packaging_artifacts(workdir, name, binding)
+        when "nuget"
+          return generate_nuget_packaging_artifacts(workdir, name, binding)
+        else
+          raise Vanagon::Error, "I don't know how create packaging artifacts for package type '#{project.platform.package_type}' for Windows. Teach me?"
+        end
+      end
+
+      # Method to generate the files required to build an MSI  package for the project
+      #
+      # @param workdir [String] working directory to stage the evaluated templates in
+      # @param name [String] name of the project
+      # @param binding [Binding] binding to use in evaluating the packaging templates
+      def generate_msi_packaging_artifacts(workdir, name, binding)
+        FileUtils.mkdir_p(File.join(workdir, "wix"))
+        erb_file(File.join(VANAGON_ROOT, "resources/windows/wix/service.component.wxs.erb"), File.join(workdir, "wix", "service.#{name}.wxs"), false, { :binding => binding })
+      end
+
+      # Method to generate the files required to build a nuget package for the project
+      #
+      # @param workdir [String] working directory to stage the evaluated templates in
+      # @param name [String] name of the project
+      # @param binding [Binding] binding to use in evaluating the packaging templates
+      def generate_nuget_packaging_artifacts(workdir, name, binding)
+        # nuget templates that do require a name change
+        erb_file(File.join(VANAGON_ROOT, "resources/windows/nuget/project.nuspec.erb"), File.join(workdir, "#{name}.nuspec"), false, { :binding => binding })
+
+        # nuget static resources to be copied into place
+        ["chocolateyInstall.ps1", "chocolateyUninstall.ps1"].each do |win_file|
+          FileUtils.copy(File.join(VANAGON_ROOT, "resources/windows/nuget/#{win_file}"), File.join(workdir, win_file))
+        end
+      end
+
+      # The specific bits used to generate a windows nuget package for a given project
+      # Nexus expects packages to be named #{name}-#{version}.nupkg. However, chocolatey
+      # will generate them to be #{name}.#{version}.nupkg. So, we have to rename the
+      # package after we build it.
+      #
+      # @param project [Vanagon::Project] project to build a nuget package of
+      # @return [Array] list of commands required to build a nuget package for
+      # the given project from a tarball
+      def generate_nuget_package(project)
+        target_dir = project.repo ? output_dir(project.repo) : output_dir
+        ["mkdir -p output/#{target_dir}",
+        "mkdir -p $(tempdir)/#{project.name}/tools",
+        "#{@copy} #{project.name}.nuspec $(tempdir)/#{project.name}/",
+        "#{@copy} chocolateyInstall.ps1 chocolateyUninstall.ps1 $(tempdir)/#{project.name}/tools/",
+        "#{@copy} file-list $(tempdir)/#{project.name}/tools/file-list.txt",
+        "gunzip -c #{project.name}-#{project.version}.tar.gz | '#{@tar}' -C '$(tempdir)/#{project.name}/tools' --strip-components 1 -xf -",
+        "(cd $(tempdir)/#{project.name} ; C:/ProgramData/chocolatey/bin/choco.exe pack #{project.name}.nuspec)",
+        "#{@copy} $(tempdir)/#{project.name}/#{project.name}-#{@architecture}.#{nuget_package_version(project.version, project.release)}.nupkg ./output/#{target_dir}/#{nuget_package_name(project)}"]
+      end
+
+      # The specific bits used to generate a windows msi package for a given project
+      # Have changed this to reflect the overall commands we need to generate the package.
+      # Question - should we break this down into some simpler Make tasks ?
+      # 1. Heat the directory tree to produce the file list
+      # 2. Compile (candle) all the wxs files into wixobj files
+      # 3. Run light to produce the final MSI
+      #
+      # @param project [Vanagon::Project] project to build a msi package of
+      # @return [Array] list of commands required to build an msi package for the given project from a tarball
+      def generate_msi_package(project)
+        target_dir = project.repo ? output_dir(project.repo) : output_dir
+        cg_name = "compfiles"
+        dir_ref = "INSTALLDIR"
+        # Actual array of commands to be written to the Makefile
+        ["mkdir -p output/#{target_dir}",
+        "mkdir -p $(tempdir)/staging",
+        "gunzip -c #{project.name}-#{project.version}.tar.gz | '#{@tar}' -C '$(tempdir)/staging' --strip-components 1 -xf -",
+        # Run the Heat command in a single pass
+        # Heat command documentation at: http://wixtoolset.org/documentation/manual/v3/overview/heat.html
+        #   dir <directory> - Traverse directory to find all sub-files and directories.
+        #   -ke             - Keep Empty directories
+        #   -cg             - Component Group Name
+        #   -gg             - Generate GUIDS now
+        #   -dr             - Directory reference to root directories (cannot contains spaces e.g. -dr MyAppDirRef)
+        #   -sreg           - Suppress registry harvesting.
+        #   -var <variable> - Substitute File/@Source="SourceDir" with a preprocessor or a wix variable
+        "cd $(tempdir); \"$$WIX/bin/heat.exe\" dir staging -v -ke -indent 2 -cg #{cg_name} -gg -dr #{dir_ref} -sreg -var var.StageDir -out wix/#{project.name}-harvest.wxs",
+        ]
+      end
+
+      # Method to derive the msi (Windows Installer) package name for the project.
+      #
+      # @param project [Vanagon::Project] project to name
+      # @return [String] name of the windows package for this project
+      def msi_package_name(project)
+        # Decided to use native project version in hope msi versioning doesn't have same resrictions as nuget
+        "#{project.name}-#{project.version}.#{project.release}-#{@architecture}.msi"
+      end
+
+      # Method to derive the package name for the project.
+      # Neither chocolatey nor nexus know how to deal with architecture, so
+      # we are just pretending it's part of the package name.
+      #
+      # @param project [Vanagon::Project] project to name
+      # @return [String] name of the windows package for this project
+      def nuget_package_name(project)
+        "#{project.name}-#{@architecture}-#{nuget_package_version(project.version, project.release)}.nupkg"
+      end
+
+      # Nuget versioning is awesome!
+      #
+      # Nuget and chocolatey have some expectations about version numbers.
+      #
+      # First, if this is a final package (built from a tag), the version must
+      # only contain digits with each element of the version separated by
+      # periods.
+      #
+      # If we are creating the version for a prerelease package (built from a
+      # commit that does not have a corresponding tag), we have the
+      # option to append a string to the version. The string must start with a
+      # letter, be separated from the rest of the version with a dash, and
+      # contain no punctuation.
+      #
+      # We assume we are working from a semver tag as the base of our version.
+      # If this is a final release, we only have to worry about that tag. We
+      # can also include the release number in the package version. If this is
+      # a prerelease package, then we assume we have a semver compliant tag,
+      # followed by the number of commits beyond the tag and the short sha of
+      # the latest change. Because we are working with git, if the version
+      # contains a short sha, it will begin with 'g'. We check for this to
+      # determine what version type to deliver.
+      #
+      # Examples of final versions:
+      #   1.2.3
+      #   1.5.3.1
+      #
+      # Examples of prerelease versions:
+      #   1.2.3.1234-g124dm9302
+      #   3.2.5.23-gd329nd
+      #
+      # @param project [Vanagon::Project] project to version
+      # @return [String] the version of the nuget package for this project
+      def nuget_package_version(version, release)
+        version_elements = version.split('.')
+        if version_elements.last.start_with?('g')
+          # Version for the prerelease package
+          "#{version_elements[0..-2].join('.').gsub(/[a-zA-Z]/, '')}-#{version_elements[-1]}"
+        else
+          "#{version}.#{release}".gsub(/[a-zA-Z]/, '')
+        end
+      end
+
+      # Add a repository (or install Chocolatey)
+      # Note - this only prepares the list of commands to be executed once the Platform
+      # has been setup
+      #
+      # @param definition [String] Definition for adding repo, can be a Repo URL (including file:)
+      #    If file suffix is 'ps1' it is downloaded and executed to install chocolatey
+      # @return [Array]  Commands to executed after platform startup
+      def add_repository(definition)
+        definition = URI.parse(definition)
+        commands = []
+
+        if definition.scheme =~ /^(http|ftp|file)/
+          if File.extname(definition.path) == '.ps1'
+            commands << %(powershell.exe -NoProfile -ExecutionPolicy Bypass -Command 'iex ((new-object net.webclient).DownloadString(\"#{definition}\"))')
+          else
+            commands << %(C:/ProgramData/chocolatey/bin/choco.exe source add -n #{definition.host}-#{definition.path.gsub('/', '-')} -s "#{definition}" --debug || echo "Oops, it seems that you don't have chocolatey installed on this system. Please ensure it's there by adding something like 'plat.add_repository 'https://chocolatey.org/install.ps1'' to your platform definition.")
+          end
+        else
+          raise Vanagon::Error, "Invalid repo specification #{definition}"
+        end
+
+        commands
+      end
+
+      # Get the expected output dir for the windows packages. This allows us to
+      # use some standard tools to ship internally.
+      #
+      # @param target_repo [String] optional repo target for built packages defined
+      #   at the project level
+      # @return [String] relative path to where windows packages should be staged
+      def output_dir(target_repo = "")
+        File.join("windows", target_repo, @architecture)
+      end
+
+      # Constructor. Sets up some defaults for the windows platform and calls the parent constructor
+      #
+      # Mingw varies on where it is installed based on architecture. We want to use which ever is on the system.
+      #
+      # @param name [String] name of the platform
+      # @return [Vanagon::Platform::Windows] the win derived platform with the given name
+      def initialize(name)
+        @target_user = "Administrator"
+        @make = "/usr/bin/make"
+        @tar = "/usr/bin/tar"
+        @find = "/usr/bin/find"
+        @sort = "/usr/bin/sort"
+        @num_cores = "/usr/bin/nproc"
+        @install = "/usr/bin/install"
+        @copy = "/usr/bin/cp"
+        @package_type = "msi"
+        super(name)
+      end
+    end
+  end
+end