From 265c42d4bf2999034289df08418a29f014696091 Mon Sep 17 00:00:00 2001
From: Akira Yamamoto <3007213+akirayamamoto@users.noreply.github.com>
Date: Tue, 9 Jun 2026 22:27:23 +1000
Subject: [PATCH] docs: trim Docker PHP ini comments

Drop the Debian COPY comment (it only restated the code) and the
version-fragility framing in the Alpine comment.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 Dockerfile        | 3 ---
 Dockerfile.alpine | 2 +-
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index f969242..ce75766 100755
--- a/Dockerfile
+++ b/Dockerfile
@@ -10,9 +10,6 @@ RUN install-php-extensions iconv gd pdo pdo_mysql pdo_pgsql pgsql \
     && apt-get clean \
     && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*
 
-# PHP_INI_DIR is set by the official php:8-apache image to /usr/local/etc/php
-# and has been stable across PHP majors. Using the env var documents intent
-# and follows any future upstream change to the path automatically.
 COPY docker/librespeed-php.ini ${PHP_INI_DIR}/conf.d/99-librespeed.ini
 
 # Prepare files and folders
diff --git a/Dockerfile.alpine b/Dockerfile.alpine
index 1eecd35..e62ad9e 100755
--- a/Dockerfile.alpine
+++ b/Dockerfile.alpine
@@ -20,7 +20,7 @@ RUN apk add --quiet --no-cache \
 RUN ln -sf /dev/stdout /var/log/apache2/access.log && \
     ln -sf /dev/stderr /var/log/apache2/error.log
 
-# Use PHP's scan dir without pinning the apk PHP major.
+# Install the ini into the scan dir PHP itself reports.
 COPY docker/librespeed-php.ini /tmp/librespeed-php.ini
 RUN set -eu; \
     scan_dir="$(php -r 'echo rtrim(PHP_CONFIG_FILE_SCAN_DIR);')"; \