Skip to content

Commit c9c5716

Browse files
committed
fix: allow installing extensions on PHP 8.1 normally
When releasing 1.14.0, I missed that the extensions script had a special case for 8.1, and discovered it later (https://github.com/mezzio/mezzio-authentication/runs/4397696325?check_suite_focus=true#step:3:17). This patch removes that case by commenting it out, but leaves the comment as an example of how to vary behavior based on PHP version. Signed-off-by: Matthew Weier O'Phinney <[email protected]>
1 parent 645680c commit c9c5716

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

scripts/extensions.sh

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,13 @@ function install_extensions {
1414
done
1515

1616
case "$PHP" in
17-
8.1)
18-
echo "Cannot install extensions for the current PHP version."
19-
echo "Please use \".laminas-ci/pre-run.sh\" to setup specific extensions for PHP $PHP"
20-
echo "Additional details can be found on https://stackoverflow.com/q/8141407"
21-
echo "The following extensions were not installed: ${EXTENSIONS[*]}"
22-
;;
17+
# Example for handling extensions for different PHP versions:
18+
# 8.1)
19+
# echo "Cannot install extensions for the current PHP version."
20+
# echo "Please use \".laminas-ci/pre-run.sh\" to setup specific extensions for PHP $PHP"
21+
# echo "Additional details can be found on https://stackoverflow.com/q/8141407"
22+
# echo "The following extensions were not installed: ${EXTENSIONS[*]}"
23+
# ;;
2324
*)
2425
install_packaged_extensions "$PHP" "${EXTENSIONS[@]}"
2526
;;

0 commit comments

Comments
 (0)