Skip to content

ext/zip: Reject broken libzip versions on Windows - #24371

Merged
LamentXU123 merged 3 commits into
php:PHP-8.4from
LamentXU123:zip-windows-version-check
Oct 25, 2026
Merged

LamentXU123 merged 3 commits into
php:PHP-8.4from
LamentXU123:zip-windows-version-check

Conversation

@LamentXU123

Copy link
Copy Markdown
Member

Six years ago in f3e6b12 we reject them in .m4 but sadly forget to reject them in .w32

@saundefined

Copy link
Copy Markdown
Member

I have no objections, but please wait for the other RMs.

@LamentXU123 LamentXU123 mentioned this pull request Oct 18, 2026

@ramsey ramsey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no objections.

@ericmann

Copy link
Copy Markdown
Contributor

I like that libzip fixed their regression, but this doesn't feel like it warrants being bundled into a security release for 8.3.

@LamentXU123
LamentXU123 changed the base branch from PHP-8.2 to PHP-8.4 October 25, 2026 05:58
@LamentXU123
LamentXU123 force-pushed the zip-windows-version-check branch from d445d38 to 2823c30 Compare October 25, 2026 05:58
@LamentXU123
LamentXU123 force-pushed the zip-windows-version-check branch from 2823c30 to b107dce Compare October 25, 2026 16:26
@LamentXU123
LamentXU123 merged commit a1eb2f8 into php:PHP-8.4 Oct 25, 2026
1 of 2 checks passed
LamentXU123 added a commit that referenced this pull request Oct 25, 2026
* PHP-8.6:
  Merge branch 'PHP-8.5' into PHP-8.6
  Merge branch 'PHP-8.4' into PHP-8.5
  ext/zip: Reject broken libzip versions on Windows (#24371)
opensource-fork-sync Bot pushed a commit to cla7aye15I4nd/opensource-php__php-src that referenced this pull request Oct 25, 2026
* PHP-8.4:
  ext/zip: Reject broken libzip versions on Windows (php#24371)
opensource-fork-sync Bot pushed a commit to cla7aye15I4nd/opensource-php__php-src that referenced this pull request Oct 25, 2026
* PHP-8.5:
  Merge branch 'PHP-8.4' into PHP-8.5
  ext/zip: Reject broken libzip versions on Windows (php#24371)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants