Skip to content

Commit

Permalink
Merge pull request #113 from Automattic/CANTINA-982
Browse files Browse the repository at this point in the history
fix(php): fix dependencies for PHP 8.0, 8.2, and 8.3
  • Loading branch information
sjinks authored Oct 23, 2023
2 parents 608b264 + 4c0e258 commit 91709e5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion features/src/php/devcontainer-feature.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"id": "php",
"name": "PHP",
"description": "Installs PHP into the Dev Environment",
"version": "2.2.2",
"version": "2.2.3",
"containerEnv": {
"PHP_INI_DIR": "/etc/php"
},
Expand Down
6 changes: 3 additions & 3 deletions features/src/php/install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ PATH=/usr/local/bin:/usr/local/sbin:/bin:/sbin:/usr/bin:/usr/sbin

setup_php80() {
apk add --no-cache \
icu-data-full@edgem icu-libs@edgem libssl3@edgem ghostscript \
icu-data-full@edgem icu-libs@edgem libssl3@edgem libsodium@edgem ghostscript \
php8@edget \
php8-fpm@edget \
php8-pear@edget \
Expand Down Expand Up @@ -134,7 +134,7 @@ setup_php81() {

setup_php82() {
apk add --no-cache \
icu-data-full icu-libs@edgem libavif@edgem ghostscript \
icu-data-full icu-libs@edgem libavif@edgem libsodium@edgem ghostscript \
php82@edgec php82-fpm@edgec php82-pear@edgec \
php82-pecl-apcu@edgec \
php82-bcmath@edgec \
Expand Down Expand Up @@ -200,7 +200,7 @@ setup_php82() {

setup_php83() {
apk add --no-cache \
icu-data-full icu-libs@edgem libavif@edgem ghostscript \
icu-data-full icu-libs@edgem libavif@edgem libsodium@edgem ghostscript \
php83@edget php83-fpm@edget php83-pear@edget \
php83-pecl-apcu@edget \
php83-bcmath@edget \
Expand Down

0 comments on commit 91709e5

Please sign in to comment.