core: drop specifier expansion when parsing transient dbus properties
authorLennart Poettering <lennart@poettering.net>
Tue, 21 Nov 2017 16:32:01 +0000 (17:32 +0100)
committerLennart Poettering <lennart@poettering.net>
Wed, 29 Nov 2017 11:32:53 +0000 (12:32 +0100)
commit6171b8225f468fc90c79087f653ad4fc7ac21453
treeb2c152c12df7c35f5e26adbf3ac7a8edc399d28f
parent3235b4e70cf42cd13fe1ac4f7341b623a852aed9
core: drop specifier expansion when parsing transient dbus properties

Specifier expansion (much like C escape handling) should be a helper for
writing unit files, but should be nothing we do on programatic APIs. For
those, the client can do the necessary replacements anyway, and we
really should be careful with doing such string processing of data we
get via lower level programmatic APIs.

We currently do specifier expansion only for the env var transient unit
APIs, no other properties do this. Let's remove it here too, to be fully
systematic.

Yes, in a way this is API breakage, but then again this API isn't
documented yet, and an outlier, hence let's clear this up now, before it
is too late.
src/core/dbus-execute.c