Page MenuHomePhabricator
Differential D17200

Where available, use async_signals in libphutil
ClosedPublic
Actions

Authored by epriestley on Jan 12 2017, 11:53 PM.
Tags
None
Referenced Files
F15580034: D17200.id41353.diff
Wed, May 7, 3:05 AM
F15551885: D17200.id41353.diff
Sun, Apr 27, 7:08 PM
F15540439: D17200.id.diff
Fri, Apr 25, 7:35 AM
F15538162: D17200.diff
Thu, Apr 24, 5:42 PM
F15526343: D17200.id41353.diff
Mon, Apr 21, 8:37 PM
F15516306: D17200.id41361.diff
Fri, Apr 18, 5:56 PM
F15490378: D17200.diff
Fri, Apr 11, 4:47 PM
F15483096: D17200.diff
Wed, Apr 9, 9:55 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T9640: Make Phabricator compatible with PHP7
Commits
rPHU9d85dfab0f53: Where available, use async_signals in libphutil
Summary

Ref T9640. In libphutil, switches to the PHP 7.1 async_signals mechanism when it is available.

Test Plan

With the next change, ran daemons with bin/phd start and bin/phd debug, then ^C'd them.

  • Before patch: long hang.
  • After patch: They responded to the signals normally and exited promptly.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
scripts/
6 lines
daemon/
exec/
6 lines
6 lines

Diff 41361

View Options

scripts/__init_script__.php

Loading...
View Options

scripts/daemon/exec/exec_daemon.php

Loading...
View Options

scripts/daemon/launch_daemon.php

Loading...
Phabricator · Built by Phacility