[Buildroot] [External] Re: [PATCH v2] package/restorecond: Add new init script

[Weber, Matthew L Collins]

Yann / Jose,

> From: buildroot <buildroot-bounces at busybox.net> on behalf of José Pekkarinen <jose.pekkarinen at unikie.com>
> Sent: Monday, August 23, 2021 12:45 AM
> To: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
> Cc: buildroot at buildroot.org <buildroot at buildroot.org>
> Subject: [External] Re: [Buildroot] [PATCH v2] package/restorecond: Add new init script
>
> On Sat, Aug 21, 2021 at 12:36 AM Thomas Petazzoni <thomas.petazzoni at bootlin.com> wrote:
> Hello José,
>
> On Fri, 20 Aug 2021 14:23:22 +0300
> José Pekkarinen <jose.pekkarinen at unikie.com> wrote:
>
> > The current restorecond upstream init script is no
> > good fit for the user space generated by buildroot,
> > this script is an extension of the original, that
> > brings some changes from the debian init script to
> > use start-stop-daemon instead of daemon, while
> > removing dependencies on /etc/rc.d/init.d/functions
> > and /lib/lsb/init-functions.

Oops, I was reviewing patches this morning and missed this one when I responded to the "autorelabel" email thread.

I think the example script I referenced in that email is still the most basic way to do the boot fixup, however, it would be worth going through the upstream scripts/daemons to see if they are viable and fit the general criteria Buildroot projects might desire.  Some things that come to mind the upstream scripts would need to consider.
- .autorelabel is different from handling all our memory-based filesystems on each boot.  Does "fixfiles" cover this without requiring a .autorelabel set?
- No impact to boot time or requirement to run on first boot
- read-only filesystem support (may need to tweak the scripts)
- Additional Buildroot package dependencies to support "fixfiles" as is.  Can we make the selection of this conditional on using ".autorelabel"?

Regards,
Matt