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
F15413313: D17200.id.diff
Wed, Mar 19, 5:59 PM
F15413285: D17200.id41353.diff
Wed, Mar 19, 5:47 PM
F15413284: D17200.id41361.diff
Wed, Mar 19, 5:47 PM
F15413210: D17200.diff
Wed, Mar 19, 5:30 PM
F15408792: D17200.id41361.diff
Wed, Mar 19, 1:22 AM
F15377713: D17200.id.diff
Thu, Mar 13, 10:23 AM
F15337050: D17200.diff
Sun, Mar 9, 3:43 AM
Unknown Object (File)
Fri, Feb 21, 3:04 PM
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