Skip to content

Stdlib: withhold brotli on 8.2 reference profile (#17563)#17569

Merged
PurHur merged 1 commit into
masterfrom
agent/issue-17563-brotli-phantom-extension
Jul 10, 2026
Merged

Stdlib: withhold brotli on 8.2 reference profile (#17563)#17569
PurHur merged 1 commit into
masterfrom
agent/issue-17563-brotli-phantom-extension

Conversation

@PurHur

@PurHur PurHur commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Summary

Test plan

  • Issue repro (reference profile):
./script/docker-exec.sh -- bash -lc 'export PHP_COMPILER_PROFILE=8.2
php bin/vm.php test/repro/maintainer_gap_brotli_phantom_extension.php'
# ok
  • Compliance phantom guard:
./script/docker-exec.sh -- bash -lc 'source script/php-env.sh && vendor/bin/phpunit --filter brotli_phantom test/compliance/VMTest.php'
# OK (1 test)
  • Forward profile round-trip still works:
./script/docker-exec.sh -- bash -lc 'export PHP_COMPILER_PROFILE=8.4
source script/php-env.sh && vendor/bin/phpunit --filter brotli_roundtrip test/compliance/VMTest.php'
# OK (1 test)
  • Unit: BrotliExtensionPolicyTest, CompilerVersionGateTest::testSupportsBrotli*
  • ./script/ci-fast.sh (inventory + spine footnotes regen)

Closes #17563

Made with Cursor

Gate ext/brotli registration and extension_loaded() behind BrotliExtensionPolicy
matching the bz2 phantom pattern — Zend 8.2 Docker has no ext-brotli.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur PurHur merged commit 2dd4097 into master Jul 10, 2026
1 check failed
@PurHur PurHur deleted the agent/issue-17563-brotli-phantom-extension branch July 10, 2026 07:00
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.

Regression: extension_loaded('brotli') / brotli_compress() true on 8.2 reference profile (ext/brotli)

1 participant