diff mbox series

[bug#49650] services: dovecot: Fix "/var/run/dovecot" owner.

Message ID 20210719211943.23824-1-brice@waegenei.re
State New
Headers show
Series [bug#49650] services: dovecot: Fix "/var/run/dovecot" owner. | expand

Checks

Context Check Description
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch success View Laminar job
cbaines/issue success View issue

Commit Message

Brice Waegeneire July 19, 2021, 9:19 p.m. UTC 
* gnu/services/mail.scm (%dovecot-activation)[mkdir-p/perms]: Change
  owner of "directory" not the static string "/var/run/dovecot".
---
 gnu/services/mail.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Following a discussion on IRC¹. There are several other bug in that service
but those should only manifest when using non default configuration, for
example using a user other that "dovecot" or a run time directory other than
"/var/run/dovevot", etc...

¹ <http://logs.guix.gnu.org/guix/2021-07-17.log#151005>
diff mbox series

Patch

diff --git a/gnu/services/mail.scm b/gnu/services/mail.scm
index 72dc123f41..805f4ec864 100644
--- a/gnu/services/mail.scm
+++ b/gnu/services/mail.scm
@@ -1515,7 +1515,7 @@  greyed out, instead of only later giving \"not selectable\" popup error.
         (use-modules (guix build utils))
         (define (mkdir-p/perms directory owner perms)
           (mkdir-p directory)
-          (chown "/var/run/dovecot" (passwd:uid owner) (passwd:gid owner))
+          (chown directory (passwd:uid owner) (passwd:gid owner))
           (chmod directory perms))
         (define (build-subject parameters)
           (string-concatenate