-
Notifications
You must be signed in to change notification settings - Fork 239
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
Dockerfile / Docker image #13
Comments
I guess thath is necessary Flask for use this project with a container |
@aledv can you explain why should it require Flask? 🤔 IMHO for the no change should be necessary (except from adding a $ docker run smd -s spotify:track:7ARveOiD31w2Nq0n5FsSf8 -v ./downloads:/downloads instead of $ ./main.py -s spotify:track:7ARveOiD31w2Nq0n5FsSf8 |
Ok, you have to run the container everytime... Whit flask you can leave the container always run and pass the parameters when you need a new request. Sorry for my english |
I still don't see why flask is needed. This is why I don't think what you describes is related to Docker. Right now flask is IMHO not needed because |
I second this. The project has a lot of dependencies. I'm not a python developer and I don't know how to troubleshoot various python dependency issues. Dockerfile would ensure this works for anyone. FROM python:3.8.0-buster
WORKDIR /app
RUN git clone https://github.com/artyshko/smd.git
WORKDIR /app/smd
RUN apt-get update
RUN apt-get install -y python-dev libsdl-image1.2-dev libsdl-mixer1.2-dev libsdl-ttf2.0-dev libsdl1.2-dev libsmpeg-dev python-numpy subversion libportmidi-dev ffmpeg libswscale-dev libavformat-dev libavcodec-dev
RUN pip3 install -r requirements.txt
RUN printf "import imageio\nimageio.plugins.ffmpeg.download()" > dl.py && python3 dl.py && rm dl.py
#RUN pip3 install PyQtWebEngine
#RUN apt-get install -y python3-pyqt5.qtwebengine
CMD python3 main.py
If feel like this is close, but still it doesn't work. Can someone help? |
Check out freyr..https://github.com/miraclx/freyr-js See #101 |
It would be nice to have setup & dependencies available via Docker.
The text was updated successfully, but these errors were encountered: