Skip to content

Fix ev and event extensions on PHP 7.4 - #414

Merged
mistraloz merged 1 commit into
thecodingmachine:v5from
mistraloz:fix/php74-ev-event
Oct 6, 2026
Merged

mistraloz merged 1 commit into
thecodingmachine:v5from
mistraloz:fix/php74-ev-event

Conversation

@mistraloz

Copy link
Copy Markdown
Collaborator

Problem

Every PHP 7.4 job of "Direct build v5 of legacy php" fails at the install_all.sh step:

***************** Failed extensions: ev event ******************

These extensions were already broken, but the error was silently ignored. Since #411, install_all.sh uses pipefail and fails the build when an extension cannot be installed.

  • ev: the latest PECL release (1.2.x) requires PHP 8.0 (configure: error: need at least PHP 8.0.0).
  • event: the libssl1.1 dependency does not exist on Ubuntu 24.04 (Unable to locateevent packages were also renamed (t64` transition).

Changes

  • extensions/7.4/ev: replace the symlink to core/ev with a real directory that pins ompatible with PHP 7.4. PHP 8.0 still uses core/ev`.
  • extensions/core/event/install.sh: use the Ubuntu 24.04 package names (libevent-*-2.1-7t64, libssl3t64). Only PHP 7.4 uses this extension.
  • .github/workflows/legacy-v5.yml: docker --quiet pull → docker pull --quiet. The flag: --quiet), so cached images were never pulled. This did not fail the job because of|| echo ""`.

Tests

  • Local docker buildx bake php74-cli: build OK, no failed extension.
  • In the image, with PHP_EXTENSION_EV=1 PHP_EXTENSION_EVENT=1 PHP_EXTENSION_SOCKETS=1: are loaded.

Since install_all.sh fails on extension errors, every PHP 7.4 build fails:

- ev: the latest PECL release (1.2.x) requires PHP 8.0. Pin ev-1.1.5 for
  PHP 7.4 only (the 8.0 link to core is unchanged).
- event: libssl1.1 and the libevent packages were renamed on Ubuntu 24.04
  (t64 transition). Only PHP 7.4 uses this extension.

Also fix "docker --quiet pull" in the legacy v5 workflow: the flag was
misplaced, so cached images were never pulled.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mistraloz
mistraloz merged commit 7820fc8 into thecodingmachine:v5 Oct 6, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant