-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
PHP7 Support #10378
Comments
This is not an easy task, migration to PHP7 must have to be done in Zephir: zephir-lang/zephir#952 |
@andresgutierrez PHP 7 will be far faster than php5, also there is HHVM and Hacklang. Additionally, you have said migrating Zephir to PHP 7 is really time and effort demanding if not impossible. What is the point in migrating Zephir to PHP7. Why not you port the lovely Phalon to plain PHP instead? A huge number of PHP programmer can contribute, we will have 16 github issues instead of 612 issues. Those who started large project using Phalcon have not to migrate all of their code to another PHP framework and Phalcon will live forever. Phalcon can be "The PHP Framework For Web Artisans". Facebook and Zend have a lot of money to make PHP fast and faster. Please focus on pleasure and quality of PHP programming. |
@salarmehr I said is not an easy task not that it is impossible, I can't see your point, PHP7 is not faster than Zephir can be, also, Zephir is a powerful and easy language to use and more people is contributing to Phalcon. Do you think that write the PHQL/Volt parser in PHP7 will be faster than the one written in pure C? One thing is that you want to believe something and other is that it's real |
@salarmehr facepalm |
I did not claimed that PHP7 is faster than Zephir. I said "PHP 7 will be far faster than php5". I can understand you. You spent long time developing Zephir, on the other hand, I and others in Phalcon community investigate on Phalcon, we learned it deeply and used it as a foundation for our projects. In Zephir home page you introduced it as a single-purpose language: "designed to ease the creation and maintainability of extensions for PHP ". Its a programming language designed for another language. If you plan to make it more versatile and writing pure-zephir web application possible (similar to what Hack do) it's a different story. Please also take a look at this page: laravel/framework#3295 |
What is PHP 6? |
@hdogan My bad, PHP5 ◕‿◕ |
Major 👎 to switching away from zephir. |
Guys: http://php.net/archive/2015.php#id2015-12-03-1
|
@andresgutierrez , maybe if you create a list of "things" that has to be changed, with files and a very tiny hints, in a separated issue, create a new branch ( maybe 2.2 or directly on 2.1.x or 3.0 ?), and see if any of us can help you make phalcon works with php7. Unless you are already at the finishing point :D |
I'd like to see Phalcon/Zephir ported to PHP 7. I'm using it for 18 months on a very big project (the project wasn't launched yet) with 10 other developers and would like to be able to continue using it. And now that PHP 7 has been released, a lot of people will start using it. Of course, there is a window of time where the most paranoid, like myself, will not update to the new version so soon. But definitely we would like that to happen in a good time-frame. |
Zephir works with PHP 7 (zephir-lang/zephir/issues/756), but there is still some C code in Phalcon which needs to be fixed. |
+1 |
2 similar comments
+1 |
👍 |
+1 |
+1 |
@quantumpacket, @phpclub, @JakeGlascock, @PHEcho, @zjhzxhz, @Rewt0r, @hakimio,@andrefauth, @Izopi4a, @JCMais, @Green-Cat, @salarmehr, @hdogan Hey mans, now you can try Phalcon 2.1.x with PHP 7. Just use latest Zephir with |
Well, is there anything need for test or it should work as it is ? |
i will report tomorrow, busy playing diablo 3 atm :D |
@Izopi4a 😄 anyway, create a separated issue |
zephir build —backend=ZendEngine3 |
Under centos 6.x
|
/root/Soft/cphalcon/ext/phalcon/annotations/reader.zep.c: В функции ‘zim_Phalcon_Annotations_Reader_parse’: |
PHP 7.0.3 (cli) (built: Feb 3 2016 11:40:05) ( NTS ) |
In Phalcon 2.1 support for APC should be dropped in favor of APCu. |
@phpclub build and isntall this: https://github.com/krakjoe/apcu also make sure you do zephir fullclean before zephir build and i am not 100% sure but i think i increased my memory limit ( while building it ) i am not sure if problem was fixed as soon as i instakked apcu |
Oh, nice. Thanks for the heads up. I'll let you know what happens.
|
Thanx zephir fullclean
PHP Notice: Use of undefined constant APC_ITER_ALL - assumed 'APC_ITER_ALL' in /root/Soft/zephir/Library/ClassDefinition.php on line 1743
Warning: Variable "possibleSetter" declared but not used in Phalcon\Mvc\Model::assign in /root/Soft/cphalcon/phalcon/mvc/model.zep on 440 [unused-variable]
Warning: Function "\sodium\randombytes_buf" does not exist at compile time in /root/Soft/cphalcon/phalcon/security/random.zep on 119 [nonexistent-function]
Warning: Function "\sodium\randombytes_uniform" does not exist at compile time in /root/Soft/cphalcon/phalcon/security/random.zep on 310 [nonexistent-function]
Copying new kernel files... |
Done! Where to download the stress tests for phalcon? PHP Version 7.0.3 |
If someone puts out some DLLs for Windows I can stress test them on my project. |
Failed to compile it in OS X 10.11. PHP Information:
Zephir Information:
Error Log:
|
@zjhzxhz
try to compile again |
Debian x64 Jessie Everything compiled just fine, the Phalcon extension is loaded without error and shows up in Any hints on how to get a stacktrace / give you guys useful info? Edit: Please disregard, I've tried it on another machine and it's working! Must be some environmental problem at my end, because the 2nd is a virtual machine with the same software versions. |
Here's the output of executing
And it's still not working. I got the same error message as previous. |
@zjhzxhz do you have extension=/private/tmp/zephir/Library/../parser/zephir_parser.so in your php.ini by any chance ? if so remove it. if you type do you have skype or something that we can chat and give it a try to fix it ? |
just wanted to say , it worked perfectly thanks |
Under Ubuntu 14.04 cphalcon# zephir build —backend=ZendEngine3
|
No installation problems with ubuntu 14.04 here but with defaults events like beforeExecuteRoute. |
opened a bug here about contruct methods zephir-lang/zephir#1203 i need those in order to test the rest |
@andresgutierrez could you please fix this zephir-lang/zephir#1203 so we can test the rest ? |
+1 |
I believe I have found another bug. |
What happens if you use:
|
same error |
Well we need to deal currently with seg faults in tests, then if in test it will fail too i will add it to this list: |
For me view doesn't even work in php 7. Such a code:
Displays nothing form test.volt even its adasdasdasdasd EDIT: on php7 you have to create volt engine as an object, string doesn't work |
+1 |
Introduced in v3.0.0 |
PHP7 betas should be rolling out next month, so I think at least master should support it for testing purposes.
In PHP7
etc/standard/php_smart_str.h
has been renamed toetc/standard/php_smart_string.h
.Compilation for master, 2.0.0, and 2.0.x both fail with the following error:
I don't know C, otherwise would submit a patch. :)
The text was updated successfully, but these errors were encountered: