[Buildroot] [PATCH 08/20] system: with no init system, only allow custom skeleton

[Yann E. MORIN]

Ranout, All,

On 2017-07-22 15:28 +0200, Arnout Vandecappelle spake thusly:
> On 18-07-17 19:25, Yann E. MORIN wrote:
> > When there is no init system (i.e. a custom one), our bundled default
> > skeleton is most probably un-fit for that (non-)init system.
> > 
> > This will be the case when we introduce per init system skeletons. The
> > systemd skeleton will be unfit except for a systemd init; the sysv
> > skeleton will be unfit except for a sysv-like init system.
> > 
> > In case they are using no init system (really, a custom one), the user
> > should be responsible for providing their own, custom skeleton as well.
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> 
>  NACK on this one. Our skeleton (as in, the contents of system/skeleton) is
> already init-agnostic. If you use another init system (shell script, or no init
> at all because you just use it as a chroot, or whatever), you will *still* need
> that skelecton. Indeed, the skeleton contains things like the /sys and /dev
> directories, a passwd file, ....
> 
>  If there are things in that skeleton that you don't need for a custom init
> system, then they probably shouldn't even be in the skeleton.

This patch is reverted later in the series, once all the types of
skeleton are in place. I could not see how to do it sanely and cleanly
otherwise.

Regards,
Yann E. MORIN.

>  Regards,
>  Arnout
> 
> > ---
> >  system/Config.in | 6 ++++++
> >  1 file changed, 6 insertions(+)
> > 
> > diff --git a/system/Config.in b/system/Config.in
> > index 828df4217e..ed539dcbe0 100644
> > --- a/system/Config.in
> > +++ b/system/Config.in
> > @@ -3,11 +3,17 @@ menu "System configuration"
> >  choice
> >  	prompt "Root FS skeleton"
> >  
> > +# When not using an init system (i.e. a custom one), our default skeleton
> > +# is most probably unfit; so, allow only the custom skeleton in that case.
> >  config BR2_ROOTFS_SKELETON_DEFAULT
> >  	bool "default target skeleton"
> > +	depends on !BR2_INIT_NONE
> >  	help
> >  	  Use default target skeleton
> >  
> > +comment "default target skeleton needs an init system"
> > +	depends on BR2_INIT_NONE
> > +
> >  config BR2_ROOTFS_SKELETON_CUSTOM
> >  	bool "custom target skeleton"
> >  	help
> > 
> 
> -- 
> Arnout Vandecappelle                          arnout at mind be
> Senior Embedded Software Architect            +32-16-286500
> Essensium/Mind                                http://www.mind.be
> G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
> LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
> GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'