GeezerVison

carlos-granados left a comment

Add support for compiling php_debugger as a static extension into php-src
using ./configure --enable-php-debugger (without --enable-shared).

Changes:
- MINIT self-registration: in static builds, the Zend extension is registered
  from PHP_MINIT_FUNCTION since zend_startup_extensions() has already run by
  the time MINIT fires. Manually calls xdebug_library_zend_startup() and
  xdebug_debugger_zend_startup() to replicate xdebug_zend_startup() logic.
- xdebug_zend_startup() early return: prevents double module registration
  when zend_startup_extensions() calls it after MINIT already initialized.
- php_php_debugger.h: bridge header providing phpext_php_debugger_ptr for
  PHP's static extension table (main/internal_functions.c).
- config.m4: add PHP_ADD_INCLUDE($ext_srcdir) for header discovery.
- Fix COMPILE_DL_XDEBUG -> COMPILE_DL_PHP_DEBUGGER in RINIT ZTS guard.

Benchmark (PHP 8.6.0-dev, bench.php):
  Vanilla PHP:           0.573s (baseline)
  Static php_debugger:   0.605s (~5.6% overhead, observer API floor)

Dedicated CI job that builds PHP from source with --enable-php-debugger
and verifies the extension loads correctly as a static build.

Only triggers on changes to config.m4, php_php_debugger.h, xdebug.c,
or the workflow itself — does not slow down the main test suite.

Smoke tests:
- Module list (php_debugger + xdebug compat)
- Zend extension registration
- extension_loaded() for both names
- INI settings (both prefixes)
- Function existence (both aliases)

config.m4 includes m4/pkg.m4 and m4/clocks.m4 which resolve relative
to the php-src root, not the extension directory.

Remove cache step (not worth complexity for a rarely-triggered workflow).
Always clone fresh — simpler and more reliable.

Registers php_debugger_info as a ZEND_FALIAS of xdebug_info so the
php_debugger_* function namespace has at least one user-visible entry
point alongside the php_debugger.* INI namespace.

Updates the static-build smoke test to assert php_debugger_info()
exists, replacing a stale assertion for php_debugger_break() (an alias
that was removed when arginfo got regenerated in #48).