Page MenuHomePhorge

Correct call to non-existing PhutilFileLockException in support/unit/lock.php
ClosedPublic

Authored by aklapper on May 16 2024, 08:53.
Tags
None
Referenced Files
F3360301: D25641.1743979437.diff
Sat, Apr 5, 22:43
F3360299: D25641.1743979429.diff
Sat, Apr 5, 22:43
F3359425: D25641.1743944181.diff
Sat, Apr 5, 12:56
F3358226: D25641.1743920095.diff
Sat, Apr 5, 06:14
F3331576: D25641.1743509839.diff
Mon, Mar 31, 12:17
F3331202: D25641.1743503001.diff
Mon, Mar 31, 10:23
F3328976: D25641.1743467330.diff
Mon, Mar 31, 00:28
F3328022: D25641.1743448516.diff
Sun, Mar 30, 19:15

Details

Summary

PhutilFileLockException does not exist. Per https://we.phorge.it/source/arcanist/browse/master/src/filesystem/PhutilFileLock.php$54-66 , if lock acquisition fails it is supposed to throw a PhutilLockException instead.

Test Plan

Grep the code base.

Or, run this, from arcanist, from two different terminals:

php ./support/unit/lock.php asd.txt

After this change, the exception is correctly managed.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable