[Buildroot] [PATCH-NEXT 2/2] chrony: S49chrony: drop /etc/chrony.conf check

Peter Korsgaard peter at korsgaard.com
Mon May 28 18:21:19 UTC 2018


>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni at bootlin.com> writes:

 > Hello,
 > On Mon, 28 May 2018 15:22:58 +0200, Peter Korsgaard wrote:
 >> With CHRONY_ARGS in /etc/default/chrony, chrony may asked to use an
 >> alternative configuration file (with the -f option), so drop the explicit
 >> /etc/chrony.conf check.
 >> 
 >> In case the configuration file isn't present chrony will anyway loudly
 >> complain which is preferable to this silent failure:
 >> 
 >> chronyd -f /non/existing/file.conf
 >> Could not open configuration file /non/existing/file.conf : No such file or directory
 >> 
 >> Signed-off-by: Peter Korsgaard <peter at korsgaard.com>

 > Acked-by: Thomas Petazzoni <thomas.petazzoni at bootlin.com>

 > I really hate those stupid checks, they cause more harm than good. So
 > it's great to remove those checks.

Yes, indeed. Committed both to next, thanks.

-- 
Bye, Peter Korsgaard



More information about the buildroot mailing list