PHP8 Support - #55
Merged
Merged
Conversation
added 5 commits
December 9, 2020 13:54
…n 24h, which makes debug faster.
- Renamed PHP recipe so it's not version specific - Removed LibMaxMindRecipe as Ubuntu bionic has a recent enough version - MaxMindRecipe now based off PeclRecipe as that extension is now available in PECL - Added PkgConfigLibRecipe base class which can be used as a by recipes installing libraries that support pkg-config. The benefit is that it won't recompile the library if it already exists. - Removed GeoipRecipe as this is no longer used - HiredisRecipe, LibSodiumRecipe, LibRdKafkaRecipe, CassandraCppDriverRecipe, and RabbitMQRecipe have all been switched to base off of PkgConfigLibRecipe - Removed LibmemcachedRecipe as this takes a very long time to compile & the version in Ubuntu bionic is now the most recent one. - Removed UnixOdbcRecipe as we can now use the version of UnixODBC that ships with Ubuntu bionic - OdbcRecipe & PdoOdbcRecipe have been switched to use version of UnixODBC bundled with Ubuntu bionic - PsrRecipe & PhalconRecipe have been removed as these extension are available on PECL now so they can be build with the standard PeclRecipe - Added EnchantFakePecl recipe as we need a patch to make enchant extension compile under PHP8. It is not needed for PHP7.x. - Don't delete PHP build files when packaging up the archive
Author
|
I hacked together a little example to try loading all of the extensions and it seems to work. The SNMP errors have always been present. Not sure what's up with YAF, that might be something that needs to be fixed with PHP8, but they seem to be just warnings. Otherwise, everything seems to load with is a good sign. |
Author
|
Test with PHP 7.4 went well. Here's the extensions YAML that worked for me. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds support to build PHP8 and some other clean up.
Used the attached extensions file. Please see the extensions file as there are comments regarding extensions which do not yet support PHP8.
php8-extensions.zip
To execute, run:
./bin/binary-builder --name=php --version=8.0.0 --sha256=3ed7b48d64357d3e8fa9e828dbe7416228f84105b8290c2f9779cd66be31ea71 --php-extension-file=./php8-extensions.yml. This is slightly different from before as I've consolidatedphpandphp7into justphpfor all versions.