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

Dockerize #1

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

Conversation

vrdeveloperone
Copy link

No description provided.

@ChrisTopherTa54321
Copy link
Owner

ChrisTopherTa54321 commented May 25, 2018

I've heard great things about Docker, but I just tried to get it to work on Windows and it does not seem to be for the faint of heart on this platform. (It changes system settings that break VirtualBox that require being manually undone via command line... "docker run -v $(pwd)/:/var/app/ vdo/foto2vam:1.0.0" is likely "docker run -v%cwd%/:/var/app/ vdo/foto2vam:1.0.0" on Windows, but then this runs into errors from Docker about mounting drives... I never actually ended up getting this to work on Windows...)

I'm fine merging the Dockerfile, especially for people on platforms that Docker works well on. However, I don't think this is the solution for the people running Windows that are unable to get the code to work otherwise. With my (limited) experience of Docker on Windows, I'd say if they were unable to get the python dependencies to work without docker, they won't be able to get docker to work, and it will make difficult-to-undo changes to their system.

Maybe if you removed the link to the Windows version, and maybe just say "This may work in Windows if you can run Docker, but is untested?" or something to that affect? Then people who know how to work Docker can use this change, but we don't have to worry about people messing up their computers trying to get Docker running.

I just don't want to make it too easy for people to mess up their computers :)

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.

2 participants