Re: Ported some PHP5 PECL extensions to PHP7

From: Date: Sun, 28 Feb 2016 13:25:54 +0000
Subject: Re: Ported some PHP5 PECL extensions to PHP7
References: 1 2 3 4 5  Groups: php.pecl.dev 
Request: Send a blank email to pecl-dev+get-13578@lists.php.net to get a copy of this message
I have accepted PR`s. And - yes I have backup of bitbucket repo - I can try to push them to https://github.com/esminis/php_pecl_rar as a separate branch or send you a zip. But I did not quite understand why you need it - there was only PHP7 rewrite, there was nothing for PHP5. On 02/28/2016 01:02 PM, Jan Ehrhardt wrote:
Tautvydas in php.pecl.dev (Sun, 28 Feb 2016 09:57:41 +0200):
I have created repository in github: https://github.com/esminis/php_pecl_rar If you can - create pull request (if you can`t then I can of course copy paste but there will be no references to you).
2 PR's created (with due reference to @weltling).
And about TS build - you will probably need some #ifdef because seems that if ZTS is defined it is different, see #define RAR_G(v) in php_rar.h I will look into this sometime later(not exactly sure when) if you won`t.
The same code is used in the PHP5 version, without #ifdef's. Something must have changed in the ZTS version of RAR_G(v) or VC14 is more strict than VC11 was. I do not know how to fix it.
After windows version is fixed I will try to do pull request to main repository maybe maintainer will accept it.
OK.
https://bitbucket.org/esminis/php_pecl_rar
It is a pity you deleted that repo, because the history of the non-PHP7 commits is now lost. I wanted to improve the PHP5 branch with your findings, but cannot do that anymore. Do you still have the commit history as it was on bitbucket? Jan


« previous php.pecl.dev (#13578) next »