Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid permissions error in Vagrant #756

Closed
wants to merge 1 commit into from

Conversation

tranzium
Copy link

Vagrant + Windows can prevent file permission changes

Vagrant + Windows can prevent file permission changes
@zbjornson
Copy link
Collaborator

This looks closely related to the first commit in #754 and subject to the same issues. But I don't understand how this is a Windows problem -- the change is in the non-Windows part of the binding file -- can you explain that please?

@tranzium
Copy link
Author

From what I could diagnose, changes to the file's permissions (including +x) could not be applied, thus the util script could not be executed for the install.

Using both chmod and chown result in the file being the exact same.

Although this could be a Vagrant issue, I made the changes in the assumption that one was not able to change their Vagrant configuration.

@zbjornson
Copy link
Collaborator

I see, thanks for the info. I'm still not quite clear if this relates to Windows (mentioned in the commit message but not the PR title). Are you using a set of extensions on top of Windows to get chmod and chown, or are you actually running on a linux distro?

@tranzium
Copy link
Author

I am running Ubuntu in Vagrant (virtual machine).
I will test adjusting the sync_folder permissions and see if I can get the /util/has_lib.sh script running directly from a npm install canvas call. Will report any further findings.

@LinusU
Copy link
Collaborator

LinusU commented May 3, 2017

This script has been migrated to node now so I don't think this should be a problem anymore...

@LinusU LinusU closed this May 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants