[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: services: Add mysql-service.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH] gnu: services: Add mysql-service. |
Date: |
Wed, 15 Jun 2016 15:09:46 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Hello!
宋文武 <address@hidden> skribis:
> * gnu/services/database.scm (<mysql-configuration>): New record type.
> (%mysql-accounts, mysql-service-type): New variables.
> (mysql-configuration-file, %mysql-activation, mysql-shepherd-services)
> (mysql-services): New procedures.
> * doc/guix.texi (Database Services): Document it.
[...]
> address@hidden {Scheme Procedure} mysql-service [#:config
> (mysql-configuration)]
> +Return a service that runs @command{mysqld}, the MySQL database server.
s/MySQL/MySQL or MariaDB/
> address@hidden {Data Type} mysql-configuration
> +Data type representing the configuration of @var{mysql-service}.
> +
> address@hidden @asis
> address@hidden @code{mysql} (default: @var{mariadb})
> +Package object of the MySQL database server, can be either @var{mariadb}
> +or @var{mysql}.
> address@hidden table
> address@hidden deftp
Do you have plans to extend this type eventually?
> +(define (%mysql-activation config)
Please add a docstring. IIUC it initializes the ‘mysql’ database for
user ‘mysql’, and that database contains configuration info, right?
Otherwise LGTM!
If you want, it would be awesome if you could come up with a (gnu tests
databases) module that would run a GuixSD with the mysql service, and
then spawn a mysql client to make sure the basics work as expected.
Thank you for working on it!
Ludo’.