/vagrant

New issue

Editing /etc/exports shouldn't try to use sudo if write permissions already exist #2643

Closed
purpleidea opened this Issue Dec 12, 2013 · 4 comments

Comments

Assignees
No one assigned
Labels
enhancement synced-folders/general
Projects
None yet
Milestone
No milestone
2 participants
@purpleidea @mitchellh
@purpleidea
Contributor

purpleidea commented Dec 12, 2013

Opening a new issue, related to:
#2642

For users who have added write access to /etc/exports as their user account, vagrant will try to use sudoers even if these priviledges are not necessary. It should check if it is necessary first.

This was referenced Dec 12, 2013

Closed
Closed

@mitchellh mitchellh added synced_folders NFS enhancement and removed enhancement labels Oct 24, 2014

@mitchellh mitchellh added a commit that referenced this issue Oct 24, 2014

@mitchellh mitchellh hosts/bsd: only use sudo if we can't write /etc/exports [GH-2643] aa981cf
@mitchellh
Owner

mitchellh commented Oct 24, 2014

Done!

mitchellh closed this Oct 24, 2014

@purpleidea
Contributor

purpleidea commented Oct 24, 2014

@mitchellh Sweet! ... Although this looks like it wasn't done for GNU/Linux, only BSD :( Can we re-open?

@mitchellh mitchellh added a commit that referenced this issue Oct 24, 2014

@mitchellh mitchellh hosts/linux: don't use sudo to write NFS if writable [GH-2643] cc01f01
@mitchellh
Owner

mitchellh commented Oct 24, 2014

Done
@purpleidea
Contributor

purpleidea commented Oct 24, 2014

Sweet!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment