config: PID_FILE_PATH required for FEATURE_CROND_SPECIAL_TIMES
authorJames Byrne <james.byrne@origamienergy.com>
Wed, 17 Jul 2019 15:53:04 +0000 (15:53 +0000)
committerDenys Vlasenko <vda.linux@googlemail.com>
Mon, 14 Oct 2019 10:48:38 +0000 (12:48 +0200)
When crond is built with FEATURE_CROND_SPECIAL_TIMES enabled, it creates
a file called 'crond.reboot' at CONFIG_PID_FILE_PATH, but if
FEATURE_PIDFILE is disabled, this will be an empty string and the file
will be created in the root directory, which is undesirable.

This commit makes PID_FILE_PATH depend on FEATURE_CROND_SPECIAL_TIMES as
well as FEATURE_PIDFILE so that you get sensible behaviour in crond when
FEATURE_PIDFILE is switched off.

Signed-off-by: James Byrne <james.byrne@origamienergy.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Config.in

index 14f54aacca1765d98acb55eca42715a9202d786a..b62b2abb8c339d1942c6bf2b166fdfd52f200af1 100644 (file)
--- a/Config.in
+++ b/Config.in
@@ -156,12 +156,13 @@ config FEATURE_PIDFILE
 config PID_FILE_PATH
        string "Directory for pidfiles"
        default "/var/run"
-       depends on FEATURE_PIDFILE
+       depends on FEATURE_PIDFILE || FEATURE_CROND_SPECIAL_TIMES
        help
        This is the default path where pidfiles are created.  Applets which
        allow you to set the pidfile path on the command line will override
        this value.  The option has no effect on applets that require you to
-       specify a pidfile path.
+       specify a pidfile path.  When crond has the 'Support special times'
+       option enabled, the 'crond.reboot' file is also stored here.
 
 config BUSYBOX
        bool "Include busybox applet"