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
F15490378: D17200.diff
Fri, Apr 11, 4:47 PM
F15483096: D17200.diff
Wed, Apr 9, 9:55 AM
F15473915: D17200.id41361.diff
Sun, Apr 6, 3:23 AM
F15473004: D17200.id41353.diff
Sat, Apr 5, 6:37 PM
F15447745: D17200.id.diff
Fri, Mar 28, 1:20 AM
F15442554: D17200.id41361.diff
Wed, Mar 26, 11:48 PM
F15442552: D17200.id41353.diff
Wed, Mar 26, 11:48 PM
F15442551: D17200.id.diff
Wed, Mar 26, 11:48 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