Skip to content

Commit 2e928a4

Browse files
vlaucianibestlong
authored andcommitted
laravel-horizon: Install 'sockets' without install 'AMQP` (laradock#2211)
* Add LARAVEL_HORIZON_INSTALL_SOCKETS variable. Issue laradock#2209 * Get LARAVEL_HORIZON_INSTALL_SOCKETS from .env. Issue laradock#2209 * Get INSTALL_SOCKETS variable and install 'sockets' ext. Issue laradock#2209
1 parent 51c6d35 commit 2e928a4

3 files changed

Lines changed: 7 additions & 3 deletions

File tree

docker-compose.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,7 @@ services:
251251
- INSTALL_PGSQL=${PHP_FPM_INSTALL_PGSQL}
252252
- INSTALL_BCMATH=${PHP_FPM_INSTALL_BCMATH}
253253
- INSTALL_MEMCACHED=${PHP_FPM_INSTALL_MEMCACHED}
254-
- INSTALL_AMQP=${PHP_FPM_INSTALL_AMQP}
254+
- INSTALL_SOCKETS=${LARAVEL_HORIZON_INSTALL_SOCKETS}
255255
- INSTALL_CASSANDRA=${PHP_FPM_INSTALL_CASSANDRA}
256256
volumes:
257257
- ${APP_CODE_PATH_HOST}:${APP_CODE_PATH_CONTAINER}

env-example

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -222,6 +222,10 @@ NGINX_PHP_UPSTREAM_CONTAINER=php-fpm
222222
NGINX_PHP_UPSTREAM_PORT=9000
223223
NGINX_SSL_PATH=./nginx/ssl/
224224

225+
### LARAVEL_HORIZON ################################################
226+
227+
LARAVEL_HORIZON_INSTALL_SOCKETS=false
228+
225229
### APACHE ################################################
226230

227231
APACHE_HOST_HTTP_PORT=80

laravel-horizon/Dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,8 @@ RUN if [ ${INSTALL_BCMATH} = true ]; then \
3333
;fi
3434

3535
#Install Sockets package:
36-
ARG INSTALL_AMQP=false
37-
RUN if [ ${INSTALL_AMQP} = true ]; then \
36+
ARG INSTALL_SOCKETS=false
37+
RUN if [ ${INSTALL_SOCKETS} = true ]; then \
3838
docker-php-ext-install sockets \
3939
;fi
4040

0 commit comments

Comments
 (0)