GNU bug report logs - #37412
[PATCH 0/2] Add package and service for the Guix Data Service.

Previous Next

Package: guix-patches;

Reported by: Christopher Baines <mail <at> cbaines.net>

Date: Sun, 15 Sep 2019 18:19:02 UTC

Severity: normal

Tags: patch

Done: Christopher Baines <mail <at> cbaines.net>

Bug is archived. No further changes may be made.

Full log


Message #47 received at 37412 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 37412 <at> debbugs.gnu.org
Subject: Re: [bug#37412] [PATCH 1/2] gnu: Add guix-data-service.
Date: Sun, 22 Sep 2019 23:11:16 +0200
Christopher Baines <mail <at> cbaines.net> skribis:

> * gnu/packages/web.scm (guix-data-service): New variable.

[...]

> +      (native-inputs
> +       `(("autoconf" ,autoconf)
> +         ("automake" ,automake)
> +         ("emacs-with-modules" ,(directory-union
> +                                 "emacs-union"
> +                                 (list emacs-no-x
> +                                       emacs-htmlize)))

Should it be ‘emacs-minimal’?

> +      (native-search-paths
> +       ;; XXX guile-git requires this to be set, maybe there's a better way
> +       (list (search-path-specification
> +              (variable "GIT_SSL_CAINFO")
> +              (file-type 'regular)
> +              (separator #f)
> +              (files '("etc/ssl/certs/ca-certificates.crt")))))

Like I wrote, I’d rather remove the search path and add the environment
variable…

> +      (start #~(make-forkexec-constructor
> +                (list #$(file-append package
> +                                     "/bin/guix-data-service")
> +                      "--pid-file=/var/run/guix-data-service/pid"
> +                      #$(string-append "--port=" (number->string port))
> +                      #$(string-append "--host=" host)
> +                      ;; Perform any database migrations when the
> +                      ;; service is started
> +                      "--update-database")
> +
> +                #:user #$user
> +                #:group #$group
> +                #:pid-file "/var/run/guix-data-service/pid"
> +                ;; Allow time for migrations to run
> +                #:pid-file-timeout 60
> +                #:environment-variables
> +                `(,(string-append
> +                    "GUIX_LOCPATH=" #$glibc-utf8-locales "/lib/locale")
> +                  "LC_ALL=en_US.utf8")
> +                #:log-file "/var/log/guix-data-service/web.log"))

… here.           ↑

But anyway, these are details so I think you can go ahead and push as
you see fit.

Thanks!

Ludo’.




This bug report was last modified 5 years and 243 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.