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

[José Pekkarinen]

On Mon, Aug 23, 2021 at 6:10 PM Weber, Matthew L Collins <
Matthew.Weber at collins.com> wrote:

> 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"?
>

Hi,

To ease the reading/tracking of the script

Matthew was refering to, here is the link.

[1]
https://patchwork.ozlabs.org/project/buildroot/patch/1458128701-14841-1-git-send-email-niranjan.reddy@rockwellcollins.com/

Thanks!


José.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.buildroot.org/pipermail/buildroot/attachments/20210824/a97227d7/attachment-0001.html>