[PATCH] config: PID_FILE_PATH required for FEATURE_CROND_SPECIAL_TIMES
Markus Gothe
nietzsche at lysator.liu.se
Tue Aug 27 18:50:28 UTC 2019
Having a quick look, I'd say it introduces an unexpected behaviour at best.
I can in no way personally see any benefits with the patch, since most sane users run services supervised and actually want a pidfile. The rare exception is if you don't know about "runit" or have really small RAM.
If a change is only good for a special purpose; then it would shouldn't be upstreamed imho.
//M
Sent from my BlackBerry — the most secure mobile device
Original Message
From: james.byrne at origamienergy.com
Sent: 27 August 2019 15:51
To: busybox at busybox.net
Subject: Re: [PATCH] config: PID_FILE_PATH required for FEATURE_CROND_SPECIAL_TIMES
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 <james.byrne at origamienergy.com>
Hi,
I posted this patch a while back but haven't seen any replies. Are there
any issues with this change?
Thanks,
James
_______________________________________________
busybox mailing list
busybox at busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
More information about the busybox
mailing list