public-offering
/
pad.pub0.org
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Docker: Install plugins before core deps

pull/5091/head
Richard Hansen 2021-06-17 20:56:34 -04:00
parent 5946244bef
commit e9785f37ee
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Dockerfile
View File

@ -68,9 +68,17 @@ WORKDIR "${EP_DIR}"
COPY --chown=etherpad:etherpad ./ ./ COPY --chown=etherpad:etherpad ./ ./
RUN src/bin/installDeps.sh && \ # Plugins must be installed before installing Etherpad's dependencies, otherwise
{ [ -z "${ETHERPAD_PLUGINS}" ] || \ # npm will try to hoist common dependencies by removing them from
# src/node_modules and installing them in the top-level node_modules. As of
# v6.14.10, npm's hoist logic appears to be buggy, because it sometimes removes
# dependencies from src/node_modules but fails to add them to the top-level
# node_modules. Even if npm correctly hoists the dependencies, the hoisting
# seems to confuse tools such as `npm outdated`, `npm update`, and some ESLint
# rules.
RUN { [ -z "${ETHERPAD_PLUGINS}" ] || \
npm install --no-save ${ETHERPAD_PLUGINS}; } && \ npm install --no-save ${ETHERPAD_PLUGINS}; } && \
src/bin/installDeps.sh && \
rm -rf ~/.npm rm -rf ~/.npm
# Copy the configuration file. # Copy the configuration file.