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

Update the Dockerfile to work on Windows machines #195

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

auggiecc87
Copy link

This is a relatively simple change to allow people using windows to not get hosed when they pull this repo (I know, "but why are you using windows?", maybe because that is what my company uses).

@auggiecc87
Copy link
Author

I understand that there is a setting on git that allows the checkouts (and checkins) to keep linux line endings, but that would require a check in for testing, and/or only pulling with no editing.

I think this is a great way to work around people using windows to edit / add files, and give them the template to add more scripts. I leave for reference this article which basically says: "Yeah, it kinda sucks but this is the best work around still"

http://willi.am/blog/2016/08/11/docker-for-windows-dealing-with-windows-line-endings/

@sscaling
Copy link
Collaborator

Fixes #146

@sscaling
Copy link
Collaborator

Potentially fixes #262

@odin88
Copy link

odin88 commented Aug 25, 2018

Not working for me. Using Windows 10. I clone this repo and change the Dockerfile based on @auggiecc87 fix but it is not working

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.

3 participants