bug#65731: closed ([PATCH] services: syncthing: ensure that service runs after mounting home directories)

[GNU bug Tracking System]

Your message dated Sun, 17 Sep 2023 11:56:22 +0200
with message-id <87cyyh15tl.fsf_-_@gnu.org>
and subject line Re: bug#65731: [PATCH] services: syncthing: ensure that 
service runs after mounting home directories
has caused the debbugs.gnu.org bug report #65731,
regarding [PATCH] services: syncthing: ensure that service runs after mounting 
home directories
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
65731: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=65731
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: [PATCH] services: syncthing: ensure that service runs after mounting home directories Date: Mon, 04 Sep 2023 14:25:23 +0200

Sometimes `syncthing-service-type' fails during startup because it tries
to read configuration files from the user's home directory:

Failure on home directory: mkdir /home/xyz/.config: permission denied

This patch adds `user-processes' to the shepherd service requirements to
ensure that `user-homes' is fired before `syncthing' tries to read data
from the home directory.

* gnu/services/syncthing.scm (syncthing-shepherd-service): add
`user-processes' to requirements

---
 gnu/services/syncthing.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/services/syncthing.scm b/gnu/services/syncthing.scm
index 2f7c822827..9bb623186b 100644
--- a/gnu/services/syncthing.scm
+++ b/gnu/services/syncthing.scm
@@ -63,7 +63,7 @@ (define syncthing-shepherd-service
                      (list (string->symbol
                             (string-append "syncthing-" user)))))
       (documentation "Run syncthing.")
-      (requirement (if home-service? '() '(loopback)))
+      (requirement (if home-service? '() '(loopback user-processes)))
       (start #~(make-forkexec-constructor
                 (append (list (string-append #$syncthing "/bin/syncthing")
                               "--no-browser"
-- 
2.41.0

signature.asc
Description: PGP signature

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#65731: [PATCH] services: syncthing: ensure that service runs after mounting home directories
>
>
>
>
> Date: 
>
> Sun, 17 Sep 2023 11:56:22 +0200
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)
>
>
>
>
> Hi,
>
> Adam Maleszka <adam_maleszka@aol.com> skribis:
>
> > Sometimes `syncthing-service-type' fails during startup because it tries
> > to read configuration files from the user's home directory:
> >
> > Failure on home directory: mkdir /home/xyz/.config: permission denied
> >
> > This patch adds `user-processes' to the shepherd service requirements to
> > ensure that `user-homes' is fired before `syncthing' tries to read data
> > from the home directory.
> >
> > * gnu/services/syncthing.scm (syncthing-shepherd-service): add
> > `user-processes' to requirements
>
> Applied, thanks!
>
> Ludo’.
>
>
> --- End Message ---