From 811e792a176e8e57412f57113c94de362b00e091 Mon Sep 17 00:00:00 2001 From: Muhammad Faraz Maqsood Date: Wed, 8 Nov 2023 11:50:17 +0500 Subject: [PATCH] fix: wrong variable name for extra_requirement for pip extra requirements Co-authored-by: Muhammad Faraz Maqsood --- ...0231102_142414_faraz.maqsood_fix_extra_requirements_issue.md | 1 + tutordiscovery/templates/discovery/build/discovery/Dockerfile | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) create mode 100644 changelog.d/20231102_142414_faraz.maqsood_fix_extra_requirements_issue.md diff --git a/changelog.d/20231102_142414_faraz.maqsood_fix_extra_requirements_issue.md b/changelog.d/20231102_142414_faraz.maqsood_fix_extra_requirements_issue.md new file mode 100644 index 0000000..40ac0f7 --- /dev/null +++ b/changelog.d/20231102_142414_faraz.maqsood_fix_extra_requirements_issue.md @@ -0,0 +1 @@ +- [BugFix] Corrected variable name for installing extra pip requirements. (by @Faraz32123) \ No newline at end of file diff --git a/tutordiscovery/templates/discovery/build/discovery/Dockerfile b/tutordiscovery/templates/discovery/build/discovery/Dockerfile index 92fa176..c799623 100644 --- a/tutordiscovery/templates/discovery/build/discovery/Dockerfile +++ b/tutordiscovery/templates/discovery/build/discovery/Dockerfile @@ -47,7 +47,7 @@ ENV PATH /openedx/nodeenv/bin:${PATH} # instructions to benefit from docker image caching # Install base requirements RUN {% if is_buildkit_enabled() %}--mount=type=cache,target=/openedx/.cache/pip,sharing=shared {% endif %}pip install -r requirements.txt -{% for extra_requirement in DISCOVERY_EXTRA_PIP_REQUIREMENTS %}RUN {% if is_buildkit_enabled() %}--mount=type=cache,target=/openedx/.cache/pip,sharing=shared {% endif %}pip install '{{ extra_requirements }}' +{% for extra_requirement in DISCOVERY_EXTRA_PIP_REQUIREMENTS %}RUN {% if is_buildkit_enabled() %}--mount=type=cache,target=/openedx/.cache/pip,sharing=shared {% endif %}pip install '{{ extra_requirement }}' {% endfor %} # Install npm, bower requirements