Skip to content

Commit 848c098

Browse files
authored
Merge pull request #170 from andrewnicols/renameStandardIni
Rename moodle ini to support easier lexical sorting
2 parents ff023e2 + 3e9bf7c commit 848c098

File tree

4 files changed

+19
-5
lines changed

4 files changed

+19
-5
lines changed

.github/workflows/test_buildx_and_publish.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ jobs:
2929
-v $PWD/tests/docker-entrypoint.d:/docker-entrypoint.d \
3030
-e PHP_INI-memory_limit=256M \
3131
-e PHP_INI-apc.enabled=0 \
32+
-e PHP_INI-apc.enable_cli=0 \
33+
-e PHP_INI-pcov.enabled=1 \
3234
moodle-php-apache
3335
docker exec test0 php /var/www/html/test.php
3436
docker exec test0 php /var/www/html/check-ini.php

root/tmp/setup/php-extensions.sh

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -67,12 +67,12 @@ docker-php-ext-install -j$(nproc) ldap
6767
pecl install apcu igbinary memcached pcov redis solr timezonedb uuid xmlrpc-beta
6868
docker-php-ext-enable apcu igbinary memcached pcov redis solr timezonedb uuid xmlrpc
6969

70-
echo 'apc.enable_cli = On' >> /usr/local/etc/php/conf.d/docker-php-ext-apcu.ini
70+
echo 'apc.enable_cli = On' >> /usr/local/etc/php/conf.d/10-docker-php-ext-apcu.ini
7171

72-
echo "pcov.enabled=0" >> /usr/local/etc/php/conf.d/docker-php-ext-pcov.ini
73-
echo "pcov.exclude='~\/(tests|coverage|vendor|node_modules)\/~'" >> /usr/local/etc/php/conf.d/docker-php-ext-pcov.ini
74-
echo "pcov.directory=." >> /usr/local/etc/php/conf.d/docker-php-ext-pcov.ini
75-
echo "pcov.initial.files=1024" >> /usr/local/etc/php/conf.d/docker-php-ext-pcov.ini
72+
echo "pcov.enabled=0" >> /usr/local/etc/php/conf.d/10-docker-php-ext-pcov.ini
73+
echo "pcov.exclude='~\/(tests|coverage|vendor|node_modules)\/~'" >> /usr/local/etc/php/conf.d/10-docker-php-ext-pcov.ini
74+
echo "pcov.directory=." >> /usr/local/etc/php/conf.d/10-docker-php-ext-pcov.ini
75+
echo "pcov.initial.files=1024" >> /usr/local/etc/php/conf.d/10-docker-php-ext-pcov.ini
7676

7777
# Keep our image size down..
7878
pecl clear-cache

tests/fixtures/check-ini.php

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,23 @@
1717
$message[] = "apc.enabled is not Off (0): ({$apcenabled})";
1818
}
1919

20+
$apcenabledcli = ini_get('apc.enabled_cli');
21+
if (!empty($apcenabledcli)) {
22+
$allokay = false;
23+
$message[] = "apc.enabled_cli is not Off (0): ({$apcenabledcli})";
24+
}
25+
2026
if ($memorylimit !== '256M') {
2127
$allokay = false;
2228
$message[] = "Memory limit not set to 256M: ({$memorylimit})";
2329
}
2430

31+
$pcovenabled = ini_get('pcov.enabled');
32+
if (empty($pcovenabled)) {
33+
$allokay = false;
34+
$message[] = "pcov.enabled is not On (1): ({$pcovenabled})";
35+
}
36+
2537
if (php_sapi_name() === 'cli') {
2638
if ($allokay) {
2739
echo "OK\n";

0 commit comments

Comments
 (0)