[pkg-php-pear] twig update to PHP 7.0

Daniel Beyer dabe at deb.ymc.ch
Fri Apr 15 15:35:50 UTC 2016


Hi Ondřej,

Am Freitag, den 15.04.2016, 16:45 +0200 schrieb Ondřej Surý:
> David and Daniel,
> 
> twig and symfony are the two last packages depending on dh-php5 which I
> intend to fill.
> 

For the last upload of those, we decided to stick with the php5
extensions as long as php5 is around in sid. Are you only removing
dh-php5 or php5 itself?

> As for twig I suggest:
> 
> - get rid of everything php5
> - merge PEAR and PECL parts into one single package into single php-twig
> arch:any package[1]
> 

I think this is a good idea for twig.
@David: Any objections from your side against this approach?


> and because PHP 7.0 for ext/twig hasn't even started:
> 
> - disable ext/twig building for now and leave php-twig arch:all for the
> moment (change to arch:any when PHP 7.0 support is added by upstream)
> 

Upstream is undecided whether or not to provide a PHP 7 version of the
extension [1]. Thus not building it at all is our only option for now.
[1] https://github.com/twigphp/Twig/issues/1695


> I pushed these changes to the git repository.
> 

I only had a quick look on it so far, the php-twig.README.Debian should
be updated, since it now mentions the non-existing PHP extension - but
the rest looks good. Thanks a lot for your work.

> Please let me know how do you want to proceed - I could do the upload,
> you could do the upload, you can prepare something really different, or
> I can shrug my shoulders, remove dh-php5 anyway, fill a serious FTBFS
> bug and just don't care :). Your choice...
> 

I check on twig as soon as possible (hopefully before Monday). If you
could do the upload, that would be great so I do not need to bother
David doing it for me.


> Same goes for symfony where ext/README.md states:
> 
> > It is compatible with PHP 5.3, 5.4, 5.5 and 5.6; with ZTS and non-ZTS modes.
> > It is not required thus not provided for PHP 7.
> 

Yes, there will be no PHP 7 extension and it thus is safe to permanently
remove it from the packaging.

> As there's a symfony in experimental, I took a liberty of updating the
> package in the experimental directly.

That's alright.
For stretch we target the 2.8 series of Symfony, since it is an LTS
series. I thus cherry-picked your commit into 2.8 - again, thanks for
your work here.


Greetings
Daniel




More information about the pkg-php-pear mailing list