Skip to content

Commit

Permalink
PHP 8.1: fix Phar building
Browse files Browse the repository at this point in the history
For the time being, until box-project/box#567 has been fixed:
1. Disable the addition of the requirement checker from Box as the version included in the last release of Box is not compatible with PHP 8.1.
2. Change the PHP version used to generate the Phar to PHP 8.0 to get round the problem the PHP compactor has with attributes.
    Note: the generated Phar file should still be compatible with all supported PHP versions.
  • Loading branch information
jrfnl committed Oct 10, 2021
1 parent a2d64fa commit 7cf03af
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: 7.3
php-version: 8.0
extensions: exif, phar, openssl
coverage: none
ini-values: phar.readonly=Off
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: 7.3
php-version: 8.0
extensions: exif, phar, openssl
coverage: none
tools: phive
Expand Down
1 change: 1 addition & 0 deletions box.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
"base-path": null,
"output": "parallel-lint.phar",
"chmod": "0755",
"check-requirements": false,
"compactors": [
"KevinGH\\Box\\Compactor\\Php"
],
Expand Down

0 comments on commit 7cf03af

Please sign in to comment.