HomePhorge

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

Description

Correct notification.pid to notification.pidfile

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.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D11768

Details

Provenance
Elias ProbstAuthored on Feb 14 2015, 2:33 PM
epriestleyCommitted on Feb 14 2015, 2:33 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP05377ef48c7a: Expand Subscription handles slightly
Branches
Unknown
Tags
Unknown

Event Timeline