From 03fc9c723cfc59467a7fccc305f34273f8564b25 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Wed, 7 Dec 2016 19:03:22 +0100 Subject: [PATCH] core: add a note clarifying that we should be careful when adding new specifiers --- src/core/unit-printf.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/core/unit-printf.c b/src/core/unit-printf.c index 8f7eb84c61..746e1a46ef 100644 --- a/src/core/unit-printf.c +++ b/src/core/unit-printf.c @@ -257,6 +257,10 @@ int unit_full_printf(Unit *u, const char *format, char **ret) { * %s: the shell of the running user * * %v: `uname -r` of the running system + * + * NOTICE: When you add new entries here, please be careful: specifiers which depend on settings of the unit + * file itself are broken by design, as they would resolve differently depending on whether they are used + * before or after the relevant configuration setting. Hence: don't add them. */ const Specifier table[] = {