Remove requirements-docker.txt

This file needs to be maintained parallel to requirements.txt, but isn't, causes problems when new dependencies are introduced. Instead, derive the Docker dependencies from the stock ones. Signed-off-by: Alastair D'Silva <alastair@d-silva.org>
Significant-Gravitas · Apr 17, 2023 · 60b779a · 60b779a · mikekelly · Apr 17, 2023
1 parent f41febd
commit 60b779a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 30 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -17,8 +17,9 @@ RUN chown appuser:appuser /home/appuser
 USER appuser
 
 # Copy the requirements.txt file and install the requirements
-COPY --chown=appuser:appuser requirements-docker.txt .
-RUN pip install --no-cache-dir --user -r requirements-docker.txt
+COPY --chown=appuser:appuser requirements.txt .
+RUN sed -i '/Items below this point will not be included in the Docker Image/,$d' requirements.txt && \
+	pip install --no-cache-dir --user -r requirements.txt
 
 # Copy the application files
 COPY --chown=appuser:appuser autogpt/ ./autogpt

diff --git a/requirements-docker.txt b/requirements-docker.txt
diff --git a/requirements.txt b/requirements.txt
@@ -30,6 +30,8 @@ sourcery
 isort
 gitpython==3.1.31
 
+# Items below this point will not be included in the Docker Image
+
 # Testing dependencies
 pytest
 asynctest