PHP runner does exit if script file is locked
オープン
まだ誰も着手していません。
Bug
OS: Windows
Status: Needs Triage
- 主要言語
- C
- スター
- 40.4k
- フォーク
- 8.2k
- 平均マージ
- 2日 13時間
- マージ済み PR(30日)
- 96
説明
Description
The following code:
<?php
echo "Start script\n";
$fp = fopen(__FILE__, 'r+');
if (flock($fp, LOCK_EX, $block)) {
echo "Job running\n";
sleep(5);
echo "Job finished\n";
flock($fp, LOCK_UN);
} else {
if ($block) {
echo "Could not get the lock! Other script in run\n";
} else {
echo "Some other problem\n";
}
}
How to reproduce
- Save script above in some test file, for example,
a.php - Open 2 command prompts on Windows
- Run script in first command prompt
php -n a.php - While script is still running in first command prompt run the same in second command prompt window
php -n a.php
Result
- In first window script get lock and echo "Start script", "Job running" and after 5 seconds "Job finished"
- In second window script is not even run as do not echo even "Start script" and exits immediately without any error message or something
Expected output
- In second command prompt window script should execute and if condition with flock set $block=1 and echo "Could not get the lock! Other script in run"
PHP Version
PHP 8.2.5 (cli) (built: Apr 12 2023 08:41:53) (ZTS Visual C++ 2019 x86)
Copyright (c) The PHP Group
Zend Engine v4.2.5, Copyright (c) Zend Technologies
with Zend OPcache v8.2.5, Copyright (c), by Zend Technologies
Tested also on PHP 7.4 and that's the same there.
Operating System
Microsoft Windows [Version 10.0.19045.5679]
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
Windows 上で php -n を使用して a.php スクリプトの問題を再現し、2 つのコマンドプロンプトから同時に実行します。PHP CLI と Windows のファイルロック動作から調査を始めます。完了条件は、2 番目のプロセスが "Start script" を出力し、何も表示せずに終了するのではなく、既存のロックを報告する flock ブランチに到達することです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- php
- 領域
- cli, operating-systems
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100