guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#37412] [PATCH 2/2] services: Add the Guix Data Service.


From: Ludovic Courtès
Subject: [bug#37412] [PATCH 2/2] services: Add the Guix Data Service.
Date: Mon, 16 Sep 2019 09:53:39 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux)

Christopher Baines <address@hidden> skribis:

> * gnu/services/guix.scm: New file.
> * gnu/tests/guix.scm: New file.
> * gnu/local.mk (GNU_SYSTEM_MODULES): Add both new files.

Yay, nice!

> +  (commits-getmail-retriever-configuration
> +   guix-data-service-commits-getmail-retriever-configuration

New world record: longer than ‘call-with-current-continuation’.  :-)

> +(define (guix-data-service-shepherd-services config)
> +  (match config
> +    (($ <guix-data-service-configuration> package user group
> +                                          port host)

Perhaps use ‘match-record’ here and elsewhere, to avoid mistakes when
the record type changes.

> +(define %test-guix-data-service
> +  (system-test
> +   (name "guix-data-service")
> +   (description "Connect to a running Guix Data Service.")
> +   (value (run-guix-data-service-test))))

Nice.

Could you add a bit to guix.texi, maybe under “Continuous Integration”?
You don’t have to go into too much detail if you think some of the
configuration elements are subject to change.

Thank you!

Ludo’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]