Do "pecl install" in series so it fails properly

14 files changed, 84 insertions, 70 deletions

diff --git a/12.0-rc/apache/Dockerfile b/12.0-rc/apache/Dockerfile
index b26eded9..78cd3f16 100644
--- a/12.0-rc/apache/Dockerfile
+++ b/12.0-rc/apache/Dockerfile
@@ -50,11 +50,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/12.0-rc/fpm-alpine/Dockerfile b/12.0-rc/fpm-alpine/Dockerfile
index a0edea2e..2831c80e 100644
--- a/12.0-rc/fpm-alpine/Dockerfile
+++ b/12.0-rc/fpm-alpine/Dockerfile
@@ -44,11 +44,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/12.0-rc/fpm/Dockerfile b/12.0-rc/fpm/Dockerfile
index 2b672966..de2f31ee 100644
--- a/12.0-rc/fpm/Dockerfile
+++ b/12.0-rc/fpm/Dockerfile
@@ -50,11 +50,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/12.0/apache/Dockerfile b/12.0/apache/Dockerfile
index e9e059d7..bdaf1cc6 100644
--- a/12.0/apache/Dockerfile
+++ b/12.0/apache/Dockerfile
@@ -50,11 +50,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/12.0/fpm-alpine/Dockerfile b/12.0/fpm-alpine/Dockerfile
index c75b3369..1029d501 100644
--- a/12.0/fpm-alpine/Dockerfile
+++ b/12.0/fpm-alpine/Dockerfile
@@ -44,11 +44,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/12.0/fpm/Dockerfile b/12.0/fpm/Dockerfile
index 019b6645..486f6564 100644
--- a/12.0/fpm/Dockerfile
+++ b/12.0/fpm/Dockerfile
@@ -50,11 +50,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/13.0-rc/apache/Dockerfile b/13.0-rc/apache/Dockerfile
index 4685a24f..9d909d4a 100644
--- a/13.0-rc/apache/Dockerfile
+++ b/13.0-rc/apache/Dockerfile
@@ -50,11 +50,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/13.0-rc/fpm-alpine/Dockerfile b/13.0-rc/fpm-alpine/Dockerfile
index 07262842..c90ec1e6 100644
--- a/13.0-rc/fpm-alpine/Dockerfile
+++ b/13.0-rc/fpm-alpine/Dockerfile
@@ -44,11 +44,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/13.0-rc/fpm/Dockerfile b/13.0-rc/fpm/Dockerfile
index 12fd39bf..a24a3eec 100644
--- a/13.0-rc/fpm/Dockerfile
+++ b/13.0-rc/fpm/Dockerfile
@@ -50,11 +50,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/13.0/apache/Dockerfile b/13.0/apache/Dockerfile
index 8f173530..fd710615 100644
--- a/13.0/apache/Dockerfile
+++ b/13.0/apache/Dockerfile
@@ -50,11 +50,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/13.0/fpm-alpine/Dockerfile b/13.0/fpm-alpine/Dockerfile
index 2b753276..1362f57c 100644
--- a/13.0/fpm-alpine/Dockerfile
+++ b/13.0/fpm-alpine/Dockerfile
@@ -44,11 +44,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/13.0/fpm/Dockerfile b/13.0/fpm/Dockerfile
index 26a67825..eca7f70d 100644
--- a/13.0/fpm/Dockerfile
+++ b/13.0/fpm/Dockerfile
@@ -50,11 +50,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-5.1.11 \
-        memcached-3.0.4 \
-        redis-3.1.6 \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-5.1.11; \
+    pecl install memcached-3.0.4; \
+    pecl install redis-3.1.6; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/Dockerfile-alpine.template b/Dockerfile-alpine.template
index 05cfc01b..392eb428 100644
--- a/Dockerfile-alpine.template
+++ b/Dockerfile-alpine.template
@@ -43,11 +43,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-%%APCU_VERSION%% \
-        memcached-%%MEMCACHED_VERSION%% \
-        redis-%%REDIS_VERSION%% \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-%%APCU_VERSION%%; \
+    pecl install memcached-%%MEMCACHED_VERSION%%; \
+    pecl install redis-%%REDIS_VERSION%%; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \
diff --git a/Dockerfile-debian.template b/Dockerfile-debian.template
index b2e2c7c1..ff0785b3 100644
--- a/Dockerfile-debian.template
+++ b/Dockerfile-debian.template
@@ -49,11 +49,12 @@ RUN set -ex; \
         pdo_pgsql \
         zip \
     ; \
-    pecl install \
-        APCu-%%APCU_VERSION%% \
-        memcached-%%MEMCACHED_VERSION%% \
-        redis-%%REDIS_VERSION%% \
-    ; \
+    \
+# pecl will claim success even if one install fails, so we need to perform each install separately
+    pecl install APCu-%%APCU_VERSION%%; \
+    pecl install memcached-%%MEMCACHED_VERSION%%; \
+    pecl install redis-%%REDIS_VERSION%%; \
+    \
     docker-php-ext-enable \
         apcu \
         memcached \