-
Notifications
You must be signed in to change notification settings - Fork 522
feat: dockerize (with requested changes) #2392
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
FROM python:3.9.22 | ||
|
||
RUN mkdir /www | ||
WORKDIR /www | ||
COPY . . | ||
|
||
# Update the Linux environment/Operating System | ||
RUN apt-get update | ||
RUN apt-get autoremove -y | ||
RUN apt-get autoclean -y | ||
RUN apt-get upgrade -y | ||
|
||
# Download the Python3 dependencies | ||
RUN python3 -m pip install --upgrade pip | ||
RUN python3 -m pip install -r requirements.txt | ||
|
||
WORKDIR /www/docs | ||
|
||
RUN chmod +x make.bat | ||
|
||
# The following CMD is run every time the Container starts | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This actually took a bit - thought I could simply copy over my prior BASH script but it seems I need to/should reset the WORKDIR to use CMD (over ENTRYPOINT) for a slightly more succinct command. |
||
CMD ["make", "dockerhtml"] |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -59,6 +59,10 @@ html: | |
livehtml: | ||
sphinx-autobuild --port 8888 -j auto --watch _data -b dirhtml $(ALLSPHINXOPTS) $(BUILDDIR)/html | ||
|
||
# Used solely to bind sphinx to the Docker Container - to do so requires deliberately setting --host "0.0.0.0" | ||
dockerhtml: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Added comments per request. |
||
sphinx-autobuild --port 8888 --host "0.0.0.0" -j auto --watch _data -b dirhtml $(ALLSPHINXOPTS) $(BUILDDIR)/html | ||
|
||
oldhtml: | ||
$(SPHINXBUILD) -b html $(ALLSPHINXOPTS) $(BUILDDIR)/dirhtml | ||
@echo | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@plaindocs - I tweaked the recommended build per your prior comment here: #2379 (comment) - loved that suggestion - thanks!