Skip to content

Comments

Dockerize!#91

Open
wanovak wants to merge 1 commit intojakehilborn:masterfrom
wanovak:master
Open

Dockerize!#91
wanovak wants to merge 1 commit intojakehilborn:masterfrom
wanovak:master

Conversation

@wanovak
Copy link

@wanovak wanovak commented May 24, 2022

This is a very bare bones implementation to get this containerized. I don't know python well so if I did something stupid with its tooling, I apologize :-)

  1. Edit Makefile to specify the path of debbit
  2. (assuming docker is installed) sudo make build
  3. sudo make run

Please feel free to suggest modifications.

RUN cd /home/debbit/src && \
ln -s /usr/local/bin/geckodriver ./program_files/geckodriver && \
ln -s /opt/firefox/firefox /usr/local/bin/firefox && \
cp ./debbit.py /opt/firefox/ && \
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be necessary. I'm thinking there's something wrong with the python source but did not investigate.


WORKDIR /home/debbit/src

CMD ["python3", "-m", "pipenv", "run", "python3", "./debbit.py"]
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would love for someone to tell my why this is wrong. It looks wrong, but I don't know the python ecosystem well enough to say why.

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.

1 participant