bcgen icon indicating copy to clipboard operation
bcgen copied to clipboard

PHP 8.0 ?

Open henrywood opened this issue 3 years ago • 8 comments

Hi !

Since PHP 8.0 will be released soon, I just wondered if we have "someone" to work on making bcgen PHP8.0 compatible ?

henrywood avatar Aug 04 '20 12:08 henrywood

I don't think anyone cares about bcgen anymore.

eoksum avatar Dec 23 '20 14:12 eoksum

@vjardin I would like to be candidate for further development of bcgen. I don't think you want to continue working on this project. I will, most likely. If you don't have any plans for future, can you transfer this repo to me?

eoksum avatar Dec 23 '20 14:12 eoksum

Hi! I do not think that it is necessary to transfer the repo to start working on a new version of bcgen which works for PHP-8.0. When I started with bcgen I studied the source code of bcompiler for old PHP versions to get an idea of how it is working.

Then with bcgen, I have decided that I did not want to create a bcgen version which is backwards compatible with older PHP releases. Instead it must follow each PHP version closely.

How to start? Just go to this directory where you find OpCache: https://github.com/php/php-src/tree/php-8.0.0/ext/opcache

You will immediately recognize that bcgen was created out of Opcache. bcgen is just a stripped down version of Opcache. To strip down PHP8 Opcache just analyse the code while setting the config option "opcache.file_cache_only" to true. So this will deactivate memory caching and so on.

You could also compare the PHP-7.2 opcache code with current bcgen to get an idea what I had removed.

Keep in mind that bcgen was never brought to a really stable state where all edge cases work. PHP-8.0 has changed a lot under the hood.

Furthermore, I do not need bcgen anymore. So do not expect any work from me.

nanosonde avatar Dec 23 '20 15:12 nanosonde

Hi there,

Thanks for your reply. I was thinking the same. But there are few links around to this repo so people came here and leave empty handed. I think you need to change the README to reflect the current status of Bcgen or maybe even make this repo a read-only one. It would be very convenient for people who will visit this page in the future.

eoksum avatar Dec 23 '20 17:12 eoksum

@eoksum , you get granted a write access.

vjardin avatar Dec 23 '20 18:12 vjardin

Hi @eoksum, did you manage to work on any upgrades ?

vjardin avatar Feb 02 '21 13:02 vjardin