Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Improve git permissions guidance in docs/other/windows.md #235

Closed
rhuffstedtler opened this issue Sep 7, 2015 · 2 comments
Closed

Improve git permissions guidance in docs/other/windows.md #235

rhuffstedtler opened this issue Sep 7, 2015 · 2 comments

Comments

@rhuffstedtler
Copy link
Contributor

I disagree with your suggested approach regarding workflow on windows. I figured it would be better to start with an issue rather than submitting a PR.

There are a number of advantages to being able to work directly in the VM rather than through the shared folder (speed, consistency when it comes to deployment, etc.). Because we always use a fix permissions script as part of the build process, we don't worry too much about the permissions in the repo, so anytime windows is in the picture we have git ignore file mode changes:

git config core.fileMode false

It seems to me that would be better guidance to give people as it would make their use of drupalvm more flexible.

@geerlingguy
Copy link
Owner

And to be clear, are you suggesting setting that as a global inside the VM, or recommending it for Windows users on the host? (Or both?)

@rhuffstedtler
Copy link
Contributor Author

I usually do it in both places if I think I'm going to be touching the files from the host at all.

ravbaba pushed a commit to ravbaba/drupal-vm that referenced this issue Sep 23, 2015
ravbaba pushed a commit to ravbaba/drupal-vm that referenced this issue Sep 23, 2015
ravbaba pushed a commit to ravbaba/drupal-vm that referenced this issue Sep 23, 2015
ashabed pushed a commit to debugacademy/academyvm that referenced this issue Oct 5, 2015
joestewart pushed a commit to joestewart/drupal-vm that referenced this issue Jun 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants