New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nixos/ddclient: Fix #49258 #49312
nixos/ddclient: Fix #49258 #49312
Conversation
See #49258 |
4b3da28
to
42567d2
Compare
If this change is also relevant to master, please change the target branch. |
42567d2
to
0f0f9c0
Compare
Sorry for the mess with the branches. |
Not to overcomplicate things but iirc there is a |
I think |
0f0f9c0
to
1b4e310
Compare
Sure, whichever you think fits best |
Motivation for this change
from 18.03 to 18.09 the option
services.ddclient.domain
was changed toservices.ddclient.domains
and
services.ddclient.domain
became a rename for theservices.ddclient.domains option
.Sadly that was also a type change from
str
tolistOf str
and so, one can't keep the configurationfrom 18.03 and has to change it, which makes for a rough transition from 18.03 to 18.09.
Otherwise introducing the change would break the configuration for those people,
who changed the type of the value but kept the name
domain
for the configurationoption.
People, which also changed the name of the option from
domain
todomains
will not be affected by the fix and so I think the fix is still a good idea, as most likely
people will also change the name of the option.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)/etc/ddclient.conf
.