Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

Commit

Permalink
synced_folder: Warn in case :group or :owner are specified [GH-196]
Browse files Browse the repository at this point in the history
  • Loading branch information
fgrehm committed Mar 14, 2014
1 parent d0be2f3 commit d6a22c9
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 1 deletion.
4 changes: 3 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,14 @@ IMPROVEMENTS:

- Set container utsname to configured hostname by default [[GH-253]]
- Added libvirt dnsmasq leases file to the lookup paths [[GH-251]]
- Acceptance specs are now powered by `vagrant-spec` [[GH-213]]
- Improved compatibility with Vagrant 1.4 / 1.5 including the ability
to use `rsync` and `nfs` shared folders to work around synced folders
permission problems. More information can be found on the following
issues: [[GH-151]] [[GH-191]] [[GH-241]] [[GH-242]]
- Warn in case `:group` or `:owner` are specified for synced folders [[GH-251]]
- Acceptance specs are now powered by `vagrant-spec` [[GH-213]]

[GH-196]: https://github.com/fgrehm/vagrant-lxc/issues/196
[GH-251]: https://github.com/fgrehm/vagrant-lxc/pull/251
[GH-253]: https://github.com/fgrehm/vagrant-lxc/pull/253
[GH-151]: https://github.com/fgrehm/vagrant-lxc/issues/151
Expand Down
3 changes: 3 additions & 0 deletions lib/vagrant-lxc/synced_folder.rb
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,9 @@ def prepare(machine, folders, _opts)
host_path = Pathname.new(File.expand_path(data[:hostpath], machine.env.root_path))
guest_path = data[:guestpath]

env[:ui].warn(I18n.t("vagrant_lxc.messages.warn_owner")) if data[:owner]
env[:ui].warn(I18n.t("vagrant_lxc.messages.warn_group")) if data[:group]

if !host_path.directory? && data[:create]
# Host path doesn't exist, so let's create it.
@logger.info("Host path doesn't exist, creating: #{host_path}")
Expand Down
6 changes: 6 additions & 0 deletions locales/en.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,12 @@ en:
Warning! The LXC provider doesn't support any of the Vagrant public / private
network configurations (ex: `config.vm.network :private_network, ip: "some-ip"`).
They will be silently ignored.
warn_group: |-
Warning! The LXC provider doesn't support the :group parameter for synced
folders. It will be silently ignored.
warn_owner: |-
Warning! The LXC provider doesn't support the :owner parameter for synced
folders. It will be silently ignored.
vagrant:
commands:
Expand Down

0 comments on commit d6a22c9

Please sign in to comment.