[bug#43816] [PATCH] services: gitolite: Install the rc file as gitolite's user.

[bug#43816] [PATCH] services: gitolite: Install the rc file as gitolite's user.

[Julien Lepiller]

[-- Attachment #1: Type: text/plain, Size: 227 bytes --]

Hi Guix!

I had some issues when installing my gitolite service. Turns out the
.gitolite.rc file, that contains gitolite's configuration, was owned by
root, with no access to others. This patch fixes that by using chown on
it.

[-- Attachment #2: 0001-services-gitolite-Install-the-rc-file-as-gitolite-s-.patch --]
[-- Type: text/x-patch, Size: 1541 bytes --]

From abaac031616bbae42c455fdbe44b680e7da93d44 Mon Sep 17 00:00:00 2001
From: Julien Lepiller <julien@lepiller.eu>
Date: Mon, 5 Oct 2020 17:47:13 +0200
Subject: [PATCH] services: gitolite: Install the rc file as gitolite's user.

* gnu/services/version-control.scm (gitolite-activation): Install the rc
file as gitolite's user.
---
 gnu/services/version-control.scm | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/gnu/services/version-control.scm b/gnu/services/version-control.scm
index cc07f8025b..f3df0b979f 100644
--- a/gnu/services/version-control.scm
+++ b/gnu/services/version-control.scm
@@ -307,10 +307,15 @@ access to exported repositories under @file{/srv/git}."
                 (pubkey-file (string-append
                               #$home "/"
                               (basename
-                               (strip-store-file-name admin-pubkey)))))
+                               (strip-store-file-name admin-pubkey))))
+                (rc-file #$(string-append home "/.gitolite.rc")))
 
            (simple-format #t "guix: gitolite: installing ~A\n" #$rc-file)
-           (copy-file #$rc-file #$(string-append home "/.gitolite.rc"))
+           (copy-file #$rc-file rc-file)
+           ;; ensure gitolite's user can read the configuration
+           (chown rc-file
+                  (passwd:uid user-info)
+                  (passwd:gid user-info))
 
            ;; The key must be writable, so copy it from the store
            (copy-file admin-pubkey pubkey-file)
-- 
2.28.0

[bug#43816] [PATCH] services: gitolite: Install the rc file as gitolite's user.

[Christopher Baines]

[-- Attachment #1: Type: text/plain, Size: 332 bytes --]


Julien Lepiller <julien@lepiller.eu> writes:

> Hi Guix!
>
> I had some issues when installing my gitolite service. Turns out the
> .gitolite.rc file, that contains gitolite's configuration, was owned by
> root, with no access to others. This patch fixes that by using chown on
> it.

As mentioned on IRC, this looks good to me :)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

bug#43816: [PATCH] services: gitolite: Install the rc file as gitolite's user.

[Julien Lepiller]

Thank you, pushed as 4e3ed9bad9ed5758cdee6e636805f65e9ab710eb.