Re: New libfann binding

On Thu, Jul 18, 2013 at 12:58 AM, Evan Nemerson <evan@coeus-group.com>wrote: > On Thu, 2013-07-18 at 01:37 +0200, Johannes Schlüter wrote: > > On Wed, 2013-07-17 at 23:53 +0100, Jakub Zelenka wrote: > > > There already is an extension for fann lib ( > > > http://pecl.php.net/package/fann ) in PECL. I would > > > like to start > > > maintaining that extension and replace it with my new extensions if > there > > > are no objections? I CC'd Evan Nemerson who is the current maintainer > of > > > the fann extension as he can confirm that he is ok with that. > > > > Is you're API compatible with the old extension? If not: Is there some > > migration path? > > I don't think this should be a concern. The FANN 2.0 API is not > backwards compatible, I don't think it's reasonable to expect language > bindings to be. Furthermore, I don't think many people (if any) > actually use this extension--I'm not aware of anyone who does, and the > only messages I get concerning it are occasional requests for a version > which works with FANN 2.0. To be honest, I don't even know if it works > with newer versions of PHP. > > > -Evan > > As Evan said the FANN 2 is not BC. FANN 1 is not available in any new distro (fann 2.1 was released 6 years ago). I wouldn't mind to add a migration path anyway but the main problem is that the old extension does not work with newer versions of PHP. And when I say newer I don't really mean new :). It had to stop working when ZEND_METHOD(fce_name) macro evaluation was changed and used zim prefix in place of zif. I guess that it was changed in PHP 5.0 but not sure about that. Is it ok to update the extension without migration path in this case? Regards Jakub