[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openembedded-core
Subject:    Re: [OE-core] [OE-Core][PATCH v5 5/6] systemd-systemctl-native: simplify and support preset-all
From:       Alex Kiernan <alex.kiernan () gmail ! com>
Date:       2019-04-30 10:31:21
Message-ID: CAO5Uq5Rr66_Hho-rdt28EVgDwS28Wx-nTQs6UC5awrLmsuK7jA () mail ! gmail ! com
[Download RAW message or body]

On Mon, Apr 29, 2019 at 3:43 PM Alex Kiernan <alex.kiernan@gmail.com> wrote:
>
> From: Jonas Bonn <jonas@norrbonn.se>
>
> Now that the systemd class sets up service presets instead of actively
> enabling services, the 'enable' and 'disable' subcommands for systemctl
> are not actually used anywhere.  As such, we can remove these to make
> sure that nobody inadvertently introduces new uses of them.
>
> We do, however, one case where the enable/disable machinery is still
> required; that is for the read-only-rootfs case where the 'preset-all'
> command can not be called at runtime but needs to be called when creatng
> the image.  For this case, we implement 'preset-all' here.
>
> There was also a previous implement of 'preset'... not sure that this
> ever worked as the implementation looks bogus and there aren't any users
> anyway.  This patch removes the 'preset' subcommand, as well.
>
> Signed-off-by: Jonas Bonn <jonas@norrbonn.se>
> Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>

So it turns out we need something more sophisticated for the read-only
rootfs case... assuming this series passes CI (and if it got applied),
it's still better than where we were, or I can split it in two, one
which addresses the machine-id changes and another for the read-only
root case.

I'm just rewriting systemctl-native in python and adding in handling
of preset files... the shell's gotten way too knarly!

-- 
Alex Kiernan
-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
[prev in list] [next in list] [prev in thread] [next in thread]