Re: [INTERNALS-WIN] Re: ZIP stream missing from beta builds of 8.2

--0000000000005e5d4305e57f407e
Content-Type: text/plain; charset="UTF-8"

I would argue that it should be active in the example INI files by default
and that there should be a huge warning about it in the migration to 8.2
documentation. Without it a lot of things are quite likely to break when
moving to 8.2.

Dave


On Fri, Aug 5, 2022 at 10:24 AM Jan Ehrhardt <[email protected]> wrote:

> "Christoph M. Becker" in php.internals.win (Fri, 5 Aug 2022 16:14:08
> +0200):
> >On 05.08.2022 at 15:29, David Martin wrote:
> >
> >> Both beta 1 and beta 2 are missing the ZIP stream in their builds which
> >> appears to be breaking just about anything that tries to use ZipArchive
> >> (for example).
> >
> >The zip extension is now built as shared library (DLL)[1], so you need
> >to load it explicitly (usually via php.ini).
> >
> >[1]
> ><
> https://github.com/php/php-src/blob/1cd2d731ef62b25165c45ae8475f4aa3fedb5f5b/UPGRADING#L513
> >
>
> But extension=zip should have been added to the templates for php.ini
>
> https://github.com/php/php-src/blob/33fc9e5a770d54e0d3c38c2ba0cfad7c40e05b00/php.ini-development#L951
> and
>
> https://github.com/php/php-src/blob/33fc9e5a770d54e0d3c38c2ba0cfad7c40e05b00/php.ini-production#L953
> --
> Jan
>
> --
> Windows Internals Mailing List (http://www.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

--0000000000005e5d4305e57f407e--