[Buildroot] [PATCH 1/1] package/chrony: set the permissions with CHRONY_PERMISSIONS

[Buildroot] [PATCH 1/1] package/chrony: set the permissions with CHRONY_PERMISSIONS

[TIAN Yuanhao]

Fixes:
  chronyd[241]: Could not open /var/lib/chrony/chrony.drift.tmp : Permission denied

Refs:
https://github.com/jens-maus/RaspberryMatic/blob/3.77.7.20240826/buildroot-external/overlay/base/etc/init.d/S46chrony#L44
https://github.com/troglobit/myLinux/blob/5ab2ec53d70904b1c3f2570afaa31eeaa4c4487b/package/skeleton-init-finit/skeleton/etc/tmpfiles.d/chrony.conf

Signed-off-by: TIAN Yuanhao <tianyuanhao3@163.com>
---
 package/chrony/chrony.mk | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/package/chrony/chrony.mk b/package/chrony/chrony.mk
index 9ed9dcff56..11270206bd 100644
--- a/package/chrony/chrony.mk
+++ b/package/chrony/chrony.mk
@@ -24,6 +24,10 @@ define CHRONY_USERS
 	chrony -1 chrony -1 * /run/chrony - - Time daemon
 endef
 
+define CHRONY_PERMISSIONS
+	/var/lib/chrony d 755 chrony chrony - - - - -
+endef
+
 ifeq ($(BR2_PACKAGE_LIBNSS),y)
 CHRONY_DEPENDENCIES += libnss
 else
-- 
2.45.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/1] package/chrony: set the permissions with CHRONY_PERMISSIONS

[Thomas Petazzoni via buildroot]

On Thu, 19 Sep 2024 05:58:20 -0700
TIAN Yuanhao <tianyuanhao3@163.com> wrote:

> Fixes:
>   chronyd[241]: Could not open /var/lib/chrony/chrony.drift.tmp : Permission denied
> 
> Refs:
> https://github.com/jens-maus/RaspberryMatic/blob/3.77.7.20240826/buildroot-external/overlay/base/etc/init.d/S46chrony#L44
> https://github.com/troglobit/myLinux/blob/5ab2ec53d70904b1c3f2570afaa31eeaa4c4487b/package/skeleton-init-finit/skeleton/etc/tmpfiles.d/chrony.conf
> 
> Signed-off-by: TIAN Yuanhao <tianyuanhao3@163.com>
> ---
>  package/chrony/chrony.mk | 4 ++++
>  1 file changed, 4 insertions(+)

Applied to master, thanks. It would be good to add a runtime test in
support/testing/ for chrony.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot