Page MenuHomePhabricator

Correct `notification.pid` to `notification.pidfile`
ClosedPublic

Authored by eliaspro on Feb 14 2015, 7:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 1, 1:17 PM
Unknown Object (File)
Dec 9 2022, 12:41 AM
Unknown Object (File)
Dec 8 2022, 10:54 AM
Unknown Object (File)
Dec 8 2022, 10:54 AM
Unknown Object (File)
Dec 8 2022, 10:54 AM
Unknown Object (File)
Nov 20 2022, 8:40 PM
Unknown Object (File)
Nov 16 2022, 1:32 AM
Unknown Object (File)
Nov 15 2022, 11:57 PM
Subscribers

Details

Summary

The correct conf variable is notification.pidfile, not notification.pid.
See also:

  • src/applications/config/option/PhabricatorNotificationConfigOptions.php:58
  • src/applications/aphlict/management/PhabricatorAphlictManagementWorkflow.php:34
Test Plan

Grepped the whole source of Phabricator for occurences of notification.pid,
but all matches pointed to notification.pidfile instead.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4497
Build 4511: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

eliaspro retitled this revision from to Correct `notification.pid` to `notification.pidfile`.
eliaspro updated this object.
eliaspro edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.Feb 14 2015, 10:32 PM
This revision was automatically updated to reflect the committed changes.