From ea44834c419dc85ae9ea70e12a98bf379a451ab8 Mon Sep 17 00:00:00 2001 From: Adrien Ferrand Date: Mon, 2 Dec 2019 21:39:55 +0100 Subject: [PATCH] Fix docker build regarding the new certbot layout (#11) This PR adds appropriate corrections to certbot dockerfile to work with the new layout (moving certbot python project in its own subdirectory). This PR has been tested with success using the `build.sh` script on a fake v1.0 version of certbot published on my fork (https://github.com/adferrand/certbot/releases/tag/v1.0) instead of archives from `certbot/certbot`. --- tools/docker/core/Dockerfile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/docker/core/Dockerfile b/tools/docker/core/Dockerfile index 1cf71dc2f..4034eb67d 100644 --- a/tools/docker/core/Dockerfile +++ b/tools/docker/core/Dockerfile @@ -19,7 +19,7 @@ WORKDIR /opt/certbot RUN mkdir -p src \ && wget -O certbot-${CERTBOT_VERSION}.tar.gz https://github.com/certbot/certbot/archive/v${CERTBOT_VERSION}.tar.gz \ && tar xf certbot-${CERTBOT_VERSION}.tar.gz \ - && cp certbot-${CERTBOT_VERSION}/CHANGELOG.md certbot-${CERTBOT_VERSION}/README.rst certbot-${CERTBOT_VERSION}/setup.py src/ \ + && cp certbot-${CERTBOT_VERSION}/CHANGELOG.md certbot-${CERTBOT_VERSION}/README.rst src/ \ && cp certbot-${CERTBOT_VERSION}/letsencrypt-auto-source/pieces/dependency-requirements.txt . \ && cp -r certbot-${CERTBOT_VERSION}/tools tools \ && cp -r certbot-${CERTBOT_VERSION}/acme src/acme \ @@ -48,5 +48,5 @@ RUN apk add --no-cache --virtual .build-deps \ && pip install -r dependency-requirements.txt \ && pip install --no-cache-dir --no-deps \ --editable src/acme \ - --editable src \ + --editable src/certbot \ && apk del .build-deps