Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Hotfix] Fix Phar building #70

Merged
merged 2 commits into from
Dec 1, 2021
Merged

[Hotfix] Fix Phar building #70

merged 2 commits into from
Dec 1, 2021

Commits on Dec 1, 2021

  1. Upgrade to Box 3 for Phar generation 

    Box 2 is no longer maintained and will not be made compatible with more recent PHP versions.

Refs:
* https://github.com/box-project/box/blob/master/UPGRADE.md#from-27-to-30
* https://github.com/box-project/box/blob/master/doc/configuration.md
    @jrfnl
    jrfnl committed Dec 1, 2021
    Configuration menu
    Copy the full SHA
    2dc7c7c View commit details
    Browse the repository at this point in the history

  2. PHP 8.1: fix Phar building 

    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.
    @jrfnl
    jrfnl committed Dec 1, 2021
    Configuration menu
    Copy the full SHA
    daf0dfa View commit details
    Browse the repository at this point in the history