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
F13078206: D17200.diff
Tue, Apr 23, 11:50 AM
Unknown Object (File)
Fri, Apr 19, 6:05 PM
Unknown Object (File)
Wed, Apr 17, 3:05 PM
Unknown Object (File)
Thu, Apr 11, 7:15 AM
Unknown Object (File)
Sun, Apr 7, 12:13 PM
Unknown Object (File)
Thu, Mar 28, 9:19 AM
Unknown Object (File)
Mon, Mar 25, 1:53 PM
Unknown Object (File)
Mon, Mar 25, 9:57 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