[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#63985] [PATCH v3 00/11] Service subsystem improvements
From: |
Bruno Victal |
Subject: |
[bug#63985] [PATCH v3 00/11] Service subsystem improvements |
Date: |
Mon, 26 Jun 2023 22:57:27 +0100 |
This patch-series is an agglomeration of smaller but weakly-related
patch-series, done so in order to build up the case for the changes.
I structured the series in order to make it suitable for cherry-picking.
Summary of changes:
* Plumbing changes to serialize-configuration
By orienting it around SRFI-171 transducers, it's now easier
to build custom configuration serializing procedures.
* New define-configuration syntax: serializer-options
Serializers may now accept more than two arguments.
* New module (gnu services configuration generic-ini)
* Deduplicate often used predicates.
* network-manager-service-type cleanup & new fields.
Notable changes since v2:
* Documentation changes
The documentation for define-configuration was reworded to make later
syntax extensions easier to document.
In addition, the new 'serializer-options' is now documented.
* (gnu services configuration) changes
** New predicates.
Reduce code duplication by migrating some commonly used predicates into
this module.
* generic-ini changes
** Initial field testing
Some deficiencies were found & corrected while doing a first field
testing when network-manager-service-type was refactored to make use
of this module.
** generic-ini- serializers
** Handling of multiple entries
Implemented as a transducer, this is useful to deal with escape-hatch
fields.
* network-manager-service-type changes
** Use define-configuration for <network-manager-configuration>
** Refactored serialization process to use the new generic-ini module
** New configuration fields: log-configuration and extra-options
Provides an escape hatch for options not yet implemented in the
record type.
** Renamed fields: 'network-manager' to 'package'
Naming the field 'package' is more informative & less confusing than a
reduplication of the package-name with the field-name itself.
Omissions in generic-ini:
For now, I've omitted:
* Custom leading (presumed to be whitespace) characters for entries
à la gitconfig, mostly beautifying purposes
* Configurable delimiter (\n, \r\n, \0, ...)
* Configurable Key-value separator (this is usually =)
These can be implemented later if required.
Notes:
The interface in (gnu services configuration generic-ini) is still in its
infancy and might require further adjustments/additions and I'm still
thinking about its potential generalizations to TOML & co.
For the time being I'd prefer not to promise any interface stability.
Nice to haves:
I didn't have much luck in replacing the memq approach in the predicate
with define-enumeration:
--8<---------------cut here---------------start------------->8---
;; This works.
(define (network-manager-log-level? x)
(memq x '(off err warn info debug trace)))
;; This does not.
(define-enumeration network-manager-log-level?
(off err warn info debug trace)
network-manager-log-level-set)
;; While executing meta-command:
;; ERROR:
;; 1. &origin: "network-manager-log-level?"
;; 2. &message: "not a member of the set"
;; 3. &syntax:
;; form: #f
;; subform: #f
--8<---------------cut here---------------end--------------->8---
Would be nice to know what went wrong and whether an enumeration could
be used here instead.
Bruno Victal (11):
services: configuration: Simplify normalize-extra-args.
services: configuration: Use transducers within
serialize-configuration.
services: fstrim-service-type: Serialize with SRFI-171 transducers.
doc: Rewrite define-configuration.
services: configuration: Add serializer-options field.
services: configuration: New generic-ini module.
services: configuration: Add some commonly used predicates.
services: NetworkManager: Use define-configuration and generic-ini.
services: NetworkManager: Prefer package over network-manager.
services: NetworkManager: add log-configuration field.
services: NetworkManager: Add extra-options field.
Makefile.am | 1 +
doc/guix.texi | 161 +++++----
gnu/local.mk | 1 +
gnu/services/audio.scm | 7 +-
gnu/services/configuration.scm | 108 ++++--
gnu/services/configuration/generic-ini.scm | 165 +++++++++
gnu/services/linux.scm | 11 +-
gnu/services/networking.scm | 352 ++++++++++++++-----
gnu/services/telephony.scm | 49 ++-
tests/services/configuration.scm | 88 ++++-
tests/services/configuration/generic-ini.scm | 129 +++++++
11 files changed, 861 insertions(+), 211 deletions(-)
create mode 100644 gnu/services/configuration/generic-ini.scm
create mode 100644 tests/services/configuration/generic-ini.scm
base-commit: ac86174e22fcd762893bd4515786b1376af9397b
--
2.39.2
- [bug#63985] [PATCH RFC 1/5] services: configuration: Simplify normalize-extra-args., (continued)
- [bug#63985] [PATCH RFC 1/5] services: configuration: Simplify normalize-extra-args., Bruno Victal, 2023/06/09
- [bug#63985] [PATCH RFC 2/5] services: configuration: Use transducers within serialize-configuration., Bruno Victal, 2023/06/09
- [bug#63985] [PATCH RFC 3/5] services: fstrim-service-type: Serialize with SRFI-171 transducers., Bruno Victal, 2023/06/09
- [bug#63985] [PATCH RFC 5/5] services: configuration: New generic-ini module., Bruno Victal, 2023/06/09
- [bug#63985] [PATCH RFC 4/5] services: configuration: Add serializer-kwargs field., Bruno Victal, 2023/06/09
- [bug#63985] [PATCH RFC v2 1/5] services: configuration: Simplify normalize-extra-args., Bruno Victal, 2023/06/10
- [bug#63985] [PATCH RFC v2 3/5] services: fstrim-service-type: Serialize with SRFI-171 transducers., Bruno Victal, 2023/06/10
- [bug#63985] [PATCH RFC v2 2/5] services: configuration: Use transducers within serialize-configuration., Bruno Victal, 2023/06/10
- [bug#63985] [PATCH RFC v2 4/5] services: configuration: Add serializer-options field., Bruno Victal, 2023/06/10
- [bug#63985] [PATCH RFC v2 5/5] services: configuration: New generic-ini module., Bruno Victal, 2023/06/10
- [bug#63985] [PATCH v3 00/11] Service subsystem improvements,
Bruno Victal <=
- [bug#63985] [PATCH v3 06/11] services: configuration: New generic-ini module., Bruno Victal, 2023/06/26
- [bug#63985] [PATCH v3 11/11] services: NetworkManager: Add extra-options field., Bruno Victal, 2023/06/26
- [bug#63985] [PATCH v3 07/11] services: configuration: Add some commonly used predicates., Bruno Victal, 2023/06/26
- [bug#63985] [PATCH v3 05/11] services: configuration: Add serializer-options field., Bruno Victal, 2023/06/26
- [bug#63985] [PATCH v3 10/11] services: NetworkManager: add log-configuration field., Bruno Victal, 2023/06/26
- [bug#63985] [PATCH v3 09/11] services: NetworkManager: Prefer package over network-manager., Bruno Victal, 2023/06/26
- [bug#63985] [PATCH v3 03/11] services: fstrim-service-type: Serialize with SRFI-171 transducers., Bruno Victal, 2023/06/26
- [bug#63985] [PATCH v3 08/11] services: NetworkManager: Use define-configuration and generic-ini., Bruno Victal, 2023/06/26
- [bug#63985] [PATCH v3 04/11] doc: Rewrite define-configuration., Bruno Victal, 2023/06/26
- [bug#63985] [PATCH v3 02/11] services: configuration: Use transducers within serialize-configuration., Bruno Victal, 2023/06/26