vagrant-unbundled 1.8.1.2 → 1.8.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e29f22ac5a9b0c8690fcef6cb3b98499b15d0880
-  data.tar.gz: e3e7c13be58f8205d26ba844f5a15fc2e167ce89
+  metadata.gz: ed1de343220b28131608158c1db4ff499abb2ff6
+  data.tar.gz: c696e8dd36edf81c1ba2b12ace45f2b60b96aa3a
 SHA512:
-  metadata.gz: c7508d37ab983ac93c23c52c109996ddd2f9958136b00f89b36b68deefb6d17fac5a8ae72e28f67f353095fdbdae6e1737839c44059c777472d8f4f6512eac32
-  data.tar.gz: 8e165ba4563cce4752ab01bf896e818c92c709e2061b50d45f75ae58552f68f64711d09cffb2d1ece7aabd0ce5b3d57234c458460cbcb3ffb6e406991746885e
+  metadata.gz: 06d583f8d794ec15caa5dd5cbb3fb86b7bce98fb49f3a5bf08742b1bff885867ac7e1e3dbe1d213105171329fe7134bcdd2c2e8d6fca351fea3417b351596752
+  data.tar.gz: f84bd662999a52cd859af10044414f5f25e9e2a076a3c46fec7ab31aa3fa17cf9a2ee135d534f69872841bdfea820bc7847c2f24880de227f806177337f166f4

data/.gitignore

@@ -9,8 +9,8 @@
 acceptance_config.yml
 boxes/*
 /.vagrant
-/website/docs/.vagrant
-/website/www/.vagrant
+/website/.vagrant
+/website/build
 /vagrant-spec.config.rb
 
 # Bundler/Rubygems
@@ -21,6 +21,7 @@ tags
 /Gemfile.lock
 test/tmp/
 vendor/
+/exec
 
 # Documentation
 _site/*
@@ -43,14 +44,3 @@ doc/
 .ruby-gemset
 .ruby-version
 .rvmrc
-
-# Website: docs
-website/docs/.sass-cache
-website/docs/build
-website/docs/Rakefile
-
-# Website: www
-website/www/.sass-cache
-website/www/build
-website/www/Rakefile
-exec/

data/.travis.yml

@@ -4,6 +4,9 @@ sudo: false
 
 cache: bundler
 
+before_install:
+  - gem install bundler -v '1.12.5'
+
 addons:
   apt:
     packages:

data/CHANGELOG.md

@@ -1,4 +1,165 @@
-## 1.8.1.dev (December 21, 2015)
+## 1.8.4 (June 13, 2016)
+
+BUG FIXES:
+
+  - core: Fix bundler plugin issue and version constraint [GH-7418, GH-7415]
+  - providers/virtualbox: Use 8 network interfaces (due to Windows limitation)
+      [GH-7417, GH-7419]
+  - provisioners/ansible(both): Honor "galaxy_roles_path" option when running
+      ansible-playbook [GH-7269, GH-7420]
+  - provisioners/ansible_local: Add quotes around "ansible-galaxy" arguments
+      [GH-7420]
+
+IMPROVEMENTS:
+
+  - guests/redhat: Add CloudLinux detection [GH-7428, GH-7427]
+
+## 1.8.3 (June 10, 2016)
+
+BREAKING CHANGES:
+
+  - The `winrm` communicator now shares the same upload behavior as the `ssh`
+      communicator. This change should have no impact to most vagrant operations
+      but may break behavior when uploading directories to an existing
+      destination target. The `file` provisioner should be the only builtin
+      provisioner affected by this change. When uploading a directory and the
+      destination directory exists on the endpoint, the source base directory
+      will be created below the destination directory on the endpoint and the
+      source directory contents will be unzipped to that location. Prior to this
+      release, the contents of the source directory would be unzipped to an
+      existing destination directory without creating the source base directory.
+      This new behavior is more consistent with SCP and other well known shell copy commands.
+  - The Chef provisioner's `channel` default value has changed from "current" to
+      "stable". The "current" channel includes nightly releases and should be
+      opt-in only. Note that users wishing to download the Chef Development Kit
+      will need to opt into the "current" channel until Chef Software promotes
+      into the "stable" channel.
+  - The Arch Linux host capability for NFS removed support for rc.d in favor or
+      systemd which has been present since 2012. Please see GH-7181 for more
+      information.
+
+FEATURES:
+
+  - provider/docker: Allow non-linux users to opt-out of the host VM to run
+      Docker containers by setting `config.force_host_vm = false` in the
+      Vagrantfile. This is especially useful for customers who wish to use
+      the beta builds for Mac and Windows, dlite, or a custom provider.
+      [GH-7277, GH-7298, 8c11b53]
+  - provider/docker: New command: `docker-exec` allows attaching to an
+      already-running container.
+      [GH-7377, GH-6566, GH-5193, GH-4904, GH-4057, GH-4179, GH-4903]
+
+IMPROVEMENTS:
+
+  - core/downloader: increase box resume download limit to 24h
+      [GH-7352, GH-7272]
+  - core/package: run validations prior to packaging [GH-7353, GH-7351]
+  - core/action: make `start` ("vagrant up") run provisioners [GH-4467, GH-4421]
+  - commands/all: Make it clear that machine IDs can be specified
+      [GH-7356, GH-7228]
+  - commands/init: Add support for specifying the box version [GH-7363, GH-5004]
+  - commands/login: Print a warning with both the environment variable and
+      local login token are present [GH-7206, GH-7219]
+  - communicators/winrm: Upgrade to latest WinRM gems [GH-6922]
+  - provisioners/ansible_local: Allow to install Ansible from pip,
+      with version selection capability [GH-6654, GH-7167]
+  - provisioners/ansible_local: Use `provisioning_path` as working directory
+      for `ansible-galaxy` execution
+  - provisioners/ansible(both provisioners): Add basic config
+      validators/converters on `raw_arguments` and `raw_ssh_args` options
+      [GH-7103]
+  - provisioners/chef: Add the ability to install on SUSE [GH-6806]
+  - provisioners/chef: Support legacy solo mode [GH-7327]
+  - provisioners/docker: Restart container if newer image is available
+      [GH-7358, GH-6620]
+  - hosts/arch: Remove sysvinit and assume systemd [GH-7181]
+  - hosts/linux: Do not use a pager with systemctl commands [GH-7270]
+  - hosts/darwin: Add `extra_args` support for RDP [GH-5523, GH-6602]
+  - hosts/windows: Use SafeExec to capture history in Powershell [GH-6749]
+  - guests/amazon: Add detection [GH-7395, GH-7254]
+  - guests/freebsd: Add quotes around hostname [GH-6867]
+  - guests/fedora: Add support for ipv6 static networks [GH-7275, GH-7276]
+  - guests/tinycore: Add support for shared folders [GH-6977, GH-6968]
+  - guests/trisquel: Add initial support [GH-6842, GH-6843]
+  - guests/windows: Add support for automatic login (no password prompting)
+      [GH-5670]
+  - core: Add `--no-delete` and provisioning flags to snapshot restore/pop
+      [GH-6879]
+  - providers/docker: Allow TCP and UDP ports on the same number [GH-7365,
+      GH-5527]
+  - providers/hyperv: Add support for differencing disk [GH-7090]
+  - providers/hyperv: Add support for snapshots [GH-7110]
+  - providers/hyperv: Reinstate compatibility with PS 4 [GH-7108]
+  - providers/virtualbox: Add linked clone support for Virtualbox 1.4 [GH-7050]
+  - synced_folders/nfs: Read static and dynamic IPs [GH-7290, GH-7289]
+
+BUG FIXES:
+
+  - core: Bump nokogiri version to fix windows bug [GH-6766, GH-6848]
+  - core: Revert a change made to the output of the identify file [GH-6962,
+      GH-6929, GH-6589]
+  - core: Fix login command behind a proxy [GH-6898, GH-6899]
+  - core: Fix support for regular expressions on multi-machine `up`
+      [GH-6908, GH-6909]
+  - core: Allow boxes to use pre-release versions [GH-6892, GH-6893]
+  - core: Rescue `Errno:ENOTCONN` waiting for port to be open [GH-7182, GH-7184]
+  - core: Properly authenticate metadata box URLs [GH-6776, GH-7158]
+  - core: Do not run provisioners if already run on resume [GH-7059, GH-6787]
+  - core: Implement better tracking of tempfiles and tmpdirs to identify file
+      leaks [GH-7355]
+  - core: Allow SSH forwarding on Windows [GH-7287, GH-7202]
+  - core: Allow customizing `keys_only` SSH option [GH-7360, GH-4275]
+  - core: Allow customizing `paranoid` SSH option [GH-7360, GH-4275]
+  - command/box_update: Do not update the same box twice [GH-6042, GH-7379]
+  - command/init: Remove unnecessary `sudo` from generated Vagrantfile
+      [GH-7369, GH-7295]
+  - docs & core: Be consistent about the "2" in the Vagrantfile version
+      [GH-6961, GH-6963]
+  - guests/all: Refactor guest capabilities to run in a single command -
+      **please see GH-7393 for the complete list of changes!**
+  - guests/arch: Restart network after configuration [GH-7120, GH-7119]
+  - guests/debian: Do not return an error if ifdown fails [GH-7159,
+      GH-7155, GH-6871]
+  - guests/freebsd: Use `pkg` to install rsync [GH-6760]
+  - guests/freebsd: Use `netif` to configure networks [GH-5852, GH-7093]
+  - guests/coreos: Detect all interface names [GH-6608, GH-6610]
+  - providers/hyperv: Only specify Hyper-V if the parameter is support
+      [GH-7101, GH-7098]
+  - providers/virtualbox: Set maximum network adapters to 36 [GH-7293, GH-7286]
+  - providers/virtualbox: Do not fail when master VM from linked clone is
+      missing [GH-7126, GH-6742]
+  - providers/virtualbox: Use scoped overrides in preparring NFS
+      [GH-7387, GH-7386]
+  - provisioners/ansible: Fix a race condition in the concurrent generations of
+      the ansible inventory file, while running `vagrant up --parallel`
+      [GH-6526, GH-7190]
+  - provisioners/ansible_local: Don't quote the Ansible arguments defined in the
+      `raw_arguments` option [GH-7103]
+  - provisioners/ansible_local: Format json `extra_vars` with double quotes
+      [GH-6726, GH-7103]
+  - provisioners/ansible_local: Fix errors in absolute paths to playbook or
+      galaxy resources when running on a Windows host [GH-6740, GH-6757]
+  - provisioners/ansible_local: Change the way to verify `ansible-galaxy`
+      presence, to avoid a non-zero status code with Ansible 2.0 [GH-6793]
+  - provisioners/ansible(both provisioners): The Ansible configuration files
+      detection is only executed by the `provision` action [GH-6763, GH-6984]
+  - provisioners/chef: Do not use double sudo when installing
+      [GGH-6805, GH-6804]
+  - provisioners/chef: Change the default channel to "stable" (previously it
+      was "current") [GH-7001, GH-6979]
+  - provisioners/chef: Default node_name to hostname if present
+      [GH-7063, GH-7153]
+  - provisioners/docker: Fix -no-trunc command option [GH-7085]
+  - provisioners/docker: Allow provisioning when container name is specified
+      [GH-7074, GH-7086]
+  - provisioners/puppet: Use `where.exe` to locate puppet binary
+      [GH-6912, GH-6876]
+  - provisioners/salt: Move masterless config to apply to all platforms
+      [GH-7207, Gh-6924, GH-6915]
+  - pushes/ftp: Create parent directories when uploading [GH-7154, GH-6316]
+  - synced_folders/smb: Do not interpolate configuration file [GH-6906]
+
+## 1.8.1 (December 21, 2015)
 
 BUG FIXES:
 
@@ -8,6 +169,8 @@ BUG FIXES:
       UNC paths [GH-6598]
   - core: Fix a crash in parsing the config in some cases with network
       configurations [GH-6730]
+  - core: Clean up temporarily files created by bundler
+    [GH-7354, GH-6301, GH-3469, GH-6231]
   - commands/up: Smarter logic about what provider to install, avoiding
       situations where VirtualBox was installed over the correct provider [GH-6731]
   - guests/debian: Fix Docker install [GH-6722]
@@ -590,7 +753,7 @@ BUG FIXES:
   - commands/package: base package won't crash with exception [GH-4017]
   - commands/rsync-auto: Destroyed machines won't raise exceptions. [GH-4031]
   - commands/ssh: Extra args are passed through to Docker container. [GH-4378]
-  - communicators/ssh: Nicer error if remote unexpectedly disconects. [GH-4038]
+  - communicators/ssh: Nicer error if remote unexpectedly disconnects. [GH-4038]
   - communicators/ssh: Clean error when max sessions is hit. [GH-4044]
   - communicators/ssh: Fix many issues around PTY-enabled output parsing.
       [GH-4408]
@@ -1299,7 +1462,7 @@ IMPROVEMENTS:
     certs from a custom CA. [GH-2337]
   - commands/box/add: Can now specify a client cert when downloading a
     box. [GH-1889]
-  - commands/init: Add `--output` option for specifing output path, or
+  - commands/init: Add `--output` option for specifying output path, or
     "-" for stdin. [GH-1364]
   - commands/provision: Add `--no-parallel` option to disable provider
     parallelization if the provider supports it. [GH-2404]
@@ -2838,7 +3001,7 @@ compatibility.
     in a saved state. [GH-123]
   - Added `config.chef.recipe_url` which allows you to specify a URL to
     a gzipped tar file for chef solo to download cookbooks. See the
-    [chef-solo docs](http://wiki.opscode.com/display/chef/Chef+Solo#ChefSolo-RunningfromaURL) for more information.
+    [chef-solo docs](https://docs.chef.io/chef_solo.html) for more information.
     [GH-121]
   - Added `vagrant box repackage` which repackages boxes which have
     been added. This is useful in case you want to redistribute a base

data/LICENSE

@@ -1,6 +1,6 @@
 The MIT License
 
-Copyright (c) 2010-2015 Mitchell Hashimoto
+Copyright (c) 2010-2016 Mitchell Hashimoto
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -3,7 +3,7 @@
 * Website: [https://www.vagrantup.com/](https://www.vagrantup.com/)
 * Source: [https://github.com/mitchellh/vagrant](https://github.com/mitchellh/vagrant)
 * IRC: `#vagrant` on Freenode
-* Mailing list: [Google Groups](http://groups.google.com/group/vagrant-up)
+* Mailing list: [Google Groups](https://groups.google.com/group/vagrant-up)
 
 Vagrant is a tool for building and distributing development environments.
 
@@ -21,12 +21,12 @@ between Windows, Mac OS X, and Linux.
 For the quick-start, we'll bring up a development machine on
 [VirtualBox](https://www.virtualbox.org/) because it is free and works
 on all major platforms. Vagrant can, however, work with almost any
-system such as [OpenStack] (https://www.openstack.org/), [VMware] (http://www.vmware.com/), [Docker] (https://docs.docker.com/), etc.
+system such as [OpenStack] (https://www.openstack.org/), [VMware] (https://www.vmware.com/), [Docker] (https://docs.docker.com/), etc.
 
 First, make sure your development machine has
 [VirtualBox](https://www.virtualbox.org/)
 installed. After this,
-[download and install the appropriate Vagrant package for your OS](http://www.vagrantup.com/downloads).
+[download and install the appropriate Vagrant package for your OS](https://www.vagrantup.com/downloads.html).
 
 To build your first virtual environment:
 
@@ -40,7 +40,7 @@ the box doesn't already exist on your system.
 ## Getting Started Guide
 
 To learn how to build a fully functional development environment, follow the
-[getting started guide](http://docs.vagrantup.com/v2/getting-started/index.html).
+[getting started guide](https://www.vagrantup.com/docs/getting-started/index.html).
 
 ## Installing the Gem from Git
 
@@ -53,13 +53,11 @@ Ruby 2.0 is needed.
 
 ## Contributing to Vagrant
 
-### Dependencies and Unit Tests
+To install Vagrant from source, please [follow the guide in the Wiki](https://github.com/mitchellh/vagrant/wiki/Installing-Vagrant-from-Source).
 
-To hack on Vagrant, you'll need [bundler](https://github.com/bundler/bundler) which can
-be installed with a simple `gem install bundler`. Afterwards, do the following:
+You can run the test suite with:
 
-    bundle install
-    rake
+    bundle exec rake
 
 This will run the unit test suite, which should come back all green! Then you're good to go!
 

data/RELEASE.md

@@ -0,0 +1,28 @@
+# Releasing Vagrant
+
+This documents how to release Vagrant. Various steps in this document will
+require privileged access to private systems, so this document is only
+targetted at Vagrant core members who have the ability to cut a release.
+
+  1. Update `version.txt` to the version you want to release.
+
+  1. Update `CHANGELOG.md` to have a header with the release version and date.
+
+  1. Commit those changes and also tag the release with the version:
+     `git tag vX.Y.Z`. Push them.
+
+  1. Trigger an installer creation run within the HashiCorp Bamboo
+     installation. This will take around 45 minutes.
+
+  1. Download all the resulting artifacts into the `pkg/dist` folder
+     relative to the Vagrant repository.
+
+  1. Run the awkwardly-named `./scripts/bintray_upload.sh` with the version
+     that is being created. This must be run from the Vagrant repo root.
+     This will GPG sign and checksum the files.
+
+  1. Run `hc-releases -upload pkg/dist` to upload the releases to S3.
+
+  1. Update `website/config.rb` to point to the latest version. Commit and push.
+
+  1. Use Atlas `hashicorp/vagrant-www` to deploy the site by queueing a build.

data/bin/vagrant

@@ -139,7 +139,7 @@ begin
   end
 
   # Also allow users to force colors.
-  if argv.include?("--color")
+  if argv.include?("--color") || ENV["VAGRANT_FORCE_COLOR"]
     argv.delete("--color")
     opts[:ui_class] = Vagrant::UI::Colored
   end

data/contrib/bash/completion.sh

@@ -53,7 +53,7 @@ __vagrantinvestigate() {
 _vagrant() {
     cur="${COMP_WORDS[COMP_CWORD]}"
     prev="${COMP_WORDS[COMP_CWORD-1]}"
-    commands="box connect destroy docker-logs docker-run global-status halt help init list-commands login package plugin provision push rdp reload resume rsync rsync-auto share ssh ssh-config status suspend up version"	
+    commands="box connect destroy docker-exec docker-logs docker-run global-status halt help init list-commands login package plugin provision push rdp reload resume rsync rsync-auto share snapshot ssh ssh-config status suspend up version"
 
     if [ $COMP_CWORD == 1 ]
     then
@@ -104,6 +104,11 @@ _vagrant() {
               COMPREPLY=($(compgen -W "${commands}" -- ${cur}))
               return 0
             ;;
+            "snapshot")
+              snapshot_commands="delete list pop push restore save"
+              COMPREPLY=($(compgen -W "${snapshot_commands}" -- ${cur}))
+              return 0
+            ;;
             *)
             ;;
         esac

data/contrib/sudoers/linux-fedora

@@ -1,5 +1,5 @@
 Cmnd_Alias VAGRANT_EXPORTS_ADD = /usr/bin/tee -a /etc/exports
-Cmnd_Alias VAGRANT_NFSD_CHECK = /usr/bin/systemctl status nfs-server.service
+Cmnd_Alias VAGRANT_NFSD_CHECK = /usr/bin/systemctl status --no-pager nfs-server.service
 Cmnd_Alias VAGRANT_NFSD_START = /usr/bin/systemctl start nfs-server.service
 Cmnd_Alias VAGRANT_NFSD_APPLY = /usr/sbin/exportfs -ar
 Cmnd_Alias VAGRANT_EXPORTS_REMOVE = /bin/sed -r -e * d -ibak /*/exports

data/keys/README.md

@@ -1,7 +1,7 @@
 # Insecure Keypair
 
 These keys are the "insecure" public/private keypair we offer to
-[base box creators](http://docs.vagrantup.com/v2/boxes/base.html) for use in their base boxes so that
+[base box creators](https://www.vagrantup.com/docs/boxes/base.html) for use in their base boxes so that
 vagrant installations can automatically SSH into the boxes.
 
 If you're working with a team or company or with a custom box and

data/lib/vagrant/action/builtin/box_add.rb

@@ -18,6 +18,10 @@ module Vagrant
         # to NOT be metadata.
         METADATA_SIZE_LIMIT = 20971520
 
+        # This is the amount of time to "resume" downloads if a partial box
+        # file already exists.
+        RESUME_DELAY = 24 * 60 * 60
+
         def initialize(app, env)
           @app    = app
           @logger = Log4r::Logger.new("vagrant::action::builtin::box_add")
@@ -253,7 +257,7 @@ module Vagrant
           end
 
           provider_url = metadata_provider.url
-          if url != authenticated_url
+          if provider_url != authenticated_url
             # Authenticate the provider URL since we're using auth
             hook_env    = env[:hook].call(:authenticate_box_url, box_urls: [provider_url])
             authed_urls = hook_env[:box_urls]
@@ -261,7 +265,7 @@ module Vagrant
               raise "Bad box authentication hook, did not generate proper results."
             end
             provider_url = authed_urls[0]
-          end
+        end
 
           box_add(
             [[provider_url, metadata_provider.url]],
@@ -393,7 +397,7 @@ module Vagrant
             if env[:box_clean]
               @logger.info("Cleaning existing temp box file.")
               delete = true
-            elsif temp_path.mtime.to_i < (Time.now.to_i - 6 * 60 * 60)
+            elsif temp_path.mtime.to_i < (Time.now.to_i - RESUME_DELAY)
               @logger.info("Existing temp file is too old. Removing.")
               delete = true
             end

data/lib/vagrant/action/general/package.rb

@@ -3,6 +3,7 @@ require "pathname"
 
 require 'vagrant/util/safe_chdir'
 require 'vagrant/util/subprocess'
+require 'vagrant/util/presence'
 
 module Vagrant
   module Action
@@ -20,25 +21,63 @@ module Vagrant
       class Package
         include Util
 
+        # Perform sanity validations that the provided output filepath is sane.
+        # In particular, this function validates:
+        #
+        #   - The output path is a regular file (not a directory or symlink)
+        #   - No file currently exists at the given path
+        #   - A directory of package files was actually provided (internal)
+        #
+        # @param [String] output path to the output file
+        # @param [String] directory path to a directory containing the files
+        def self.validate!(output, directory)
+          filename = File.basename(output.to_s)
+          output   = fullpath(output)
+
+          if File.directory?(output)
+            raise Vagrant::Errors::PackageOutputDirectory
+          end
+
+          if File.exist?(output)
+            raise Vagrant::Errors::PackageOutputExists, filename: filename
+          end
+
+          if !Vagrant::Util::Presence.present?(directory) || !File.directory?(directory)
+            raise Vagrant::Errors::PackageRequiresDirectory
+          end
+        end
+
+        # Calculate the full path of the given path, relative to the current
+        # working directory (where the command was run).
+        #
+        # @param [String] output the relative path
+        def self.fullpath(output)
+          File.expand_path(output, Dir.pwd)
+        end
+
+        # The path to the final output file.
+        # @return [String]
+        attr_reader :fullpath
+
         def initialize(app, env)
           @app = app
 
           env["package.files"]  ||= {}
           env["package.output"] ||= "package.box"
+
+          @fullpath = self.class.fullpath(env["package.output"])
         end
 
         def call(env)
           @env = env
-          file_name = File.basename(@env["package.output"].to_s)
 
-          raise Errors::PackageOutputDirectory if File.directory?(tar_path)
-          raise Errors::PackageOutputExists, file_name:file_name if File.exist?(tar_path)
-          raise Errors::PackageRequiresDirectory if !env["package.directory"] ||
-            !File.directory?(env["package.directory"])
+          self.class.validate!(env["package.output"], env["package.directory"])
+
+          raise Errors::PackageOutputDirectory if File.directory?(fullpath)
 
           @app.call(env)
 
-          @env[:ui].info I18n.t("vagrant.actions.general.package.compressing", tar_path: tar_path)
+          @env[:ui].info I18n.t("vagrant.actions.general.package.compressing", fullpath: fullpath)
           copy_include_files
           setup_private_key
           compress
@@ -54,7 +93,7 @@ module Vagrant
           end
 
           # Cleanup any packaged files if the packaging failed at some point.
-          File.delete(tar_path) if File.exist?(tar_path)
+          File.delete(fullpath) if File.exist?(fullpath)
         end
 
         # This method copies the include files (passed in via command line)
@@ -88,7 +127,7 @@ module Vagrant
         def compress
           # Get the output path. We have to do this up here so that the
           # pwd returns the proper thing.
-          output_path = tar_path.to_s
+          output_path = fullpath.to_s
 
           # Switch into that directory and package everything up
           Util::SafeChdir.safe_chdir(@env["package.directory"]) do
@@ -147,11 +186,6 @@ module Vagrant
             f.puts %Q[end]
           end
         end
-
-        # Path to the final box output file
-        def tar_path
-          File.expand_path(@env["package.output"], FileUtils.pwd)
-        end
       end
     end
   end