Skip to content

Commit 3ad17ec

Browse files
committed
- Fix Drush installation
- Consistent naming of arguments - Add deprecated install note
1 parent 0c41fce commit 3ad17ec

3 files changed

Lines changed: 7 additions & 7 deletions

File tree

docker-compose.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,10 +86,10 @@ services:
8686
- CHROME_DRIVER_VERSION=${WORKSPACE_CHROME_DRIVER_VERSION}
8787
- NODE_VERSION=${WORKSPACE_NODE_VERSION}
8888
- YARN_VERSION=${WORKSPACE_YARN_VERSION}
89+
- DRUSH_VERSION=${WORKSPACE_DRUSH_VERSION}
8990
- TZ=${WORKSPACE_TIMEZONE}
9091
- BLACKFIRE_CLIENT_ID=${BLACKFIRE_CLIENT_ID}
9192
- BLACKFIRE_CLIENT_TOKEN=${BLACKFIRE_CLIENT_TOKEN}
92-
- DRUSH_VERSION=${DRUSH_VERSION}
9393
volumes:
9494
- ${APP_CODE_PATH_HOST}:${APP_CODE_PATH_CONTAINER}
9595
extra_hosts:

env-example

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@ DOCKER_SYNC_STRATEGY=native_osx
7070

7171
### WORKSPACE #############################################
7272

73-
7473
WORKSPACE_COMPOSER_GLOBAL_INSTALL=true
7574
WORKSPACE_COMPOSER_REPO_PACKAGIST=
7675
WORKSPACE_INSTALL_NODE=true
@@ -88,6 +87,7 @@ WORKSPACE_INSTALL_MONGO=false
8887
WORKSPACE_INSTALL_AMQP=false
8988
WORKSPACE_INSTALL_MSSQL=false
9089
WORKSPACE_INSTALL_DRUSH=false
90+
WORKSPACE_DRUSH_VERSION=8.1.16
9191
WORKSPACE_INSTALL_DRUPAL_CONSOLE=false
9292
WORKSPACE_INSTALL_AEROSPIKE=false
9393
WORKSPACE_INSTALL_V8JS=false
@@ -348,7 +348,3 @@ LARAVEL_ECHO_SERVER_PORT=6001
348348
SOLR_VERSION=5.5
349349
SOLR_PORT=8983
350350
SOLR_DATAIMPORTHANDLER_MYSQL=false
351-
352-
### DRUSH_VERSION #########################################
353-
354-
DRUSH_VERSION=9.2.3

workspace/Dockerfile

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -299,13 +299,17 @@ RUN if [ ${INSTALL_SWOOLE} = true ]; then \
299299
# Drush:
300300
###########################################################################
301301

302+
# Deprecated install of Drush 8 and earlier versions.
303+
# Drush 9 and up require Drush to be listed as a composer dependency of your site.
304+
302305
USER root
303306

304307
ARG INSTALL_DRUSH=false
308+
ARG DRUSH_VERSION
309+
ENV DRUSH_VERSION ${DRUSH_VERSION}
305310

306311
RUN if [ ${INSTALL_DRUSH} = true ]; then \
307312
apt-get -y install mysql-client && \
308-
# Install Drush 8 with the phar file.
309313
curl -fsSL -o /usr/local/bin/drush https://github.com/drush-ops/drush/releases/download/${DRUSH_VERSION}/drush.phar | bash && \
310314
chmod +x /usr/local/bin/drush && \
311315
drush core-status \

0 commit comments

Comments
 (0)