On 27/08/2019 14:51, James Byrne wrote:
On 17/07/2019 16:53, James Byrne wrote:
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 <[email protected]>

Hi,

I posted this patch back in July, and I still think it's worth applying.

Just to be clear about the problem this is solving, if you have FEATURE_CROND_SPECIAL_TIMES enabled and FEATURE_PIDFILE disabled, 'crond.reboot' will be created unexpectedly in the root directory, with no option to put it anywhere else.

Would it be possible to get this applied?

Thanks,

James
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to