diff options
| author | regnat <rg@regnat.ovh> | 2021-10-25 15:53:01 +0200 |
|---|---|---|
| committer | regnat <rg@regnat.ovh> | 2021-10-26 07:02:31 +0200 |
| commit | af99941279b80c962ec9cae3e5fa32976a3f5744 (patch) | |
| tree | 034ffb01457354b07e9b6dac02892e1b5d8fca09 /src/libstore/daemon.cc | |
| parent | 4a2b7cc68c7d11ec126bc412ffea838e629345af (diff) | |
Make experimental-features a proper type
Rather than having them plain strings scattered through the whole
codebase, create an enum containing all the known experimental features.
This means that
- Nix can now `warn` when an unkwown experimental feature is passed
(making it much nicer to spot typos and spot deprecated features)
- It’s now easy to remove a feature altogether (once the feature isn’t
experimental anymore or is dropped) by just removing the field for the
enum and letting the compiler point us to all the now invalid usages
of it.
Diffstat (limited to 'src/libstore/daemon.cc')
| -rw-r--r-- | src/libstore/daemon.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libstore/daemon.cc b/src/libstore/daemon.cc index 164a9b2be..4e8a3c417 100644 --- a/src/libstore/daemon.cc +++ b/src/libstore/daemon.cc @@ -230,7 +230,7 @@ struct ClientSettings else if (name == settings.experimentalFeatures.name) { // We don’t want to forward the experimental features to // the daemon, as that could cause some pretty weird stuff - if (tokenizeString<Strings>(value) != settings.experimentalFeatures.get()) + if (parseFeatures(tokenizeString<StringSet>(value)) != settings.experimentalFeatures.get()) debug("Ignoring the client-specified experimental features"); } else if (trusted |