From a8cc1c05e528656216958f9ef1361ebc42a93982 Mon Sep 17 00:00:00 2001 From: Elias Probst Date: Sat, 14 Feb 2015 14:33:03 -0800 Subject: [PATCH] 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 --- src/docs/user/configuration/notifications.diviner | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/docs/user/configuration/notifications.diviner b/src/docs/user/configuration/notifications.diviner index ae973f9938..dda3c1e0af 100644 --- a/src/docs/user/configuration/notifications.diviner +++ b/src/docs/user/configuration/notifications.diviner @@ -80,7 +80,7 @@ You may also want to adjust these settings: - `notification.server-uri` Internally-facing host and port that Phabricator will connect to in order to publish notifications. - `notification.log` Log file location for the server. - - `notification.pid` Pidfile location used to stop any running server when + - `notification.pidfile` Pidfile location used to stop any running server when aphlict is restarted.