[systemd-devel] [PATCH 3/3] run: introduce timer support option

Lennart Poettering lennart at poettering.net
Tue Oct 28 03:22:58 PDT 2014


On Tue, 28.10.14 12:09, WaLyong Cho (walyong.cho at samsung.com) wrote:

> > The AccuracySec= and WakeSystem= stuff I think we don't need to cover
> > with a command line argument of its own, we can cover that with
> > --property=.
> > 
> Rework is almost done. Now I'm testing. But the AccuracySec= and
> WakeSystem= options are hard to deal with --property= option. Currently
> the --property= option of systemd-run is designed for service or scope
> unit. If that option is specified then given string is parsed again by
> bus_append_unit_property_assignment(). We can AccuracySec=/WakeSystem=
> option to there. But we have to split given properties(by --property
> option) according to which unit can have that properties.

Hmm, the implied rule here is that we do not intrdouce properties with
the same name on different unit types that do not shrae the same
signature. That way it really doesn't matter to specify which
properties can be parsed for a specific unit type on the client side,
since the server side will already know.

Or in other words: go ahead, add this to
bus_append_unit_property_assignment(), and the server side will deny
what it doesn't like.

Hope that makes sense,

Lennart

-- 
Lennart Poettering, Red Hat


More information about the systemd-devel mailing list