-
Notifications
You must be signed in to change notification settings - Fork 164
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.
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
🐛 [Bug] Cypht not compatible with PHP 8.4 #1431
Comments
@IrAlfred @Baraka24 @Bienvenumoringa @amaninyumu1 @christer77 Who can take this on? |
FYI, deprecated means it still works but there will be issues in the future. The E_STRICT constant will be removed in PHP 9.0. If an application tries to support a wide range of PHP versions, it can become a little tricky. Cypht 2.x requires PHP 8.1+ as per our policy: https://github.com/cypht-org/cypht/wiki/Lifecycle "PHP core and core extensions since PHP 8.0 and later do not emit E_STRICT notices at all. It is safe to assume that any PHP applications that run on PHP 8.0 and later will never encounter E_STRICT notices, and error reporting and handling can be safely updated to ignore E_STRICT notices." Source: https://php.watch/versions/8.4/E_STRICT-deprecated |
wait One of the warnings is: Deprecation Notice: Constant E_STRICT is deprecated in phar:///opt/cpanel/composer/bin/composer/src/Composer/Util/Silencer.php:36 This looks like it's supplied by cPanel. In Tiki Manager, we did this so Tiki Manager would use its own Composer instead of one perhaps supplied by the server: https://gitlab.com/tikiwiki/tiki-manager/-/merge_requests/237 This also reminds me of #1154 |
Cypht appears to not be compatible with PHP 8.4. Running the bash script from the install page generated a log of over 7,000 lines, with most entries like the following:
So I dropped the PHP version on the server to 8.13 and it installed without those notices. The only thing that looked a little strange were the following lines:
I've attached the install log from the 8.4 attempt, in case that helps.
Cypht istall log.txt
The text was updated successfully, but these errors were encountered: