diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-08-12 06:29:43 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-08-15 09:19:39 +0900 |
| commit | bb4684f7d1c51bc1be9c1bda1b00cb52c839cd25 (patch) | |
| tree | 18fcdf1f9c9c64fc68e08bd5fab132808ca1cfcf /crates/shirabe/src/downloader | |
| parent | 9e5bb3e4253dda96345f4d1a3421077a72295048 (diff) | |
| download | php-shirabe-bb4684f7d1c51bc1be9c1bda1b00cb52c839cd25.tar.gz php-shirabe-bb4684f7d1c51bc1be9c1bda1b00cb52c839cd25.tar.zst php-shirabe-bb4684f7d1c51bc1be9c1bda1b00cb52c839cd25.zip | |
feat(php-shim): drop the modelled PHP version constants
The shim reported a fixed PHP 8.1.0 through PHP_VERSION, PHP_VERSION_ID,
the major/minor/release triple and the PHP_WINDOWS_VERSION_* trio. Their
uses split in two.
Some guarded branches PHP only needs on runtimes this port cannot be:
proc_get_status reports the exit status on every call, so Symfony's
pre-8.3 exit-code cache has nothing to work around; hash_raw and
hash_file always offer xxh3, so the sha1 fallback is unreachable; and
http_get_last_response_headers is always available, so the pre-8.4
$http_response_header branch is gone. safeJunctions reads the host
Windows version rather than PHP state, and joins the Windows work on
hold.
The rest ask about the PHP the user actually runs, and now reach the
worker through a new php-rpc PhpVersion payload: the startup banner and
the 7.2.5 warning, self-update's min-php filter, the ext-*
recommendation in VersionSelector, the stream User-Agent, and whether
PhpFileParser scans for enums.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/downloader')
| -rw-r--r-- | crates/shirabe/src/downloader/path_downloader.rs | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/crates/shirabe/src/downloader/path_downloader.rs b/crates/shirabe/src/downloader/path_downloader.rs index 5fa69ec7..8586de3d 100644 --- a/crates/shirabe/src/downloader/path_downloader.rs +++ b/crates/shirabe/src/downloader/path_downloader.rs @@ -22,8 +22,7 @@ use crate::util::Platform; use crate::util::ProcessExecutor; use indexmap::IndexMap; use shirabe_php_shim::{ - PHP_WINDOWS_VERSION_MAJOR, PHP_WINDOWS_VERSION_MINOR, PhpMixed, RuntimeException, file_exists, - function_exists, impl_php_class, is_dir, realpath, + PhpMixed, RuntimeException, file_exists, function_exists, impl_php_class, is_dir, realpath, }; use shirabe_symfony_filesystem::Filesystem as SymfonyFilesystem; @@ -188,9 +187,10 @@ impl PathDownloader { // The PHP bug was fixed in 7.2.16 and 7.3.3 (requires at least Windows 7). fn safe_junctions(&self) -> bool { // We need to call mklink, and rmdir on Windows 7 (version 6.1) - function_exists("proc_open") - && (PHP_WINDOWS_VERSION_MAJOR > 6 - || (PHP_WINDOWS_VERSION_MAJOR == 6 && PHP_WINDOWS_VERSION_MINOR >= 1)) + // TODO(windows): PHP reads the Windows version off PHP_WINDOWS_VERSION_MAJOR and + // PHP_WINDOWS_VERSION_MINOR, which describe the host rather than PHP; this port has to + // ask the OS for it. + todo!() } } |
